From 6d6851a27aa473c0cda0e59b9b8aa7eafed812d7 Mon Sep 17 00:00:00 2001 From: Sebastian Serth Date: Thu, 7 Sep 2023 22:50:14 +0200 Subject: [PATCH] Check for Sentry span before appending header for WebSocket Fix CODEOCEAN-FRONTEND-2X --- app/assets/javascripts/editor/execution.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/app/assets/javascripts/editor/execution.js b/app/assets/javascripts/editor/execution.js index 4988163e..01277422 100644 --- a/app/assets/javascripts/editor/execution.js +++ b/app/assets/javascripts/editor/execution.js @@ -11,10 +11,12 @@ CodeOceanEditorWebsocket = { // strip anchor if it is in the url sockURL.hash = ''; - sockURL.searchParams.set('HTTP_SENTRY_TRACE', span.toTraceparent()); - const dynamicContext = this.sentryTransaction.getDynamicSamplingContext(); - const baggage = SentryUtils.dynamicSamplingContextToSentryBaggageHeader(dynamicContext); - sockURL.searchParams.set('HTTP_BAGGAGE', baggage); + if (span) { + sockURL.searchParams.set('HTTP_SENTRY_TRACE', span.toTraceparent()); + const dynamicContext = this.sentryTransaction.getDynamicSamplingContext(); + const baggage = SentryUtils.dynamicSamplingContextToSentryBaggageHeader(dynamicContext); + sockURL.searchParams.set('HTTP_BAGGAGE', baggage); + } return sockURL.toString(); },