update

Author: XadillaX

doc/api/http.md

@@ -734,7 +734,7 @@ changes:
     description: The default action of calling `.destroy()` on the `socket`
                  will no longer take place if there are listeners attached
                  for `clientError`.
-  - version: VERSION
+  - version: REPLACEME
     pr-url: https://github.com/nodejs/node/pull/17672
     description: The rawPacket is the current buffer that just parsed. Adding
                  this buffer to the error object of clientError event is to make

lib/_http_server.js

@@ -475,12 +475,8 @@ function socketOnError(e) {
 function onParserExecuteCommon(server, socket, parser, state, ret, d) {
   resetSocketTimeout(server, socket, state);
 
-  if (!d) {
-    d = parser.getCurrentBuffer();
-  }
-
   if (ret instanceof Error) {
-    ret.rawPacket = d;
+    ret.rawPacket = d || parser.getCurrentBuffer();
     debug('parse error', ret);
     socketOnError.call(socket, ret);
   } else if (parser.incoming && parser.incoming.upgrade) {
@@ -489,6 +485,9 @@ function onParserExecuteCommon(server, socket, parser, state, ret, d) {
     var req = parser.incoming;
     debug('SERVER upgrade or connect', req.method);
 
+    if (!d)
+      d = parser.getCurrentBuffer();
+
     socket.removeListener('data', state.onData);
     socket.removeListener('end', state.onEnd);
     socket.removeListener('close', state.onClose);