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.

Sign up for GitHub

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

Jump to bottom

Fix: BidRequest updated by ProcessedAuction hook is getting overridde… #3045

Merged
merged 1 commit into from
Sep 7, 2023
Merged

Fix: BidRequest updated by ProcessedAuction hook is getting overridde… #3045

merged 1 commit into from
Sep 7, 2023

Conversation

pm-nikhil-vaidya
Copy link
Contributor

@pm-nikhil-vaidya pm-nikhil-vaidya commented Aug 22, 2023 •
edited
Loading

…n by requestwrapper utility

Issue : #2986

@hhhjort hhhjort self-requested a review August 30, 2023 17:11
hhhjort
hhhjort approved these changes Aug 30, 2023
View reviewed changes
Copy link
Collaborator

@hhhjort hhhjort 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. We will have to keep an eye on module submissions to insure they don't abuse it.

@hhhjort hhhjort merged commit 9980b64 into prebid:master Sep 7, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexBVolcy AlexBVolcy AlexBVolcy approved these changes

@hhhjort hhhjort hhhjort approved these changes

@gargcreation1992 gargcreation1992 Awaiting requested review from gargcreation1992

Assignees

@guscarreon guscarreon

@hhhjort hhhjort

Labels
do not port
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@pm-nikhil-vaidya @hhhjort @AlexBVolcy @SyntaxNode @guscarreon