sentry.server.config.js fills up with globalThis

[johnnydecimal]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/astro

SDK Version

9.41.0

Framework Version

Astro 5.12.3

Link to Sentry event

No response

Reproduction Example/SDK Setup

https://github.com/johnnydecimal/astro-sentry

Steps to Reproduce

1. Run npm run astro check, or a build.
2. Check sentry.server.config.js.
3. Do it again and watch the globalThis entries just pile up.

Expected Result

I don't know if we're expecting a single globalThis block there? Presumably it has some purpose? But we definitely aren't expecting them to just be concatenated to the end.

FWIW mine are 350+ lines long each. The file grows out of control real fast.

Actual Result

As above.

[linear]

JS-774 sentry.server.config.js fills up with globalThis

[johnnydecimal]

Note that the example repo has 9.42.0 but I confirmed that this change occurred at 9.41.0.

Ah here we go. Commit 726c86842915111bca94d3787bc203ba05cd07e7 introduced this:

sentry-javascript/packages/astro/src/integration/index.ts

Line 184 in 68d7ee4

const updatedServerInitContent = `${serverInitContent}\nglobalThis["__sentryRouteInfo"] = ${JSON.stringify(

To my untrained eye it's not checking whether that already exists. Hence every time you just get a new block.

I'd offer to PR this but believe me you don't want my code.

[chargome]

@johnnydecimal thanks for reporting!

cc @s1gr1d this comes from the parameterized routes PR I assume?

[s1gr1d]

Hello and thanks for raising this. I reverted the changes for now and we will release this as soon as possible.

[s1gr1d]

This is released in 9.42.1.