Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding GraphQL Module #153

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

nithisshs
Copy link

  • To Find GET Method Enabled GraphQL
  • Which helps in escalating to CSRF attacks

@Hannah-PortSwigger
Copy link
Contributor

Hi.

Thank you very much for your submission!

Unfortunately, this BCheck is very similar to our existing native GraphQL scan checks.

Have you found that your BCheck returns results above and beyond any native issues?

@nithisshs
Copy link
Author

nithisshs commented Dec 1, 2023

Well it will have detection only based on GET based CSRF attacks on GraphQL

Attaching the following h1 report as a reference

https://hackerone.com/reports/1122408

Copy link
Collaborator

@PortSwiggerWiener PortSwiggerWiener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your submission and your patience.

We've had a discussion internally and decided to accept this even though it duplicates some of the Scanner's capability. The check is nicely written and, as you pointed out, it is more restrained than the scanner's check.

Copy link
Contributor

@ps-porpoise ps-porpoise left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@PortSwiggerWiener PortSwiggerWiener merged commit 6e2864b into PortSwigger:main Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants