From dc365fa7de872e2487a05bf8ed575d39eaea3af3 Mon Sep 17 00:00:00 2001 From: Kosty Maleyev Date: Tue, 12 Mar 2024 09:05:01 -0700 Subject: [PATCH] [react] Fix grouping by SE "supertag" --- react/src/index.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/react/src/index.js b/react/src/index.js index a635ac562..1d736a68b 100644 --- a/react/src/index.js +++ b/react/src/index.js @@ -128,12 +128,20 @@ Sentry.init({ }); if (se) { + const se_tda_prefix_regex = /[^-]+-tda-[^-]+-/ + let se_fp = se; + let prefix = se_tda_prefix_regex.exec(se); + if (prefix) { + // Now that TDA puts platform/browser and test path into SE tag we want to prevent + // creating separate issues for those. See https://github.com/sentry-demos/empower/pull/332 + se_fp = prefix[0] ; + } if (se.startsWith('prod-tda-')) { // Release Health - event.fingerprint = ['{{ default }}', se, RELEASE]; + event.fingerprint = ['{{ default }}', se_fp, RELEASE]; } else { // SE Testing - event.fingerprint = ['{{ default }}', se]; + event.fingerprint = ['{{ default }}', se_fp]; } }