Wrap reason with unsafeToString() if it's a string

nodejs · Nov 20, 2017 · ab45e60 · ab45e60
1 parent ba4adc5
commit ab45e60
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/internal/process/promises.js b/lib/internal/process/promises.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const { safeToString } = process.binding('util');
+
 const promiseRejectEvent = process._promiseRejectEvent;
 const hasBeenNotifiedProperty = new WeakMap();
 const promiseToGuidProperty = new WeakMap();
@@ -62,7 +64,9 @@ function setupPromises(scheduleMicrotasks) {
       if (reason instanceof Error) {
         process.emitWarning(reason.stack, 'UnhandledPromiseRejectionWarning');
       } else {
-        process.emitWarning(reason, 'UnhandledPromiseRejectionWarning');
+        process.emitWarning(
+          safeToString(reason), 'UnhandledPromiseRejectionWarning'
+        );
       }
     } catch (e) {
       // ignored