fix: handle undefined rejections

winstonjs · Dec 10, 2021 · 2299d19 · 2299d19
1 parent 4572c2a
commit 2299d19
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/lib/winston/rejection-handler.js b/lib/winston/rejection-handler.js
@@ -74,9 +74,9 @@ module.exports = class RejectionHandler {
    * @returns {mixed} - TODO: add return description.
    */
   getAllInfo(err) {
-    let { message } = err;
-    if (!message && typeof err === 'string') {
-      message = err;
+    let message = null;
+    if (err) {
+      message = typeof err === 'string' ? err : err.message;
     }
 
     return {
@@ -85,9 +85,9 @@ module.exports = class RejectionHandler {
       level: 'error',
       message: [
         `unhandledRejection: ${message || '(no error message)'}`,
-        err.stack || '  No stack trace'
+        err && err.stack || '  No stack trace'
       ].join('\n'),
-      stack: err.stack,
+      stack: err && err.stack,
       exception: true,
       date: new Date().toString(),
       process: this.getProcessInfo(),

diff --git a/test/rejection-handler.test.js b/test/rejection-handler.test.js
@@ -106,6 +106,12 @@ describe('UnhandledRejectionHandler', function () {
     helpers.reject('wtf this rejection').then(done());
   });
 
+  it('.getAllInfo(undefined)', function () {
+    var handler = helpers.rejectionHandler();
+    // eslint-disable-next-line no-undefined
+    handler.getAllInfo(undefined);
+  });
+
   after(function () {
     //
     // Restore normal `runTest` functionality