Bug 1541860 - Omit cookies sent when preloading intermediates r=johannh

Intermediate preloading was including cookies during attachment fetches to our
Kinto attachment servers. There's no reason for that, so let's not.

Differential Revision: https://phabricator.services.mozilla.com/D26193

--HG--
extra : moz-landing-system : lando
This commit is contained in:
J.C. Jones 2019-04-04 21:43:34 +00:00
Родитель 74bc8c33e6
Коммит 6812d55c4f
1 изменённых файлов: 7 добавлений и 3 удалений

Просмотреть файл

@ -29,7 +29,9 @@ XPCOMUtils.defineLazyGetter(this, "gTextDecoder", () => new TextDecoder());
XPCOMUtils.defineLazyGetter(this, "baseAttachmentsURL", async () => {
const server = Services.prefs.getCharPref("services.settings.server");
const serverInfo = await (await fetch(`${server}/`)).json();
const serverInfo = await (await fetch(`${server}/`, {
credentials: "omit",
})).json();
const {capabilities: {attachments: {base_url}}} = serverInfo;
return base_url;
});
@ -168,8 +170,10 @@ this.RemoteSecuritySettings = class RemoteSecuritySettings {
const headers = new Headers();
headers.set("Accept-Encoding", "gzip");
return fetch(remoteFilePath, {headers})
.then(resp => {
return fetch(remoteFilePath, {
headers,
credentials: "omit",
}).then(resp => {
log.debug(`Download fetch completed: ${resp.ok} ${resp.status}`);
if (!resp.ok) {
Cu.reportError(`Failed to fetch ${remoteFilePath}: ${resp.status}`);