Просмотр исходного кода

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
Tim Down 4 лет назад
Родитель
Сommit
40bbaf168b
1 измененных файлов с 2 добавлено и 1 удалено
  1. 2 1
      services/web/frontend/js/infrastructure/error-reporter.js

+ 2 - 1
services/web/frontend/js/infrastructure/error-reporter.js

@@ -57,7 +57,8 @@ function sentryReporter() {
         const splitTestAssignments = getMeta('ol-splitTestVariants')
         const splitTestAssignments = getMeta('ol-splitTestVariants')
         if (splitTestAssignments) {
         if (splitTestAssignments) {
           for (const [name, value] of Object.entries(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())
           }
           }
         }
         }