Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs issue - using the loader script section doesn't consider EU location organizations #12788

Open
3 tasks done
gilisho opened this issue Feb 20, 2025 · 6 comments
Open
3 tasks done
Assignees

Comments

@gilisho
Copy link

gilisho commented Feb 20, 2025

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/browser

SDK Version

7.120.3

Framework Version

No response

Link to Sentry event

https://monitoring-demo-europe.sentry.io/issues/?project=4508850509119568&referrer=sidebar

Reproduction Example/SDK Setup

This docs section has a mistake for organizations configured to be stored in EU data storage location.

Steps to Reproduce

Read the docs :)

Expected Result

The expected result is to have this link in the docs:

<script src="https://js-de.sentry-cdn.com/25948d48b722913301940234eef57a6f.min.js" crossorigin="anonymous"></script>

Actual Result

This is the loader script in the docs is this (see also screenshot below):

<script
  src="https://js.sentry-cdn.com/25948d48b722913301940234eef57a6f.min.js"
  crossorigin="anonymous"
></script>

Image

However, this link leads to here: https://js.sentry-cdn.com/25948d48b722913301940234eef57a6f.min.js
And it's a no-op script.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 20, 2025
@Lms24
Copy link
Member

Lms24 commented Feb 20, 2025

Hi @gilisho thanks for reporting! Did this work for you previously and you only got the no-op script recently? Or did it never work to begin with?

@gilisho
Copy link
Author

gilisho commented Feb 20, 2025

It never worked for me to begin with, but I tried to use it only recently.
Is the link expected to work only in js-de.sentry-cdn.com domain (meaning it's a docs issue) or is it supposed to work also in js.sentry-cdn.com domain?

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 20, 2025
@Lms24
Copy link
Member

Lms24 commented Feb 20, 2025

That's a good question, let me find out 😅 Tbh, I wasn't even aware that the loader has a js-de-specific version but it makes sense of course. I'll forward this internally to confirm. Seems like a docs issue to me on first glance though.

To confirm: Do you see the correct URL in your project settings?

Image

(In my case, I do, but I just wanna rule out some other issue)

@gilisho
Copy link
Author

gilisho commented Feb 20, 2025

Yes, this is the URL I see on my end 😄

Image

@Lms24
Copy link
Member

Lms24 commented Feb 20, 2025

Thanks, I'm moving this to the docs repo then and we already asked someone internally to take a look.

@Lms24 Lms24 transferred this issue from getsentry/sentry-javascript Feb 20, 2025
@getsantry
Copy link
Contributor

getsantry bot commented Feb 20, 2025

Auto-routing to @getsentry/product-owners-issues for triage ⏲️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

4 participants