Skip to content

[supportForOtherContentTypesThanJson.bcheck] Update the match condition.#267

Merged
Hannah-PortSwigger merged 3 commits into
PortSwigger:mainfrom
righettod:main
Jul 7, 2026
Merged

[supportForOtherContentTypesThanJson.bcheck] Update the match condition.#267
Hannah-PortSwigger merged 3 commits into
PortSwigger:mainfrom
righettod:main

Conversation

@righettod

Copy link
Copy Markdown
Contributor

Change

Hi,

This PR update the condition on the HTTP return code to decide if support for other content types is real or not. The goal is to decrease the false positive.

BCheck Contributions

  • BCheck compiles and executes as expected
  • BCheck contains appropriate metadata (name, version, author, description and appropriate tags)
  • Only .bcheck files have been added or modified
  • BCheck is in the appropriate folder
  • PR contains single or limited number of BChecks (Multiple PRs are preferred)
  • BCheck attempts to minimize false positives

@righettod righettod changed the title [supportForOtherContentTypesThanJson.bcheck] Update the match condition [supportForOtherContentTypesThanJson.bcheck] Update the match condition. Jul 6, 2026
@righettod

Copy link
Copy Markdown
Contributor Author

🤔 CI error seems not related to my PR:

image

@Hannah-PortSwigger Hannah-PortSwigger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the update, this change looks good!

@Michelle-PortSwigger Michelle-PortSwigger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good :)

@Hannah-PortSwigger Hannah-PortSwigger merged commit 62f4b2a into PortSwigger:main Jul 7, 2026
2 of 3 checks passed
@righettod

Copy link
Copy Markdown
Contributor Author

Thank you very much 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants