From 0b1bf2bcf7ba2e6f0889e3fd36bad6750cce4fcb Mon Sep 17 00:00:00 2001 From: john gravois Date: Wed, 23 Jan 2019 13:15:46 -0800 Subject: [PATCH] fix(:bug:): dont set referer to null in browser apps AFFECTS PACKAGES: @esri/arcgis-rest-request --- packages/arcgis-rest-request/src/request.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/arcgis-rest-request/src/request.ts b/packages/arcgis-rest-request/src/request.ts index 47d9c1ba39..c0af2b8636 100644 --- a/packages/arcgis-rest-request/src/request.ts +++ b/packages/arcgis-rest-request/src/request.ts @@ -186,13 +186,15 @@ export function request( } // Mixin headers from request options - /* istanbul ignore next - karma reports coverage on browser tests only */ fetchOptions.headers = { - referer: - typeof window === "undefined" ? NODEJS_DEFAULT_REFERER_HEADER : null, ...requestOptions.headers }; + /* istanbul ignore next - karma reports coverage on browser tests only */ + if (typeof window === "undefined" && !fetchOptions.headers.referer) { + fetchOptions.headers.referer = NODEJS_DEFAULT_REFERER_HEADER; + } + /* istanbul ignore else blob responses are difficult to make cross platform we will just have to trust the isomorphic fetch will do its job */ if (!requiresFormData(params)) { fetchOptions.headers["Content-Type"] =