From d0cfe1adeb3b756bf45483f84e70f89f789ba851 Mon Sep 17 00:00:00 2001 From: sethkfman Date: Tue, 1 Aug 2023 16:15:54 -0600 Subject: [PATCH] remove unused code to overwrite filenames for Sentry error reporting --- app/util/sentryUtils.js | 29 +++++++++++++++-------------- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/app/util/sentryUtils.js b/app/util/sentryUtils.js index 9701c1083af..f5db117adc2 100644 --- 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; }