fixup! http: emit 'error' on aborted client response

nodejs · May 1, 2020 · 0c54b5f · 0c54b5f
1 parent 4bcd762
commit 0c54b5f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/doc/api/http.md b/doc/api/http.md
@@ -2404,7 +2404,7 @@ the following events will be emitted in the following order:
 * (connection closed here)
 * `'aborted'` on the `res` object
 * `'error'` on the `res` object with an error with message
-  `'Error: socket hang up'` and code `'ECONNRESET'`.
+  `'Error: aborted'` and code `'ECONNRESET'`.
 * `'close'`
 * `'close'` on the `res` object
 
@@ -2434,7 +2434,7 @@ events will be emitted in the following order:
 * (`req.destroy()` called here)
 * `'aborted'` on the `res` object
 * `'error'` on the `res` object with an error with message
-  `'Error: socket hang up'` and code `'ECONNRESET'`.
+  `'Error: aborted'` and code `'ECONNRESET'`.
 * `'close'`
 * `'close'` on the `res` object
 
@@ -2465,7 +2465,7 @@ events will be emitted in the following order:
 * `'abort'`
 * `'aborted'` on the `res` object
 * `'error'` on the `res` object with an error with message
-  `'Error: socket hang up'` and code `'ECONNRESET'`.
+  `'Error: aborted'` and code `'ECONNRESET'`.
 * `'close'`
 * `'close'` on the `res` object
 

diff --git a/test/parallel/test-http-aborted.js b/test/parallel/test-http-aborted.js
@@ -11,6 +11,7 @@ const assert = require('assert');
     }));
     req.on('error', common.mustCall(function(err) {
       assert.strictEqual(err.code, 'ECONNRESET');
+      assert.strictEqual(err.message, 'aborted');
       server.close();
     }));
     assert.strictEqual(req.aborted, false);
@@ -27,6 +28,7 @@ const assert = require('assert');
       }));
       res.on('error', common.mustCall((err) => {
         assert.strictEqual(err.code, 'ECONNRESET');
+        assert.strictEqual(err.message, 'aborted');
       }));
       req.abort();
     }));