Skip to content
Snippets Groups Projects
Commit 6d10a1ac authored by Eden Chuang's avatar Eden Chuang Committed by Pier Angelo Vendrame
Browse files

Bug 1738426 - Ignoring status 206 and vary header checking for opaque response...

Bug 1738426 - Ignoring status 206 and vary header checking for opaque response in Cache API. r=asuth

Differential Revision: https://phabricator.services.mozilla.com/D186431
parent a3b689c0
No related branches found
No related tags found
1 merge request!941Bug 42449: Rebase Tor Browser alpha onto Firefox 115.9.0esr
......@@ -184,7 +184,7 @@ void TypeUtils::ToCacheResponseWithoutBody(CacheResponse& aOut,
aOut.statusText() = aIn.GetUnfilteredStatusText();
RefPtr<InternalHeaders> headers = aIn.UnfilteredHeaders();
MOZ_DIAGNOSTIC_ASSERT(headers);
if (HasVaryStar(headers)) {
if (aIn.Type() != ResponseType::Opaque && HasVaryStar(headers)) {
aRv.ThrowTypeError("Invalid Response object with a 'Vary: *' header.");
return;
}
......
[cache-put.https.any.serviceworker.html]
expected:
if (os == "android") and fission: [OK, TIMEOUT]
[Cache.put with a VARY:* opaque response should not reject]
expected: FAIL
[cache-put.https.any.sharedworker.html]
expected:
if (os == "android") and fission: [OK, TIMEOUT]
[Cache.put with a VARY:* opaque response should not reject]
expected: FAIL
[cache-put.https.any.html]
expected:
if (os == "android") and fission: [OK, TIMEOUT]
[Cache.put with a VARY:* opaque response should not reject]
expected: FAIL
[cache-put.https.any.worker.html]
expected:
if (os == "android") and fission: [OK, TIMEOUT]
[Cache.put with a VARY:* opaque response should not reject]
expected: FAIL
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment