Skip to content

Comments

CORS Checker workflow improvements#48

Merged
bebiksior merged 1 commit intocaido-community:mainfrom
meme-lord:cors
Dec 8, 2025
Merged

CORS Checker workflow improvements#48
bebiksior merged 1 commit intocaido-community:mainfrom
meme-lord:cors

Conversation

@meme-lord
Copy link
Contributor

  • Excluded a false positive case ( Access-Control-Allow-Origin: * + Access-Control-Allow-Credentials: true is not useful)
  • Improved the title of findings by including whether the misconfig allows creds or not

Please ensure your pull request adheres to the following guidelines:

  • Follow the same folder structure as other workflows (see template).
  • Has a proper author name and workflow description.
  • If using compiled code in JS Nodes, provide the source code for each.
  • If using a 3rd party library, include its license as a comment in the source code.

Thanks for contributing!

… adding whether credentials are allowed to the finding title
@meme-lord meme-lord requested a review from a team as a code owner October 20, 2025 19:10
Copy link
Collaborator

@bebiksior bebiksior left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@bebiksior bebiksior merged commit e5ea4ac into caido-community:main Dec 8, 2025
2 of 3 checks passed
@meme-lord meme-lord deleted the cors branch December 8, 2025 23:32
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.

2 participants