Use object instead of Headers (#294)

croct-tech · Nov 29, 2022 · 508bac4 · 508bac4
1 parent bf0f839
commit 508bac4
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 16 deletions.
diff --git a/src/contentFetcher.ts b/src/contentFetcher.ts
@@ -159,35 +159,37 @@ export class ContentFetcher {
 
         const {apiKey, appId} = this.configuration;
 
-        const headers = new Headers();
+        const headers: Record<string, string> = {
+            'Content-Type': 'application/json',
+        };
 
-        headers.set('X-Client-Library', CLIENT_LIBRARY);
+        headers['X-Client-Library'] = CLIENT_LIBRARY;
 
         if (appId !== undefined) {
-            headers.set('X-App-Id', appId);
+            headers['X-App-Id'] = appId;
         }
 
         if (apiKey !== undefined) {
-            headers.set('X-Api-Key', apiKey);
+            headers['X-Api-Key'] = apiKey;
         }
 
         const dynamic = ContentFetcher.isDynamicContent(options);
 
         if (dynamic) {
             if (options.clientId !== undefined) {
-                headers.set('X-Client-Id', options.clientId);
+                headers['X-Client-Id'] = options.clientId;
             }
 
             if (options.clientIp !== undefined) {
-                headers.set('X-Client-Ip', options.clientIp);
+                headers['X-Client-Ip'] = options.clientIp;
             }
 
             if (options.userToken !== undefined) {
-                headers.set('X-Token', options.userToken.toString());
+                headers['X-Token'] = options.userToken.toString();
             }
 
             if (options.userAgent !== undefined) {
-                headers.set('User-Agent', options.userAgent);
+                headers['User-Agent'] = options.userAgent;
             }
 
             if (options.version !== undefined) {

diff --git a/src/evaluator.ts b/src/evaluator.ts
@@ -210,30 +210,32 @@ export class Evaluator {
         const {appId, apiKey} = this.configuration;
         const {clientId, clientIp, userAgent, userToken} = options;
 
-        const headers = new Headers();
+        const headers: Record<string, string> = {
+            'Content-Type': 'application/json',
+        };
 
-        headers.set('X-Client-Library', CLIENT_LIBRARY);
+        headers['X-Client-Library'] = CLIENT_LIBRARY;
 
         if (apiKey !== undefined) {
-            headers.set('X-Api-Key', apiKey);
+            headers['X-Api-Key'] = apiKey;
         } else if (appId !== undefined) {
-            headers.set('X-App-Id', appId);
+            headers['X-App-Id'] = appId;
         }
 
         if (clientId !== undefined) {
-            headers.set('X-Client-Id', clientId);
+            headers['X-Client-Id'] = clientId;
         }
 
         if (clientIp !== undefined) {
-            headers.set('X-Client-Ip', clientIp);
+            headers['X-Client-Ip'] = clientIp;
         }
 
         if (userToken !== undefined) {
-            headers.set('X-Token', userToken.toString());
+            headers['X-Token'] = userToken.toString();
         }
 
         if (userAgent !== undefined) {
-            headers.set('User-Agent', userAgent);
+            headers['User-Agent'] = userAgent;
         }
 
         return fetch(this.endpoint, {

diff --git a/test/contentFetcher.test.ts b/test/contentFetcher.test.ts
@@ -28,6 +28,7 @@ describe('A content fetcher', () => {
     const requestMatcher: MockOptions = {
         method: 'POST',
         headers: {
+            'Content-Type': 'application/json',
             'X-Client-Library': CLIENT_LIBRARY,
         },
         body: {

diff --git a/test/evaluator.test.ts b/test/evaluator.test.ts
@@ -30,6 +30,7 @@ describe('An evaluator', () => {
     const requestMatcher: MockOptions = {
         method: 'POST',
         headers: {
+            'Content-Type': 'application/json',
             'X-Client-Library': CLIENT_LIBRARY,
         },
         body: {