1943610 - Cross-Origin-* headers do not have a link to MDN

Status: RESOLVED FIXED

(DevTools :: Netmonitor, enhancement)

Description

[Julien Wajsberg [:julienw]]

I noticed that the 3 headers starting with Cross-Origin- do not show the icon to link to MDN.
An example for 2 of them is on browserbench.org.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

We have a hardcoded list of headers in https://searchfox.org/mozilla-release/rev/c392583c1a089a2637587f9c85e078261bf1f260/devtools/client/netmonitor/src/utils/doc-utils.js#15-105

We could add:

• Cross-Origin-Embedder-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Embedder-Policy
• Cross-Origin-Opener-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Opener-Policy
• Cross-Origin-Resource-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Resource-Policy

Comment 2

[Julien Wajsberg [:julienw]]

Attachment: Bug 1943610 - [devtools-netmonitor] Put the Cross-Origin headers to the doc-utils allow list so that the MDN link shows up

These headers are used for cross-origin isolation.

Comment 3

[Julien Wajsberg [:julienw]]

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #1)

We have a hardcoded list of headers in https://searchfox.org/mozilla-release/rev/c392583c1a089a2637587f9c85e078261bf1f260/devtools/client/netmonitor/src/utils/doc-utils.js#15-105

We could add:

• Cross-Origin-Embedder-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Embedder-Policy
• Cross-Origin-Opener-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Opener-Policy
• Cross-Origin-Resource-Policy -> https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cross-Origin-Resource-Policy

hehe yes, I figured it out already :-) patch is there !

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8431c9cbc068
[devtools-netmonitor] Put the Cross-Origin headers to the doc-utils allow list so that the MDN link shows up r=devtools-reviewers,jdescottes

Comment 5

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/8431c9cbc068