-
Notifications
You must be signed in to change notification settings - Fork 2k
JS: Add Permissive CORS query (CWE-942) #14342
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
Merged
erik-krogh
merged 31 commits into
github:main
from
maikypedia:maikypedia/javascript-cors
Jun 28, 2024
Merged
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
e171123
Add initial query for CWE-942
maikypedia 142ab01
Remove comment line
maikypedia 816eebb
Add `.qhelp` and apply some review changes
maikypedia ed06628
Add documentation string for `CorsPermissiveConfiguration`
maikypedia c0e6d7c
Merge branch 'github:main' into maikypedia/javascript-cors
maikypedia 07ad596
Add coverage for `express`
maikypedia acac534
Forgot `.js`
maikypedia d661f7f
Add Flow Labels
maikypedia 413c111
Move to `/experimental`
maikypedia abd53e9
Fix minor issues
maikypedia 4ef4c92
Move Customizations and Query
maikypedia aa24ce5
Apply suggestions from code review
maikypedia bb6ef72
`getArgument` returns `Cors::Cors`
maikypedia f623db4
Change qldoc
maikypedia 3bcb411
Using `Express::RouteSetup`
maikypedia 6a3cdc9
Add `change-node`
maikypedia e6c7fc0
Fixes CI
maikypedia 83cbbd7
Apply docstring changes
maikypedia 87cac2a
Express Argument has to be Cors
maikypedia 4f68f60
Apply review
maikypedia 191766a
Use `config.getCorsConfiguration().getOrigin())`
maikypedia 7662b2b
format
maikypedia 78e7793
Move to experimental
maikypedia 699d8d4
x
maikypedia c1fd7a6
autoformat
erik-krogh f2d6640
fix ambiguous import. It could refer both to a module or a file
erik-krogh cfd7c7a
move change-note to `javascript/ql/src/change-notes`
maikypedia e96c3a3
Move `Apollo` to experimental
maikypedia 4be5cf4
Update javascript/ql/src/experimental/Security/CWE-942/CorsPermissive…
maikypedia 8ba7ac6
Update javascript/ql/src/experimental/Security/CWE-942/CorsPermissive…
maikypedia d0cf2a9
Merge branch 'main' into maikypedia/javascript-cors
maikypedia File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Use
config.getCorsConfiguration().getOrigin())
Co-authored-by: Erik Krogh Kristensen <erik-krogh@github.com>
- Loading branch information
commit 191766a47bf16653231f3dd843568d897878bdb2
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.