Remove privacy redaction from Errors.toLogFormat

js/modules/types/errors.js

@@ -1,9 +1,7 @@
 const ensureError = require('ensure-error');
 
-const Privacy = require('../privacy');
-
 //      toLogFormat :: Error -> String
 exports.toLogFormat = (error) => {
   const normalizedError = ensureError(error);
-  return Privacy.redactAll(normalizedError.stack);
+  return normalizedError.stack;
 };

test/modules/types/errors_test.js

@@ -9,31 +9,36 @@ const APP_ROOT_PATH = Path.join(__dirname, '..', '..', '..');
 
 describe('Errors', () => {
   describe('toLogFormat', () => {
-    it('should redact sensitive paths in stack trace', () => {
+    it('should convert non-errors to errors', () => {
       try {
-        throw new Error('boom');
+        // eslint-disable-next-line no-throw-literal
-      } catch (error) {
+        throw 'boom';
-        assert.include(
+      } catch (nonError) {
-          error.stack,
+        assert.typeOf(nonError, 'string');
-          APP_ROOT_PATH,
+        assert.isUndefined(nonError.stack);
-          'Unformatted stack has sensitive paths'
-        );
 
-        const formattedStack = Errors.toLogFormat(error);
+        const formattedStack = Errors.toLogFormat(nonError);
-        assert.notInclude(
-          formattedStack,
-          APP_ROOT_PATH,
-          'Formatted stack does not have sensitive paths'
-        );
         assert.include(
           formattedStack,
-          '[REDACTED]',
+          APP_ROOT_PATH,
-          'Formatted stack has redactions'
+          'Formatted stack has app path'
         );
         return;
       }
+
       // eslint-disable-next-line no-unreachable
       assert.fail('Expected error to be thrown.');
     });
+
+    it('should add stack to errors without one', () => {
+      const error = new Error('boom');
+      error.stack = null;
+      assert.typeOf(error, 'Error');
+      assert.isNull(error.stack);
+
+      const formattedStack = Errors.toLogFormat(error);
+      assert.include(formattedStack, '<Original stack missing>');
+      assert.include(formattedStack, APP_ROOT_PATH, 'Formatted stack has app path');
+    });
   });
 });