Skip to content

fix: prevent clickjacking via X-Frame-Options header (Batch #67)#4157

Open
BossChaos wants to merge 2 commits intoScottcjn:mainfrom
BossChaos:sec-batch67
Open

fix: prevent clickjacking via X-Frame-Options header (Batch #67)#4157
BossChaos wants to merge 2 commits intoScottcjn:mainfrom
BossChaos:sec-batch67

Conversation

@BossChaos
Copy link
Copy Markdown
Contributor

fix: prevent clickjacking via X-Frame-Options header (Batch #67)

  • Replace response.headers.pop('X-Frame-Options') with SAMEORIGIN policy
  • Add Content-Security-Policy frame-ancestors directive
  • Prevent malicious iframe embedding of payment widget
  • Security best practice: allow embedding only from same origin

Co-Authored-By: Hermes Agent hermes@nous.research

BossChaos and others added 2 commits May 5, 2026 02:52
- Replace response.headers.pop('X-Frame-Options') with SAMEORIGIN policy
- Add Content-Security-Policy frame-ancestors directive
- Prevent malicious iframe embedding of payment widget
- Security best practice: allow embedding only from same origin

Co-Authored-By: Hermes Agent <hermes@nous.research>
@BossChaos BossChaos requested a review from Scottcjn as a code owner May 8, 2026 10:52
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) size/XS PR: 1-10 lines BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) security Security-related change ci labels May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) ci security Security-related change size/XS PR: 1-10 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant