Skip to content

Commit 8aad213

Browse files
committed
Fix: Improve error resistance for Sentry initializer
We check `sentrySettings.data()['enabled']` to determine whether Sentry should be enabled. By adding an additional safe-guard, we ensure that even pages without a Sentry meta tag don't cause errors in the JavaScript console.
1 parent 5acecc4 commit 8aad213

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

app/assets/javascripts/base.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ $(document).on('turbo-migration:load', function() {
4646
const sentrySettings = $('meta[name="sentry"]')
4747

4848
// Workaround for Turbo: We must not re-initialize the Relay object when visiting another page
49-
if (sentrySettings && sentrySettings.data()['enabled'] && Sentry.getReplay() === undefined) {
49+
if (sentrySettings && sentrySettings.data() && sentrySettings.data()['enabled'] && Sentry.getReplay() === undefined) {
5050
Sentry.init({
5151
dsn: sentrySettings.data('dsn'),
5252
attachStacktrace: true,

0 commit comments

Comments
 (0)