Closed
Bug 1681250
Opened 4 years ago
Closed 4 years ago
Relax the check for webCOOP+COEP remote types by only checking the prefix matches
Categories
(Core :: DOM: Navigation, defect, P2)
Core
DOM: Navigation
Tracking
()
RESOLVED
FIXED
86 Branch
| Tracking | Status | |
|---|---|---|
| firefox86 | --- | fixed |
People
(Reporter: tt, Assigned: tt)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
We use GetSiteOrigin to get the origin from the result principal.
https://searchfox.org/mozilla-central/rev/1843375acbbca68127713e402be222350ac99301/netwerk/ipc/DocumentLoadListener.cpp#1535
However, the comment here says:
// To be consistent with remote types when using a principal vs. a URI,
// always clear OAs.
// FIXME: This should be accurate.
And this causes us to fail to change the process remote type when we have COOP+COEP headers and userContextId in some situations.
We should probably use GetSiteOriginNoSuffix. I will update a patch soon.
|
Assignee | |
Updated•4 years ago
|
Component: Session Restore → DOM: Navigation
Product: Firefox → Core
Summary: Use GetSiteOriginNoSuffix to get the site origin without origin attributes → Use GetSiteOriginNoSuffix to get the site origin without origin attributes for preferredRemoteType
|
|
Assignee | |
Comment 1•4 years ago
|
||
|
||
Updated•4 years ago
|
Attachment #9191842 -
Attachment description: Bug 1681250 - Use GetSiteOriginNoSuffix to get site origin without origin attributes for preferredRemoteType; → Bug 1681250 - Relax the check for webCOOP+COEP remote types by only checking the prefix matches;
|
||
Updated•4 years ago
|
Severity: -- → S3
Priority: -- → P2
|
|
Assignee | |
Updated•4 years ago
|
Summary: Use GetSiteOriginNoSuffix to get the site origin without origin attributes for preferredRemoteType → Relax the check for webCOOP+COEP remote types by only checking the prefix matches
Pushed by ttung@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3d120f9bf225
Relax the check for webCOOP+COEP remote types by only checking the prefix matches; r=annyG
|
||
Comment 3•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox86:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•