implement adding Content-Length header

- derived from PR f#22
urkle · Aug 20, 2022 · cd8f63a · cd8f63a
1 parent 31f6fac
commit cd8f63a
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/graphql.js b/graphql.js
@@ -116,6 +116,10 @@
       }
     }
 
+    if (cleaned.body) {
+      headers['Content-Length'] = cleaned.body.length;
+    }
+
     __doRequest(
       method,
       cleaned.url,

diff --git a/test/test.js b/test/test.js
@@ -211,6 +211,13 @@ fragment auth_error on Error {messages}`;
                 expect(xhr.open).toHaveBeenCalledWith(method, expect.stringMatching(url), true)
                 expect(xhr.open).toHaveBeenCalledWith(method, expect.stringMatching(/\?query=.+&variables=/), true)
             });
+
+            it('does not send the content-length header', () => {
+                let xhr = mockXHR(200, {});
+                xhr.send = jest.fn();
+                fetchPost({id: 123});
+                expect(xhr.setRequestHeader).not.toHaveBeenCalledWith('Content-Length', expect.anything);
+            });
         });
 
         describe('when executing the queries normally', () => {
@@ -221,6 +228,13 @@ fragment auth_error on Error {messages}`;
                 expect(xhr.send).toHaveBeenCalled();
             });
 
+            it('sends the content-length header', () => {
+                let xhr = mockXHR(200, {});
+                xhr.send = jest.fn();
+                fetchPost({id: 123});
+                expect(xhr.setRequestHeader).toHaveBeenCalledWith('Content-Length', 99);
+            });
+
             it('resolves the response in the promise', () => {
                 let data = {post: {id: 123, title: 'title', text: 'text'}};
                 mockXHR(200, data);