Skip to content

Comments

New adapter DPAI#14434

Open
driftpixelai wants to merge 4 commits intoprebid:masterfrom
driftpixelai:master
Open

New adapter DPAI#14434
driftpixelai wants to merge 4 commits intoprebid:masterfrom
driftpixelai:master

Conversation

@driftpixelai
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

Other information

Docs: prebid/prebid.github.io#6427

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: dd2a6198ea

ℹ️ 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".

isBidRequestValid: isBidRequestValid(),
buildRequests: buildRequests(AD_URL),
interpretResponse,
getUserSyncs: getUserSyncs(SYNC_URL)

Choose a reason for hiding this comment

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

P1 Badge Append CCPA consent to DPAI user syncs

Wiring getUserSyncs: getUserSyncs(SYNC_URL) here pulls in the teqblaze helper that only appends CCPA when uspConsent.consentString exists, but registerSyncs passes bidderRequest.uspConsent directly from the bidder request (a raw USP string in Prebid), so in real auctions this adapter’s sync URL will omit ccpa_consent even when USP consent is available. This can send sync traffic without the expected privacy signal in CCPA flows.

Useful? React with 👍 / 👎.

@coveralls
Copy link
Collaborator

coveralls commented Feb 10, 2026

Pull Request Test Coverage Report for Build 21954470584

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 211 of 217 (97.24%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 96.234%

Changes Missing Coverage Covered Lines Changed/Added Lines %
test/spec/modules/dpaiBidAdapter_spec.js 205 211 97.16%
Totals Coverage Status
Change from base Build 21826065111: 0.002%
Covered Lines: 210701
Relevant Lines: 218947

💛 - 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