test cases for #182, style tweaks, changelog, etc.

AUTHORS

@@ -19,3 +19,4 @@ Mihai Tomescu (https://github.com/matomesc)
 Daniel Juhl (https://github.com/danieljuhl)
 Chris Barber (https://github.com/cb1kenobi)
 Manuel Schneider (https://github.com/manuelschneider)
+Martin Gausby (https://github.com/gausby)

CHANGES.md

@@ -8,7 +8,9 @@ Known issues:
 
 ## bunyan 1.3.2 (not yet released)
 
-(nothing yet)
+- [pull #182] Fallback to using the optional 'safe-json-stringify' module
+  if `JSON.stringify` throws -- possibly with an enumerable property
+  getter than throws. By Martin Gausby.
 
 
 ## bunyan 1.3.1

lib/bunyan.js

@@ -43,6 +43,9 @@ try {
 } catch (e) {
     safeJsonStringify = null;
 }
+if (process.env.BUNYAN_TEST_NO_SAFE_JSON_STRINGIFY) {
+    safeJsonStringify = null;
+}
 
 // The 'mv' module is required for rotating-file stream support.
 try {
@@ -157,11 +160,16 @@ function getCaller3Info() {
 }
 
 
+function _indent(s, indent) {
+    if (!indent) indent = '    ';
+    var lines = s.split(/\r?\n/g);
+    return indent + lines.join('\n' + indent);
+}
+
+
 /**
  * Warn about an bunyan processing error.
  *
- * If file/line are given, this makes an attempt to warn on stderr only once.
- *
  * @param msg {String} Message with which to warn.
  * @param dedupKey {String} Optional. A short string key for this warning to
  *      have its warning only printed once.
@@ -822,17 +830,19 @@ Logger.prototype._emit = function (rec, noemit) {
     if (noemit || this.haveNonRawStreams) {
         try {
             str = JSON.stringify(rec, safeCycles()) + '\n';
-        }
+        } catch (e) {
-        catch(err) {
             if (safeJsonStringify) {
                 str = safeJsonStringify(rec) + '\n';
-            }
+            } else {
-            else {
+                var dedupKey = e.stack.split(/\n/g, 2).join('\n');
-                str = '(Could not JSON stringify data. See stderr for details.)';
+                _warn('bunyan: ERROR: Exception in '
-                _warn(format('bunyan: ERROR: Could not JSON stringify a property. '
+                    + '`JSON.stringify(rec)`. You can install the '
-                    + 'Please consider installing safe-json-stringify for more '
+                    + '"safe-json-stringify" module to have Bunyan fallback '
-                    + 'information. Stack trace: %s'
+                    + 'to safer stringification. Record:\n'
-                , err.stack));
+                    + _indent(format('%s\n%s', util.inspect(rec), e.stack)),
+                    dedupKey);
+                str = format('(Exception in JSON.stringify(rec): %j. '
+                    + 'See stderr for details.)\n', e.message);
             }
         }
     }

package.json

@@ -20,7 +20,7 @@
   "optionalDependencies": {
     "dtrace-provider": "~0.3 >0.3.0",
     "mv": "~2",
-    "safe-json-stringify": "1.0.1"
+    "safe-json-stringify": "~1"
   },
   "devDependencies": {
     "nodeunit": "0.9.*",

test/safe-json-stringify-1.js

@@ -8,4 +8,3 @@ var obj = {};
 obj.__defineGetter__('boom',
     function () { throw new Error('__defineGetter__ ouch!'); });
 log.info({obj: obj}, 'using __defineGetter__');
-

test/safe-json-stringify.test.js

@@ -62,6 +62,7 @@ test('defineProperty boom, without safe-json-stringify', function (t) {
         t.ok(stderr.indexOf(
             'You can install the "safe-json-stringify" module') !== -1);
         t.equal(stderr.match(
+            /* JSSTYLED */
             /You can install the "safe-json-stringify" module/g).length, 1);
         t.end();
     });