refactor: parser.destroy

nodejs · Oct 26, 2020 · 67fde56 · 67fde56
1 parent ce104eb
commit 67fde56
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/lib/core/client.js b/lib/core/client.js
@@ -431,12 +431,13 @@ class Parser extends HTTPParser {
  }
 
  if (!socket.destroyed && !request.aborted) {
- this.destroy()
-
  clearTimeout(socket[kIdleTimeout])
  socket[kIdleTimeout] = null
  socket[kIdleTimeoutValue] = null
+
+ setImmediate((self) => self.close(), socket[kParser])
  socket[kParser] = null
+
  socket[kPause] = null
  socket[kResume] = null
  socket[kClient] = null
@@ -639,11 +640,6 @@ class Parser extends HTTPParser {
  resume(client)
  }
  }
-
- destroy () {
- this.unconsume()
- setImmediate((self) => self.close(), this)
- }
 }
 
 function onSocketConnect () {
@@ -692,7 +688,8 @@ function onSocketClose () {
 
  client[kSocket] = null
 
- parser.destroy()
+ parser.unconsume()
+ setImmediate((self) => self.close(), parser)
 
  if (err.code !== 'UND_ERR_INFO') {
  // Evict session on errors.