Skip to content

Proxistore Bid Adapter: migration to OpenRTB#14411

Open
anthonyrichir wants to merge 9 commits intoprebid:masterfrom
proxistore:master
Open

Proxistore Bid Adapter: migration to OpenRTB#14411
anthonyrichir wants to merge 9 commits intoprebid:masterfrom
proxistore:master

Conversation

@anthonyrichir
Copy link
Contributor

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

This update is a migration of the Proxistore Bid Adapter to the OpenRTB spec, using Prebid's ortbConverter.

Other information

Anthony Richir and others added 6 commits May 8, 2025 11:07
Updated the Proxistore `COOKIE_BASE_URL` and `COOKIE_LESS_URL` to the new `abs` domain in both the adapter and its corresponding test file. This ensures consistency with the updated API endpoints.
…-specific URL selection, and enhance test coverage.
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9a7c228ea1

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 21667868150

Details

  • 258 of 258 (100.0%) changed or added relevant lines in 2 files are covered.
  • 51 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+33.1%) to 96.226%

Files with Coverage Reduction New Missed Lines %
src/adloader.js 12 78.57%
src/utils.js 39 90.21%
Totals Coverage Status
Change from base Build #3: 33.1%
Covered Lines: 209407
Relevant Lines: 217619

💛 - Coveralls

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.

2 participants