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

Also inspect the path for the script tag #1306

Merged
merged 2 commits into from Mar 4, 2019

Conversation

Projects
None yet
2 participants
@fgsch
Copy link
Collaborator

fgsch commented Feb 13, 2019

Part of #1022.

@fgsch fgsch force-pushed the fgsch:xss-url branch from 748cf58 to 5d8546f Feb 14, 2019

@fgsch

This comment has been minimized.

Copy link
Collaborator Author

fgsch commented Feb 14, 2019

Test fixed. While here I've fixed another test.

I've confirmed the new test fails if I remove REQUEST_FILENAME from the rule.

Ready for review.

@fgsch fgsch force-pushed the fgsch:xss-url branch 3 times, most recently from 70ec7c2 to 87aa48a Feb 14, 2019

fgsch added some commits Feb 13, 2019

@dune73

This comment has been minimized.

Copy link
Collaborator

dune73 commented Mar 4, 2019

Thank you for the submission. Merging after conversation in the monthly chat.

@dune73 dune73 merged commit 8f10f5d into SpiderLabs:v3.2/dev Mar 4, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fgsch fgsch deleted the fgsch:xss-url branch Mar 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.