Merge pull request #7 from bgoscinski/main

refactor: Get rid of header_utils.ts
jd1378 · May 18, 2022 · a1b40c1 · a1b40c1
2 parents 0e69089 + 2dcdc74
commit a1b40c1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 142 deletions.
diff --git a/fetch_wrapper.ts b/fetch_wrapper.ts
@@ -1,5 +1,4 @@
 import { CookieJar } from "./cookie_jar.ts";
-import { setHeader } from "./header_utils.ts";
 
 export type WrapFetchOptions = {
   /** your own fetch function. defaults to global fetch. This allows wrapping your fetch function multiple times. */
@@ -31,11 +30,10 @@ export function wrapFetch(options: WrapFetchOptions) {
       interceptedInit = {};
     }
 
-    if (!interceptedInit.headers) {
-      interceptedInit.headers = new Headers();
+    if (!(interceptedInit.headers instanceof Headers)) {
+      interceptedInit.headers = new Headers(interceptedInit.headers || {});
     }
-
-    setHeader(interceptedInit.headers, "cookie", cookieString);
+    interceptedInit.headers.set("cookie", cookieString);
 
     const response = await fetch(input, interceptedInit);
     response.headers.forEach((value, key) => {

diff --git a/header_utils.ts b/header_utils.ts