Skip to content

fix(core): Pass origin as referrer for lazyLoadIntegration #12766

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

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

jaulz
Copy link
Contributor

@jaulz jaulz commented Jul 4, 2024

I was referred by support team to this repository in order to suggest a fix for an issue we encountered. In our case the referrer is not passed to the feedback script and thus results in a 403 (because we set allowed domains in our project settings). Consequently also the modal doesn't open and it's a bit disappointing for us since we would like to use the feature.

@mydea mydea requested review from ryan953 and c298lee July 4, 2024 14:19
@mydea
Copy link
Member

mydea commented Jul 4, 2024

Hey, thanks for the PR! This sounds reasonable to me, but I also pulled in feedback folks to have a look!

Copy link
Contributor

@c298lee c298lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for the PR!

@mydea mydea self-assigned this Jul 5, 2024
@mydea mydea changed the title fix: pass origin as referrer fix(feedback): pass origin as referrer Jul 5, 2024
@mydea mydea changed the title fix(feedback): pass origin as referrer fix(core): Pass origin as referrer for lazyLoadIntegration Jul 5, 2024
@mydea mydea merged commit 1845bf4 into getsentry:develop Jul 5, 2024
61 of 62 checks passed
@jaulz
Copy link
Contributor Author

jaulz commented Jul 9, 2024

thanks as well for merging it 😊 When will it be available approximately?

@mydea
Copy link
Member

mydea commented Jul 9, 2024

We are preparing a release right now, so hopefully by tomorrow this should be out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants