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

51Degrees RTD submodule: add crossorigin attribute to script tag #12329

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

jwrosewell
Copy link
Contributor

Type of change

  • Updated RTD submodule

Description of change

  • Added crossorigin="anonymous" attribute to address an issue with Chromium making two separate connections on different sockets due to CORS restrictions.

This is happening because Chromium assigns different privacy modes (pm=disabled for the script and pm=enabled for the API call). When the privacy mode is enabled for one request and disabled for another, Chromium opens separate connections on different sockets.

By adding the crossorigin="anonymous" attribute, the script request is forced into pm=enabled mode, ensuring that both the script and subsequent API calls reuse the same socket. Using the same socket reduces the elapsed time to complete processing.

A big thanks to @BohdanVV for working on this.

@ChrisHuie ChrisHuie self-requested a review October 21, 2024 12:24
@ChrisHuie ChrisHuie self-assigned this Oct 21, 2024
@patmmccann
Copy link
Collaborator

related: #8500

@patmmccann
Copy link
Collaborator

I don't think we can change the default, as some scripts may depend on headers included.

@patmmccann patmmccann merged commit 1f0dba2 into prebid:master Oct 21, 2024
6 checks passed
pdamoc pushed a commit to missena-corp/Prebid.js that referenced this pull request Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants