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

Apply new Truncate Targeting Config to Key #2204

Merged
merged 1 commit into from
Apr 1, 2022

Conversation

AlexBVolcy
Copy link
Contributor

This PR is a follow up to #2181

In this PR, I update the addKeys function to also apply the truncation to the Targeting Key itself

@bsardo bsardo self-requested a review March 28, 2022 17:27
@bsardo bsardo self-assigned this Mar 28, 2022
@bsardo bsardo requested a review from VeronikaSolovei9 March 28, 2022 17:27
Copy link
Contributor

@VeronikaSolovei9 VeronikaSolovei9 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@bsardo bsardo left a comment

Choose a reason for hiding this comment

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

LGTM

@bsardo bsardo merged commit ee5613f into prebid:master Apr 1, 2022
jorgeluisrocha pushed a commit to jwplayer/prebid-server that referenced this pull request Sep 28, 2022
shunj-nb pushed a commit to ParticleMedia/prebid-server that referenced this pull request Nov 8, 2022
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