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

Feature Request: Dynamic Bidder Modules #11862

Open
ReinoutStevens opened this issue Jun 25, 2024 · 5 comments
Open

Feature Request: Dynamic Bidder Modules #11862

ReinoutStevens opened this issue Jun 25, 2024 · 5 comments

Comments

@ReinoutStevens
Copy link
Contributor

Type of issue

Feature Request

Description

As a publisher, we're adding more and more partners. Many of these partners are country-specific (e.g. they only run in 1 specific country). Having many partners has 2 downsides:

  • it increases the bundle size of Prebid
  • Some bid adapters fetch and execute external JS, even when that bid adapters is not used in the auction

While we can mitigate this problem by building multiple prebid versions ourselves, it doesn't really scale when adding more country-specific partners.

It would be better if Prebid supported dynamically loading in bid adapters, either through a specific api, or the moment the bidder is used in an auction for the first time.

@patmmccann
Copy link
Collaborator

Which bid adapter runs external js ?

@ReinoutStevens
Copy link
Contributor Author

ReinoutStevens commented Jun 25, 2024

Afaik in our case that was Criteo and Adagio. Adagio loads in external stuff, even when they are not used in the auction. Not sure whether that was the case for Criteo, or if that only happens when they were part of the auction.

@patmmccann
Copy link
Collaborator

Those are both no longer in #11720 :)

@patmmccann
Copy link
Collaborator

patmmccann commented Jun 28, 2024

#6221 is a dupe btw, what do you think of the resolution there, to rely on #5287? There's demand for this with id modules too, no need to call some id modules until you've got an authenticated user

@ReinoutStevens
Copy link
Contributor Author

The approach of #5287 would work for us, and we could just use dynamic imports in our ads wrapper

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for Dev
Development

No branches or pull requests

2 participants