fix(parrot-fetch): normalize json request body

americanexpress · Aug 7, 2023 · b717eb4 · b717eb4
1 parent 3d062f8
commit b717eb4
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/packages/parrot-fetch/__tests__/ParrotFetch.spec.js b/packages/parrot-fetch/__tests__/ParrotFetch.spec.js
@@ -30,6 +30,24 @@ describe('ParrotFetch', () => {
     });
   });
 
+  it('should parse the body is content-type is json', () => {
+    const parrotFetch = new ParrotFetch();
+    const normalized = parrotFetch.normalizeRequest('http://www.parrot.com/squawk', {
+      headers: {
+        'content-type': 'application/json',
+      },
+      body: JSON.stringify({ ahoy: 'matey' }),
+    });
+    expect(normalized).toMatchObject({
+      path: '/squawk',
+      body: {
+        ahoy: 'matey',
+      },
+      protocol: 'http:',
+      host: 'www.parrot.com',
+    });
+  });
+
   it('should resolve to context fetch', () => {
     const input = 'http://www.parrot.com';
     const contextFetch = jest.fn(() => 'ahoy');

diff --git a/packages/parrot-fetch/src/ParrotFetch.js b/packages/parrot-fetch/src/ParrotFetch.js
@@ -23,9 +23,15 @@ class ParrotFetch extends Parrot {
 
   normalizeRequest = (input, { method = 'GET', ...init } = {}) => {
     const { pathname: path, query, ...parsed } = parse(input, true);
+    const headers = new Headers(init.headers);
+    let { body } = init;
+    if (headers.get('Content-Type') === 'application/json') {
+      body = JSON.parse(body);
+    }
     return {
       ...init,
       ...parsed,
+      body,
       path,
       query: Object.keys(query) && query,
       method: method && method.toUpperCase(),