Update datadog serializer to properly handle non-object errors

Author: Dakrs

src/index.js

@@ -4,7 +4,7 @@
  * Module dependencies.
  */
 
-const { cloneDeep, cloneDeepWith, escapeRegExp, get, set } = require('lodash');
+const { cloneDeep, cloneDeepWith, escapeRegExp, get, isPlainObject, set } = require('lodash');
 const { serializeError: errorSerializer } = require('serialize-error');
 const stringify = require('json-stringify-safe');
 const traverse = require('traverse');
@@ -214,6 +214,13 @@ module.exports.anonymizer = (
  */
 
 function datadogErrorSerializer(error) {
+  if (!isPlainObject(error)) {
+    return {
+      details: error,
+      kind: 'Error'
+    };
+  }
+
   return {
     ...error,
     kind: error.name || 'Error'

test/src/index.test.js

@@ -832,26 +832,48 @@ describe('Anonymizer', () => {
           const serializer = jest.fn(builtinSerializers.datadogError);
           const serializers = [
             { path: 'err', serializer },
+            { path: 'errArr', serializer },
+            { path: 'errStr', serializer },
             { path: 'error', serializer }
           ];
-          const whitelist = ['error.foo', 'error.kind', 'error.message', 'error.name', 'error.stack'];
+          const whitelist = [
+            'errArr.details.*',
+            'errArr.kind',
+            'errStr.details',
+            'errStr.kind',
+            'error.foo',
+            'error.kind',
+            'error.message',
+            'error.name',
+            'error.stack'
+          ];
           const anonymize = anonymizer({ whitelist }, { serializers });
 
           const result = anonymize({
             e: error,
             err: {
               statusCode: 400
             },
+            errArr: ['foo', 'bar'],
+            errStr: 'foobar',
             error,
             error2: error,
             foo: 'bar'
           });
 
-          expect(serializer).toHaveBeenCalledTimes(2);
+          expect(serializer).toHaveBeenCalledTimes(4);
           expect(result.err).toEqual({
             kind: '--REDACTED--',
             statusCode: '--REDACTED--'
           });
+          expect(result.errArr).toEqual({
+            details: ['foo', 'bar'],
+            kind: 'Error'
+          });
+          expect(result.errStr).toEqual({
+            details: 'foobar',
+            kind: 'Error'
+          });
           expect(result.error).toEqual({
             kind: 'Error',
             message: 'foobar',