remove unused code to overwrite filenames for Sentry error reporting

MetaMask · Aug 1, 2023 · d0cfe1a · d0cfe1a
1 parent c2de64e
commit d0cfe1a
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/app/util/sentryUtils.js b/app/util/sentryUtils.js
@@ -34,6 +34,7 @@ function rewriteBreadcrumb(breadcrumb) {
   if (breadcrumb.data?.from) {
     breadcrumb.data.from = getProtocolFromURL(breadcrumb.data.from);
   }
+  console.log('Breadcrumb', breadcrumb);
 
   return breadcrumb;
 }
@@ -73,17 +74,8 @@ function simplifyErrorMessages(report) {
 
 function rewriteReportUrls(report) {
   // update request url
-  report.request.url = toMetamaskUrl();
-  // update exception stack trace
-  if (report.exception && report.exception.values) {
-    report.exception.values.forEach((item) => {
-      if (item.stacktrace) {
-        item.stacktrace.frames.forEach((frame) => {
-          frame.filename = toMetamaskUrl(frame.filename);
-        });
-      }
-    });
-  }
+  console.log('rewriteReportUrls', report);
+  report.request.url = toMetamaskUrl(report.request.url);
 }
 
 function removeDeviceTimezone(report) {
@@ -96,12 +88,19 @@ function removeDeviceName(report) {
     report.contexts.device.name = null;
 }
 
-function toMetamaskUrl() {
-  const metamaskUrl = `metamask-mobile`;
-  return metamaskUrl;
+function toMetamaskUrl(origUrl) {
+    const filePath = origUrl?.split(location.origin)[1];
+    if (!filePath) {
+      return origUrl;
+    }
+
+    const metamaskUrl = 'metamask-mobile';
+    return metamaskUrl;
 }
 
+
 function rewriteReport(report) {
+  console.log('start', report.exception.values[0].stacktrace.frames);
   try {
     // simplify certain complex error messages (e.g. Ethjs)
     simplifyErrorMessages(report);
@@ -123,6 +122,8 @@ function rewriteReport(report) {
     throw err;
   }
 
+  console.log('Report', report.exception.values[0].stacktrace.frames);
+
   return report;
 }