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

Criteo bid adapter: write cookie only on TLD+1 #12323

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

dzhang-criteo
Copy link
Contributor

Type of change

  • Feature

Description of change

Avoid writing cookies on anything higher than TLD+1 level.
e.g. We should write on orange.fr and not on actu.orange.fr

@patmmccann
Copy link
Collaborator

Please title your pr with your adapter

@dzhang-criteo dzhang-criteo changed the title Write cookie only on TLD+1 Criteo bid adapter: write cookie only on TLD+1 Oct 14, 2024
@dzhang-criteo
Copy link
Contributor Author

Done! I have added an unit test as well.

@dzhang-criteo
Copy link
Contributor Author

Any update on this PR?

Avoid writing cookies on anything higher than TLD+1 level.
e.g. We should write on orange.fr and not on actu.orange.fr
@ChrisHuie ChrisHuie self-assigned this Oct 22, 2024
@ChrisHuie ChrisHuie merged commit fa44eac into prebid:master Oct 22, 2024
6 checks passed
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.

3 participants