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

Criteo bid adapter: Bump fastbid version #10696

Merged
merged 1 commit into from
Nov 26, 2023

Conversation

leonardlabat
Copy link
Contributor

Type of change

  • Bugfix

Description of change

Bump fast bid version to ensure the latest fix is available

@DemekaParis
Copy link

#10456 (review)

@DemekaParis
Copy link

Just wanted to ask what is a reason for this bid adapter to have external .js loaded. After checking many other bid adapters I haven't found any other DSP bidAdapter using this technology. If prebid.js is generaly running on a fair "max CPM wins policy" this could also violate some Prebid policy / or use internal prebid.js calls hidden to all other adapters?

@allanjun
Copy link
Collaborator

allanjun commented Nov 7, 2023

hi @DemekaParis

Thanks for raising this, you will want to follow progress on #10653 that will address your very concern around loading external files. This feature is legacy design and allowed Criteo to run some features not available in Prebid.js by the time (Criteo ID Module, timeout stats and other minor things).

Criteo team is confident and committed to address this with Prebid 9.0 or even before. Please also notice that today it is already possible to optout from this feature.

Allan

@patmmccann
Copy link
Collaborator

We're not bumping fastbid any longer

@patmmccann
Copy link
Collaborator

@DemekaParis see also #4031 (comment) and #5804 as useful background.

@patmmccann
Copy link
Collaborator

Reopened as the Fastbid changes are minor bug fixes that I reviewed, awaiting board approval before this merge.

@DemekaParis
Copy link

I believe making the repository at https://github.com/Prebid-org/prebid-js-external-js-criteo/tree/master/dist public would be mutually beneficial at this time, especially until the anticipated PR for prebid 9 is implemented. If there are no proprietary elements involved, I see no issue with this approach. Alternatively, consider presenting it as an opt-in feature for publishers, as I am confident you can effectively communicate the advantages directly to them.

I would like to express my concern about the current opt-out mechanism, as it seems impractical. Referring to

If you don't want to use the FastBid adapter feature, you can lighten criteoBidAdapter size by :
, it appears that opting out is not straightforward when downloading from the prebid.org website. Users have to modify the source code to "opt-out," as indicated below:

  If you don't want to use the FastBid adapter feature, you can lighten criteoBidAdapter size by :
  1. commenting the tryGetCriteoFastBid function inner content (see ref#1)
  2. removing the line 'verify' function import line (see ref#2)`

Also this opt-out/opt-in should be written in https://docs.prebid.org/dev-docs/bidders/criteo.html where there is no mention of opt-out.

I appreciate your consideration of these points and believe addressing them would contribute to a smoother user experience.

@allanjun
Copy link
Collaborator

hi @DemekaParis these instructions are build-time instructions. A runtime optout is also possible, it is explained on our integration guides but we will also document on https://docs.prebid.org/dev-docs/bidders/criteo.html 👍

pbjs.que.push(function () {
  pbjs.setConfig({
    'criteo': {
      fastBidVersion: 'none',
    }
  });
});

@patmmccann
Copy link
Collaborator

Ad discussed with the criteo team, this bug fix is the final bump

@patmmccann patmmccann merged commit 1205d53 into prebid:master Nov 26, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants