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

Prebid server: Custom logic for bidder adapter #6881

Closed
osazos opened this issue May 31, 2021 · 4 comments
Closed

Prebid server: Custom logic for bidder adapter #6881

osazos opened this issue May 31, 2021 · 4 comments
Labels

Comments

@osazos
Copy link
Collaborator

osazos commented May 31, 2021

Type of issue

  • question
  • feature request

Description

This issue is a question / feature request in order to let bidder adapters to add custom logic during the PrebidServer buildRequest process.

Actually there is only one method which interacts with the adapter: transformBidParams() which is fine but just pass the params set in their config (publisher side).

Can we improve the transformBidParams() method signature to pass adUnit and bidRequests? This way bidder adapters could take full advantage of this callback.

if (adapter && adapter.getSpec().transformBidParams) {
bid.params = adapter.getSpec().transformBidParams(bid.params, true);
}

Other information

#2844

@bretg
Copy link
Collaborator

bretg commented Jun 3, 2021

This is related to #6361 -- it's another reason why sometimes bidders need some client-side code even when pushed server-side.

@stale
Copy link

stale bot commented Jan 8, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@patmmccann
Copy link
Collaborator

it appears this is now superceded by the plan here #6361 (comment)

@osazos would that solve?

@patmmccann patmmccann moved this from Triage to Needs Author in Prebid.js Tactical Issues table Apr 21, 2022
@patmmccann
Copy link
Collaborator

tracking on #6361

Repository owner moved this from Needs OP to Done in Prebid.js Tactical Issues table Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

3 participants