Skip to content

Cnx bidder improvements#37

Merged
karimMourra merged 6 commits intopbs_v3.15.0-cnx-testfrom
cnx-improvements
Apr 5, 2025
Merged

Cnx bidder improvements#37
karimMourra merged 6 commits intopbs_v3.15.0-cnx-testfrom
cnx-improvements

Conversation

@karimMourra
Copy link

Miscelaneous improvements:

  • instead of unmarshalling the imp.ext which is expensive, we read the property value.
  • copies imp object instead of modifying it directly on the bid request
  • does endpoint modification outside of loop; it is only needed once per request, not per impression

@karimMourra karimMourra requested a review from dshore April 5, 2025 00:11
@karimMourra karimMourra merged commit 33fbc72 into pbs_v3.15.0-cnx-test Apr 5, 2025
1 of 3 checks passed
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.

1 participant