check for transferEncoding as well

fastify · Apr 22, 2024 · 97ede71 · 97ede71
1 parent 44dc0a5
commit 97ede71
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/lib/handleRequest.js b/lib/handleRequest.js
@@ -29,21 +29,20 @@ function handleRequest (err, request, reply) {
   if (bodyMethods.has(method)) {
     const contentType = headers['content-type']
     const contentLength = headers['content-length']
+    const transferEncoding = headers['transfer-encoding']
 
     if (contentType === undefined) {
-      const transferEncoding = headers['transfer-encoding']
-
       if (
-        transferEncoding === undefined &&
-        (contentLength === undefined || contentLength === '0')
+        (contentLength === undefined || contentLength === '0') &&
+        transferEncoding === undefined
       ) {
         // Request has no body to parse
         handler(request, reply)
       } else {
         context.contentTypeParser.run('', handler, request, reply)
       }
     } else {
-      if (contentLength === undefined && method === 'OPTIONS') {
+      if (contentLength === undefined && transferEncoding === undefined && method === 'OPTIONS') {
         // OPTIONS can have a Content-Type header without a body
         handler(request, reply)
         return