8000 Create DetectCSPReportOnlyHeader.bambda by ctflearner · Pull Request #85 · PortSwigger/bambdas · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Create DetectCSPReportOnlyHeader.bambda #85

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
May 14, 2025
Merged

Create DetectCSPReportOnlyHeader.bambda #85

merged 4 commits into from
May 14, 2025

Conversation

ctflearner
Copy link
Contributor

This script checks if the HTTP response contains the "Content-Security-Policy-Report-Only" header, which is used for monitoring CSP violations without enforcing restrictions.

Bambda Contributions

  • Bambda has a valid header, featuring an @author annotation and suitable description
  • Bambda compiles and executes as expected
  • Only .bambda files have been added or modified (README.md files are automatically updated / generated after PR merge)

@ctflearner
Create DetectCSPReportOnlyHeader.bambda
fde5f3e 
This script checks if the HTTP response contains the "Content-Security-Policy-Report-Only" header, which is used for monitoring CSP violations without enforcing restrictions.
Hannah-PortSwigger
Hannah-PortSwigger requested changes Jan 2, 2025
View reviewed changes
Copy link
Contributor
@Hannah-PortSwigger Hannah-PortSwigger left a comment

Choose a reason for hiding this comment

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

Apologies for the delay over the holiday period, and thank you for your contribution.

Could you please take a look at the feedback points below?

@ctflearner
Copy link
Contributor Author

Apologies for the delay !! I have made the changes kindly check

@Hannah-PortSwigger
Copy link
Contributor

Hi @ctflearner. Thanks for making those changes! We've made a few further edits - would you be able to check that this Bambda is still performing the same behavior as you expect? If the behavior is still the same, we'll go ahead and merge your Bambda into the repository 😊

@ctflearner
Copy link
Contributor Author

Any update ??

@Hannah-PortSwigger
Copy link
Contributor

Hi @ctflearner - was the Bambda still exhibiting the same behavior that you expected with the edits? If so, please let us know and we can adjust the Bambda to the new format and merge it into the repository 🙂

@ctflearner
Copy link
Contributor Author

@Hannah-PortSwigger yeah you can adjust and merge it into the repo

Hannah-PortSwigger
Hannah-PortSwigger approved these changes May 14, 2025
View reviewed changes
Copy link
Contributor
@Hannah-PortSwigger Hannah-PortSwigger left a comment

Choose a reason for hiding this comment

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

Thank you very much for your contribution to the Bambda repo!

@yijun-hu-portswigger yijun-hu-portswigger merged commit a2e45dc into PortSwigger:main May 14, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hannah-PortSwigger Hannah-PortSwigger Hannah-PortSwigger approved these changes

@yijun-hu-portswigger yijun-hu-portswigger yijun-hu-portswigger approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ctflearner @Hannah-PortSwigger @yijun-hu-portswigger
0