chore(node): use native md5

Author: SimonSchick

packages/browser/src/md5.ts

@@ -237,8 +237,8 @@ function str2rstrUTF8(input: string) {
 }
 
 /*
-* Take string arguments and return either raw or hex encoded strings
-*/
+ * Take string arguments and return either raw or hex encoded strings
+ */
 function rawMD5(s: string) {
   return rstrMD5(str2rstrUTF8(s));
 }

packages/browser/src/tracekit.ts

@@ -502,11 +502,14 @@ TraceKit.report = (function reportModuleWrapper() {
     // slow slow IE to see if onerror occurs or not before reporting
     // this exception; otherwise, we will end up with an incomplete
     // stack trace
-    setTimeout(function() {
-      if (lastException === ex) {
-        processLastException();
-      }
-    }, stack.incomplete ? 2000 : 0);
+    setTimeout(
+      function() {
+        if (lastException === ex) {
+          processLastException();
+        }
+      },
+      stack.incomplete ? 2000 : 0,
+    );
 
     throw ex; // re-throw to propagate to the top level (and cause window.onerror)
   }

packages/node/package.json

@@ -22,13 +22,12 @@
     "@types/stack-trace": "0.0.29",
     "cookie": "0.3.1",
     "lsmod": "1.0.0",
-    "md5": "2.2.1",
     "stack-trace": "0.0.10",
     "tslib": "^1.9.3"
   },
   "devDependencies": {
     "@types/cookie": "0.3.1",
-    "@types/md5": "2.1.32",
+    "@types/node": "^10.12.10",
     "express": "^4.16.4",
     "jest": "^22.4.3",
     "npm-run-all": "^4.1.2",

packages/node/src/backend.ts

@@ -2,7 +2,7 @@ import { BaseBackend, Dsn, getCurrentHub, Options, SentryError } from '@sentry/c
 import { SentryEvent, SentryEventHint, SentryResponse, Severity } from '@sentry/types';
 import { isError, isPlainObject } from '@sentry/utils/is';
 import { limitObjectDepthToSize, serializeKeysToEventMessage } from '@sentry/utils/object';
-import * as md5 from 'md5';
+import { createHash } from 'crypto';
 import { extractStackFromError, parseError, parseStack, prepareFramesForEvent } from './parsers';
 import { HTTPSTransport, HTTPTransport } from './transports';
 
@@ -38,7 +38,11 @@ export class NodeBackend extends BaseBackend<NodeOptions> {
 
         getCurrentHub().configureScope(scope => {
           scope.setExtra('__serialized__', limitObjectDepthToSize(exception as {}));
-          scope.setFingerprint([md5(keys.join(''))]);
+          scope.setFingerprint([
+            createHash('md5')
+              .update(keys.join(''))
+              .digest('hex'),
+          ]);
         });
 
         ex = (hint && hint.syntheticException) || new Error(message);
@@ -100,8 +104,8 @@ export class NodeBackend extends BaseBackend<NodeOptions> {
       this.transport = this.options.transport
         ? new this.options.transport({ dsn })
         : dsn.protocol === 'http'
-          ? new HTTPTransport(transportOptions)
-          : new HTTPSTransport(transportOptions);
+        ? new HTTPTransport(transportOptions)
+        : new HTTPSTransport(transportOptions);
     }
 
     return this.transport.captureEvent(event);