Merge pull request #289 from saschagoebel/master

Always send Content-Length header. Fixes compatibility issues with el…
elastic · Nov 30, 2015 · bf8ef6e · bf8ef6e
2 parents c8d5bae + 48667b2
commit bf8ef6e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/lib/connectors/http.js b/src/lib/connectors/http.js
@@ -185,10 +185,11 @@ HttpConnector.prototype.request = function (params, cb) {
     request.setHeader('Content-Length', Buffer.byteLength(params.body, 'utf8'));
     request.end(params.body);
   } else {
+    request.setHeader('Content-Length', 0);
     request.end();
   }
 
   return function () {
     request.abort();
   };
-};
+};
diff --git a/test/mocks/request.js b/test/mocks/request.js
@@ -12,6 +12,7 @@ var http = require('http');
 function MockRequest() {
   sinon.stub(this, 'end');
   sinon.stub(this, 'write');
+  sinon.stub(this, 'setHeader');
   this.log = sinon.stub(this.log);
 }
-util.inherits(MockRequest, http.ClientRequest);
+util.inherits(MockRequest, http.ClientRequest);