From 40bbaf168bc2bcf60d1d9bd21868f84fa88db909 Mon Sep 17 00:00:00 2001
From: Tim Down <tim.down@overleaf.com>
Date: Tue, 26 Apr 2022 10:37:22 +0100
Subject: [PATCH] Merge pull request #7744 from
 overleaf/td-sentry-tag-name-change

Ensure split test Sentry tag names are within the 32-character limit

GitOrigin-RevId: cc5cc5629b8b7e988d3ca6471892c8f90cf5171b
---
 services/web/frontend/js/infrastructure/error-reporter.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/services/web/frontend/js/infrastructure/error-reporter.js b/services/web/frontend/js/infrastructure/error-reporter.js
index 062f4b8f96..078b582fd7 100644
--- a/services/web/frontend/js/infrastructure/error-reporter.js
+++ b/services/web/frontend/js/infrastructure/error-reporter.js
@@ -57,7 +57,8 @@ function sentryReporter() {
         const splitTestAssignments = getMeta('ol-splitTestVariants')
         if (splitTestAssignments) {
           for (const [name, value] of Object.entries(splitTestAssignments)) {
-            Sentry.setTag(`ol.splitTest.${name}`, value.toString())
+            // Ensure Sentry tag name is within the 32-character limit
+            Sentry.setTag(`ol.${name}`.slice(0, 32), value.toString())
           }
         }