Skip to content

fix: allow same-origin PWA manifests in CSP - #2626

Open
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-2045/manifest-directive
Open

fix: allow same-origin PWA manifests in CSP#2626
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-2045/manifest-directive

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

Refs #2045

Summary

  • Add manifest-src 'self' to the API's production CSP default so the same-origin web manifest is permitted.
  • Add API regression coverage requiring the exact same-origin directive while preserving default-src 'none'.

Verification

  • SecurityHeadersApiTests: 7/7
  • Mutation check: removing only manifest-src 'self' made the new test fail because no manifest directive was emitted; configuration was restored and the class returned to 7/7.
  • git diff --check: passed
  • Packaged-browser proof, reverse-proxy parity, and documentation updates: NOT included in this bounded slice.

Two coordinator review passes completed for security semantics and integration/scope.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending

Development

Successfully merging this pull request may close these issues.

1 participant