Open
Bug 1189945
Opened 9 years ago
Updated 2 years ago
Remove nsIChannel CorsMode flag and set Request.mode based on nsILoadInfo securityFlags instead
Categories
(Core :: DOM: Service Workers, defect, P3)
Core
DOM: Service Workers
Tracking
()
NEW
People
(Reporter: bkelly, Unassigned)
References
(Blocks 1 open bug)
Details
Once bug 1182535 updates enough of the network callsites we should switch the ServiceWorkerManager code to using nsILoadInfo::GetSecurityFlags() to set the Request.mode value. Once this is done we can remove the nsIChannel CorsMode flag. The new nsILoadInfo securityFlags is more robust and being integrated with the rest of the tree. For example, the securityFlags will actually reflect same-origin policies which are currently not reflected in actual values on CorsMode. Note, we will still need to exempt the mode for document nsIContentPolicy values since navigations need to be same-origin, but the security flags won't reflect that.
Reporter | ||
Comment 1•9 years ago
|
||
I have a patch in bug 1184967 that adds the conversion from securityFlags.
Comment 2•9 years ago
|
||
Ben said elsewhere that this is nice but not entirely necessary for the initial shipping of SW. He noted some changes that he's made and others still needed but I didn't catch them quickly enough to type them here.
Reporter | ||
Comment 3•7 years ago
|
||
This is a compat issue because the legacy code paths don't reliably provide the correct FetchEvent.request values.
Updated•6 years ago
|
Severity: normal → minor
Priority: -- → P3
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•