Skip to content

Conversation

samuel871211
Copy link
Contributor

@samuel871211 samuel871211 commented Jun 28, 2025

Description

Motivation

Additional details

Related issues and pull requests

@samuel871211 samuel871211 requested a review from a team as a code owner June 28, 2025 15:32
@samuel871211 samuel871211 requested review from bsmth and removed request for a team June 28, 2025 15:32
@github-actions github-actions bot added Content:HTTP HTTP docs size/xs [PR only] 0-5 LoC changed labels Jun 28, 2025
Copy link
Contributor

github-actions bot commented Jun 28, 2025

Preview URLs

Flaws (3)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/HTTP/Reference/Headers/X-Content-Type-Options
Title: X-Content-Type-Options header
Flaw count: 3

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

URL: /en-US/docs/Web/HTTP/Guides/Cross-Origin_Resource_Policy
Title: Cross-Origin Resource Policy (CORP)


URL: /en-US/docs/Web/HTTP/Reference/Headers/X-Content-Type-Options
Title: X-Content-Type-Options header

(comment last updated: 2025-07-09 07:17:36)

@samuel871211 samuel871211 changed the title doc[cross-origin_resource_policy]: fix broken link fragment doc: fix broken URL fragment Jun 28, 2025
@github-actions github-actions bot added the size/s [PR only] 6-50 LoC changed label Jun 28, 2025
@Josh-Cena
Copy link
Member

CORB is a retired concept that is now non-standard: whatwg/fetch#1441 I'm not sure if we need to mention it

@wbamberg
Copy link
Collaborator

There's a bit of chat about CORB here: #3526, I'm +1 on removing it.

@bsmth
Copy link
Member

bsmth commented Jun 30, 2025

I had a small suggestion not to link the second occurrence, but I agree it's better to remove the references instead.

@samuel871211 samuel871211 changed the title doc: fix broken URL fragment doc: remove CORB section Jul 7, 2025
@samuel871211
Copy link
Contributor Author

@Josh-Cena @wbamberg @bsmth I've removed the section about CORB, you can edit this PR directly.

FYI, Chrome is planning to remove to whole md file of this page: https://www.chromium.org/Home/chromium-security/corb-for-developers/

The discussion can be found here, but it's still ongoing https://issues.chromium.org/issues/428412051

I'm +1 on removing the reference of CORB

@samuel871211 samuel871211 requested a review from bsmth July 8, 2025 14:17
@@ -50,8 +49,6 @@ The concept was originally proposed in 2012 (as `From-Origin`), but [resurrected

In early 2018, two side-channel hardware vulnerabilities known as _Meltdown_ and _Spectre_ were disclosed. These vulnerabilities allowed sensitive data disclosure due to a race condition which arose as part of speculative execution functionality, designed to improve performance.

In response, Chromium shipped [Cross-Origin Read Blocking](https://fetch.spec.whatwg.org/#corb), which automatically protects certain resources (of `Content-Type` HTML, JSON and XML) against cross-origin reads. If the application does not serve a [`no-sniff` directive](/en-US/docs/Web/HTTP/Reference/Headers/X-Content-Type-Options), Chromium will attempt to guess the `Content-Type` and apply the protection anyway.
Copy link
Member

@bsmth bsmth Jul 8, 2025

Choose a reason for hiding this comment

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

I'd suggest changing the History section to be something like:

The concept was originally proposed in 2012 (as a `From-Origin` header), but [resurrected](https://github.com/whatwg/fetch/issues/687) in Q2 of 2018 and implemented in Safari and Chromium.
In early 2018, two side-channel hardware vulnerabilities known as _Meltdown_ and _Spectre_ were disclosed.
These vulnerabilities allowed sensitive data disclosure due to a race condition which arose as part of speculative execution functionality, designed to improve performance.
Cross-Origin Resource Policy was developed as a direct way for sites to block unwanted `no-cors` cross-origin requests.
This is an effective defense against Spectre-like attacks, as the browser strips the body from given responses before an attacker can access them.

References:

Copy link
Contributor Author

@samuel871211 samuel871211 Jul 9, 2025

Choose a reason for hiding this comment

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

Update at b54c980

@samuel871211 samuel871211 requested a review from bsmth July 9, 2025 03:25
Copy link
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. I think this is good now. Does anyone else want a quick look? @wbamberg?

@github-actions github-actions bot removed the size/xs [PR only] 0-5 LoC changed label Jul 9, 2025
@bsmth
Copy link
Member

bsmth commented Jul 10, 2025

OK I'm going to merge this as-is now! Thanks a lot, @samuel871211

@bsmth bsmth merged commit d5c3db4 into mdn:main Jul 10, 2025
8 checks passed
@samuel871211 samuel871211 deleted the patch-2 branch July 10, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants