nodejs
diff --git a/Collapse file
‎lib/internal/process/execution.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/execution.js
+4-1Lines changed: 4 additions & 1 deletion b/Collapse file
‎lib/internal/process/execution.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/execution.js
+4-1Lines changed: 4 additions & 1 deletion
diff --git a/Collapse file
‎lib/internal/process/report.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/report.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎lib/internal/process/report.js‎
Copy file name to clipboardExpand all lines: lib/internal/process/report.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎src/node_report_module.cc‎
Copy file name to clipboardExpand all lines: src/node_report_module.cc
+7-23Lines changed: 7 additions & 23 deletions b/Collapse file
‎src/node_report_module.cc‎
Copy file name to clipboardExpand all lines: src/node_report_module.cc
+7-23Lines changed: 7 additions & 23 deletions
@@ -116,7 +116,10 @@ function createFatalException() {
           report.syncConfig(config, false);
           if (Array.isArray(config.events) &&
               config.events.includes('exception')) {
-            report.onUnCaughtException(er ? er.stack : undefined);
+            report.triggerReport(er ? er.message : 'Exception',
+                                 'Exception',
+                                 null,
+                                 er ? er.stack : undefined);
           }
         }
       } catch {}  // Ignore the exception. Diagnostic reporting is unavailable.
 
@@ -87,7 +87,7 @@ const report = {
       throw new ERR_INVALID_ARG_TYPE('err', 'Object', err);
     }
 
-    return nr.triggerReport(file, err.stack);
+    return nr.triggerReport('JavaScript API', 'API', file, err.stack);
   },
   getReport(err) {
     emitExperimentalWarning('report');
 
@@ -35,7 +35,6 @@ using v8::V8;
 using v8::Value;
 
 // Internal/static function declarations
-void OnUncaughtException(const FunctionCallbackInfo<Value>& info);
 static void Initialize(Local<Object> exports,
                        Local<Value> unused,
                        Local<Context> context);
@@ -48,14 +47,16 @@ void TriggerReport(const FunctionCallbackInfo<Value>& info) {
   std::string filename;
   Local<String> stackstr;
 
-  CHECK_EQ(info.Length(), 2);
-  stackstr = info[1].As<String>();
+  CHECK_EQ(info.Length(), 4);
+  String::Utf8Value message(isolate, info[0].As<String>());
+  String::Utf8Value trigger(isolate, info[1].As<String>());
+  stackstr = info[3].As<String>();
 
-  if (info[0]->IsString())
-    filename = *String::Utf8Value(isolate, info[0]);
+  if (info[2]->IsString())
+    filename = *String::Utf8Value(isolate, info[2]);
 
   filename = TriggerNodeReport(
-      isolate, env, "JavaScript API", __func__, filename, stackstr);
+      isolate, env, *message, *trigger, filename, stackstr);
   // Return value is the report filename
   info.GetReturnValue().Set(
       String::NewFromUtf8(isolate, filename.c_str(), v8::NewStringType::kNormal)
@@ -79,22 +80,6 @@ void GetReport(const FunctionCallbackInfo<Value>& info) {
                                 .ToLocalChecked());
 }
 
-// Callbacks for triggering report on uncaught exception.
-// Calls triggered from JS land.
-void OnUncaughtException(const FunctionCallbackInfo<Value>& info) {
-  Environment* env = Environment::GetCurrent(info);
-  Isolate* isolate = env->isolate();
-  HandleScope scope(isolate);
-  std::string filename;
-  std::shared_ptr<PerIsolateOptions> options = env->isolate_data()->options();
-
-  // Trigger report if requested
-  if (options->report_uncaught_exception) {
-    TriggerNodeReport(
-        isolate, env, "exception", __func__, filename, info[0].As<String>());
-  }
-}
-
 // Signal handler for report action, called from JS land (util.js)
 void OnUserSignal(const FunctionCallbackInfo<Value>& info) {
   Environment* env = Environment::GetCurrent(info);
@@ -239,7 +224,6 @@ static void Initialize(Local<Object> exports,
   std::shared_ptr<PerIsolateOptions> options = env->isolate_data()->options();
   env->SetMethod(exports, "triggerReport", TriggerReport);
   env->SetMethod(exports, "getReport", GetReport);
-  env->SetMethod(exports, "onUnCaughtException", OnUncaughtException);
   env->SetMethod(exports, "onUserSignal", OnUserSignal);
   env->SetMethod(exports, "syncConfig", SyncConfig);
 }
Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,10 @@ function createFatalException() {`
`116`	`116`	`report.syncConfig(config, false);`
`117`	`117`	`if (Array.isArray(config.events) &&`
`118`	`118`	`config.events.includes('exception')) {`
`119`		`- report.onUnCaughtException(er ? er.stack : undefined);`
	`119`	`+ report.triggerReport(er ? er.message : 'Exception',`
	`120`	`+ 'Exception',`
	`121`	`+ null,`
	`122`	`+ er ? er.stack : undefined);`
`120`	`123`	`}`
`121`	`124`	`}`
`122`	`125`	`} catch {} // Ignore the exception. Diagnostic reporting is unavailable.`
Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ const report = {`
`87`	`87`	`throw new ERR_INVALID_ARG_TYPE('err', 'Object', err);`
`88`	`88`	`}`
`89`	`89`
`90`		`- return nr.triggerReport(file, err.stack);`
	`90`	`+ return nr.triggerReport('JavaScript API', 'API', file, err.stack);`
`91`	`91`	`},`
`92`	`92`	`getReport(err) {`
`93`	`93`	`emitExperimentalWarning('report');`