fix: check for existence of fetch before using it (#138)

googleapis · Jun 20, 2019 · 79eb58d · 79eb58d
1 parent 2db0f66
commit 79eb58d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 22 deletions.
diff --git a/src/gaxios.ts b/src/gaxios.ts
@@ -25,13 +25,24 @@ import {
   GaxiosResponse,
   Headers,
 } from './common';
-import {isBrowser} from './isbrowser';
 import {getRetryConfig} from './retry';
 
 // tslint:disable no-any
 
-const URL = isBrowser() ? window.URL : url.URL;
-const fetch = isBrowser() ? window.fetch : nodeFetch;
+const URL = hasURL() ? window.URL : url.URL;
+const fetch = hasFetch() ? window.fetch : nodeFetch;
+
+function hasWindow() {
+  return typeof window !== 'undefined' && !!window;
+}
+
+function hasURL() {
+  return hasWindow() && !!window.URL;
+}
+
+function hasFetch() {
+  return hasWindow() && !!window.fetch;
+}
 
 // tslint:disable-next-line variable-name
 let HttpsProxyAgent: any;

diff --git a/src/isbrowser.ts b/src/isbrowser.ts