Skip to content

Improve security headers based on headers.dev analysis#70

Open
kimpepper wants to merge 2 commits into
mainfrom
feature/improve-security-headers
Open

Improve security headers based on headers.dev analysis#70
kimpepper wants to merge 2 commits into
mainfrom
feature/improve-security-headers

Conversation

@kimpepper
Copy link
Copy Markdown
Contributor

Summary

  • Migrate deprecated Feature-Policy header to Permissions-Policy (new syntax: feature=() instead of feature 'none')
  • Add Cross-Origin-Opener-Policy: same-origin-allow-popups — prevents cross-origin window references while preserving OAuth/payment popup flows
  • Add Cross-Origin-Resource-Policy: same-site — restricts resource embedding to same-site origins (allows CDN subdomains)
  • Add X-Permitted-Cross-Domain-Policies: none — blocks legacy Flash/Adobe cross-domain requests
  • Extend error page Content-Security-Policy with base-uri 'none' and upgrade-insecure-requests

Identified via https://headers.dev/analyze?url=www.previousnext.com.au

Test plan

  • Verify nginx config is valid (nginx -t)
  • Check Permissions-Policy header is present and Feature-Policy is gone
  • Check Cross-Origin-Opener-Policy, Cross-Origin-Resource-Policy, and X-Permitted-Cross-Domain-Policies headers are present
  • Re-run https://headers.dev/analyze?url=www.previousnext.com.au after deploy to confirm warnings resolved

🤖 Generated with Claude Code

kimpepper and others added 2 commits April 8, 2026 12:46
- Migrate deprecated Feature-Policy to Permissions-Policy
- Add Cross-Origin-Opener-Policy (same-origin-allow-popups)
- Add Cross-Origin-Resource-Policy (same-site)
- Add X-Permitted-Cross-Domain-Policies (none)
- Extend error page CSP with base-uri and upgrade-insecure-requests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… header tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimpepper kimpepper requested review from nickschuch and nterbogt April 8, 2026 06:13
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.

1 participant