nodejs
diff --git a/‎lib/_http_incoming.js‎
Lines changed: 5 additions & 6 deletions b/‎lib/_http_incoming.js‎
Lines changed: 5 additions & 6 deletions
diff --git a/‎test/parallel/test-http-client-incomingmessage-destroy.js‎
Lines changed: 25 additions & 0 deletions b/‎test/parallel/test-http-client-incomingmessage-destroy.js‎
Lines changed: 25 additions & 0 deletions
diff --git a/‎test/parallel/test-http-server-incomingmessage-destroy.js‎
Lines changed: 25 additions & 0 deletions b/‎test/parallel/test-http-server-incomingmessage-destroy.js‎
Lines changed: 25 additions & 0 deletions
@@ -361,14 +361,13 @@ IncomingMessage.prototype._dump = function _dump() {
   }
 };
 
-function onError(instance, error, cb) {
+function onError(self, error, cb) {
   // This is to keep backward compatible behavior.
-  // An error is emitted only if there are listeners attached to
-  // the event.
-  if (instance.listenerCount('error') > 0) {
-    cb(error);
-  } else {
+  // An error is emitted only if there are listeners attached to the event.
+  if (self.listenerCount('error') === 0) {
     cb();
+  } else {
+    cb(error);
   }
 }
 
 
@@ -0,0 +1,25 @@
+'use strict';
+
+const common = require('../common');
+const { createServer, get } = require('http');
+const assert = require('assert');
+
+const server = createServer(common.mustCall((req, res) => {
+  res.writeHead(200);
+  res.write('Part of res.');
+}));
+
+function onUncaught(error) {
+  assert.strictEqual(error.message, 'Destroy test');
+  server.close();
+}
+
+process.on('uncaughtException', common.mustCall(onUncaught));
+
+server.listen(0, () => {
+  get({
+    port: server.address().port
+  }, common.mustCall((res) => {
+    res.destroy(new Error('Destroy test'));
+  }));
+});
@@ -0,0 +1,25 @@
+'use strict';
+
+const common = require('../common');
+const { createServer, get } = require('http');
+const assert = require('assert');
+
+const server = createServer(common.mustCall((req, res) => {
+  req.destroy(new Error('Destroy test'));
+}));
+
+function onUncaught(error) {}
+
+process.on('uncaughtException', common.mustNotCall(onUncaught));
+
+server.listen(0, common.mustCall(() => {
+  get({
+    port: server.address().port
+  }, (res) => {
+    res.resume();
+  }).on('error', (error) => {
+    assert.strictEqual(error.message, 'socket hang up');
+    assert.strictEqual(error.code, 'ECONNRESET');
+    server.close();
+  });
+}));
Original file line number	Diff line number	Diff line change
@@ -361,14 +361,13 @@ IncomingMessage.prototype._dump = function _dump() {
361	361	}
362	362	};
363	363
364		- function onError(instance, error, cb) {
	364	+ function onError(self, error, cb) {
365	365	// This is to keep backward compatible behavior.
366		- // An error is emitted only if there are listeners attached to
367		- // the event.
368		- if (instance.listenerCount('error') > 0) {
369		- cb(error);
370		- } else {
	366	+ // An error is emitted only if there are listeners attached to the event.
	367	+ if (self.listenerCount('error') === 0) {
371	368	cb();
	369	+ } else {
	370	+ cb(error);
372	371	}
373	372	}
374	373