http: do not default to chunked encoding for TRACE

Fixes: nodejs#25783
lpinca · May 13, 2019 · 9c0adf2 · 9c0adf2
1 parent 3309c85
commit 9c0adf2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/_http_client.js b/lib/_http_client.js
@@ -180,6 +180,7 @@ function ClientRequest(input, options, cb) {
       method === 'HEAD' ||
       method === 'DELETE' ||
       method === 'OPTIONS' ||
+      method === 'TRACE' ||
       method === 'CONNECT') {
     this.useChunkedEncodingByDefault = false;
   } else {

diff --git a/test/parallel/test-http-client-default-headers-exist.js b/test/parallel/test-http-client-default-headers-exist.js
@@ -31,7 +31,8 @@ const expectedHeaders = {
   'HEAD': ['host', 'connection'],
   'OPTIONS': ['host', 'connection'],
   'POST': ['host', 'connection', 'content-length'],
-  'PUT': ['host', 'connection', 'content-length']
+  'PUT': ['host', 'connection', 'content-length'],
+  'TRACE': ['host', 'connection']
 };
 
 const expectedMethods = Object.keys(expectedHeaders);