fix: adds content type only on post method (#198)

Only sets the content-type header on POST methods. The SDK used to set it on GET methods too, but GET methods don't have a body, so the header isn't needed.

* fix: adds content type only on post method

* fix: adds check for header in GET and Post test

---------

Co-authored-by: mohsen <[email protected]>

src/index.test.ts

@@ -60,6 +60,9 @@ test('Should perform an initial fetch as POST', async () => {
 
     expect(request.method).toBe('POST');
     expect(body.context.appName).toBe('webAsPOST');
+    expect(request.headers).toMatchObject({
+        'Content-Type': 'application/json',
+    });
 });
 
 test('Should perform an initial fetch as GET', async () => {
@@ -75,6 +78,9 @@ test('Should perform an initial fetch as GET', async () => {
     const request = getTypeSafeRequest(fetchMock, 0);
 
     expect(request.method).toBe('GET');
+    expect(request.headers).not.toMatchObject({
+        'Content-Type': 'application/json',
+    });
 });
 
 test('Should have correct variant', async () => {

src/index.ts

@@ -400,11 +400,14 @@ export class UnleashClient extends TinyEmitter {
     }
 
     private getHeaders() {
+        const isPOST = this.usePOSTrequests;
         const headers = {
             [this.headerName]: this.clientKey,
             Accept: 'application/json',
-            'Content-Type': 'application/json',
         };
+        if (isPOST) {
+            headers['Content-Type'] = 'application/json';
+        }
         if (this.etag) {
             headers['If-None-Match'] = this.etag;
         }