fix: handle undefined rejections

Author: alexandre-abrioux

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(),

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