fix: Pass Headers to publish to fix Android's fetch (#153)

This resolves an issue with React Native's fetch API specific to Android.
xmtp · Aug 24, 2022 · 83d4d4b · 83d4d4b
2 parents 5bb4304 + 72ffb33
commit 83d4d4b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/ApiClient.ts b/src/ApiClient.ts
@@ -109,7 +109,7 @@ export default class ApiClient {
           {
             pathPrefix: this.pathPrefix,
             mode: 'cors',
-            headers: { Authorization: `Bearer ${authToken}` },
+            headers: new Headers({ Authorization: `Bearer ${authToken}` }),
           },
         ],
         this.maxRetries,

diff --git a/test/ApiClient.test.ts b/test/ApiClient.test.ts
@@ -154,9 +154,7 @@ describe('Publish', () => {
     expect(publishMock).toHaveBeenCalledWith(expectedRequest, {
       pathPrefix: PATH_PREFIX,
       mode: 'cors',
-      headers: {
-        Authorization: `Bearer ${AUTH_TOKEN}`,
-      },
+      headers: new Headers({ Authorization: `Bearer ${AUTH_TOKEN}` }),
     })
   })