nodejs
diff --git a/Collapse file
‎src/async-wrap.cc‎
Copy file name to clipboardExpand all lines: src/async-wrap.cc
+3-3Lines changed: 3 additions & 3 deletions b/Collapse file
‎src/async-wrap.cc‎
Copy file name to clipboardExpand all lines: src/async-wrap.cc
+3-3Lines changed: 3 additions & 3 deletions
diff --git a/Collapse file
‎src/node.cc‎
Copy file name to clipboardExpand all lines: src/node.cc
+5-1Lines changed: 5 additions & 1 deletion b/Collapse file
‎src/node.cc‎
Copy file name to clipboardExpand all lines: src/node.cc
+5-1Lines changed: 5 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-http-catch-uncaughtexception.js‎
Copy file name to clipboard
+23Lines changed: 23 additions & 0 deletions b/Collapse file
‎test/parallel/test-http-catch-uncaughtexception.js‎
Copy file name to clipboard
+23Lines changed: 23 additions & 0 deletions
@@ -193,7 +193,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,
     if (has_domain) {
       domain = domain_v.As<Object>();
       if (domain->Get(env()->disposed_string())->IsTrue())
-        return Undefined(env()->isolate());
+        return Local<Value>();
     }
   }
 
@@ -220,7 +220,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,
   }
 
   if (ret.IsEmpty()) {
-    return Undefined(env()->isolate());
+    return ret;
   }
 
   if (has_domain) {
@@ -249,7 +249,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,
   }
 
   if (env()->tick_callback_function()->Call(process, 0, nullptr).IsEmpty()) {
-    return Undefined(env()->isolate());
+    return Local<Value>();
   }
 
   return ret;
 
@@ -1177,7 +1177,11 @@ Local<Value> MakeCallback(Environment* env,
   }
 
   if (ret.IsEmpty()) {
-    return Undefined(env->isolate());
+    if (callback_scope.in_makecallback())
+      return ret;
+    // NOTE: Undefined() is returned here for backwards compatibility.
+    else
+      return Undefined(env->isolate());
   }
 
   if (has_domain) {
 
@@ -0,0 +1,23 @@
+'use strict';
+
+const common = require('../common');
+const assert = require('assert');
+const http = require('http');
+
+const uncaughtCallback = common.mustCall(function(er) {
+  assert.equal(er.message, 'get did fail');
+});
+
+process.on('uncaughtException', uncaughtCallback);
+
+const server = http.createServer(function(req, res) {
+  res.writeHead(200, { 'Content-Type': 'text/plain' });
+  res.end('bye');
+}).listen(common.PORT, function() {
+  http.get({ port: common.PORT }, function(res) {
+    res.resume();
+    throw new Error('get did fail');
+  }).on('close', function() {
+    server.close();
+  });
+});
Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,`
`193`	`193`	`if (has_domain) {`
`194`	`194`	`domain = domain_v.As<Object>();`
`195`	`195`	`if (domain->Get(env()->disposed_string())->IsTrue())`
`196`		`- return Undefined(env()->isolate());`
	`196`	`+ return Local<Value>();`
`197`	`197`	`}`
`198`	`198`	`}`
`199`	`199`
`@@ -220,7 +220,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,`
`220`	`220`	`}`
`221`	`221`
`222`	`222`	`if (ret.IsEmpty()) {`
`223`		`- return Undefined(env()->isolate());`
	`223`	`+ return ret;`
`224`	`224`	`}`
`225`	`225`
`226`	`226`	`if (has_domain) {`
`@@ -249,7 +249,7 @@ Local<Value> AsyncWrap::MakeCallback(const Local<Function> cb,`
`249`	`249`	`}`
`250`	`250`
`251`	`251`	`if (env()->tick_callback_function()->Call(process, 0, nullptr).IsEmpty()) {`
`252`		`- return Undefined(env()->isolate());`
	`252`	`+ return Local<Value>();`
`253`	`253`	`}`
`254`	`254`
`255`	`255`	`return ret;`
Original file line number	Diff line number	Diff line change
`@@ -1177,7 +1177,11 @@ Local<Value> MakeCallback(Environment* env,`
`1177`	`1177`	`}`
`1178`	`1178`
`1179`	`1179`	`if (ret.IsEmpty()) {`
`1180`		`- return Undefined(env->isolate());`
	`1180`	`+ if (callback_scope.in_makecallback())`
	`1181`	`+ return ret;`
	`1182`	`+ // NOTE: Undefined() is returned here for backwards compatibility.`
	`1183`	`+ else`
	`1184`	`+ return Undefined(env->isolate());`
`1181`	`1185`	`}`
`1182`	`1186`
`1183`	`1187`	`if (has_domain) {`