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

PBS bid adapter: s2s config is not formatted correctly when it's an array with string endpoints. #11878

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

edmonl
Copy link
Contributor

@edmonl edmonl commented Jun 27, 2024

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Updated bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

While this is obviously a bug, the symptom is that PBS is not called, when s2sConfig is an array with string type endpoints. An error is logged for "Check endpoints".

Other information

@patmmccann patmmccann changed the title Trivially fix a bug: s2s config is not formatted correctly when it's an array with string endpoints. PBS bid adapter: s2s config is not formatted correctly when it's an array with string endpoints. Jun 27, 2024
@patmmccann patmmccann self-assigned this Jun 27, 2024
@patmmccann patmmccann requested a review from dgirardi June 27, 2024 16:08
Copy link

@jbienkowski-fandom jbienkowski-fandom left a comment

Choose a reason for hiding this comment

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

LGTM

@patmmccann patmmccann merged commit c1c1a76 into prebid:master Jun 27, 2024
5 checks passed
@edmonl edmonl deleted the fixs2sconfigarrayendpointsbug branch June 27, 2024 16:25
@patmmccann
Copy link
Collaborator

@edmonl great catch, released immediately for you

DecayConstant pushed a commit to mediavine/Prebid.js that referenced this pull request Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants