Skip to content

Clarify frame-ancestors behavior in nested frames#40886

Merged
bsmth merged 3 commits intomdn:mainfrom
harunurhan:patch-1
Aug 26, 2025
Merged

Clarify frame-ancestors behavior in nested frames#40886
bsmth merged 3 commits intomdn:mainfrom
harunurhan:patch-1

Conversation

@harunurhan
Copy link
Copy Markdown
Contributor

@harunurhan harunurhan commented Aug 26, 2025

Description

Added clarification on the behavior of the frame-ancestors directive in nested frames.

Motivation

While CSP 2 spec is clear about the frame-ancestor behavior when it comes to nested frames, MDN docs don't have any mention of it. This could lead to CSP errors in production if someone sets frame-ancestor directive solely relying on MDN.

Additional details

https://www.w3.org/TR/CSP2/#frame-ancestors-and-frame-options

Related issues and pull requests

Added clarification on the behavior of the frame-ancestors directive in nested frames.
@harunurhan harunurhan requested a review from a team as a code owner August 26, 2025 07:38
@harunurhan harunurhan requested review from bsmth and removed request for a team August 26, 2025 07:38
@github-actions github-actions bot added Content:HTTP HTTP docs size/xs [PR only] 0-5 LoC changed labels Aug 26, 2025
…/frame-ancestors/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 26, 2025

Preview URLs

Flaws (3)

URL: /en-US/docs/Web/HTTP/Reference/Headers/Content-Security-Policy/frame-ancestors
Title: Content-Security-Policy: frame-ancestors directive
Flaw count: 3

  • unknown:
    • No generic content config found
    • no blog root
    • no blog root
External URLs (1)

URL: /en-US/docs/Web/HTTP/Reference/Headers/Content-Security-Policy/frame-ancestors
Title: Content-Security-Policy: frame-ancestors directive

(comment last updated: 2025-08-26 08:53:51)

…/frame-ancestors/index.md

Co-authored-by: Brian Smith <brian@smith.berlin>
Copy link
Copy Markdown
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the clarification! Welcome aboard!

@bsmth bsmth merged commit 39dacf3 into mdn:main Aug 26, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTTP HTTP docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants