Skip to content

gamAdServerVideo: Add GPP consent information to the DFP video url#13761

Open
robin-crazygames wants to merge 7 commits into
prebid:masterfrom
robin-crazygames:gpp_string_for_video_url
Open

gamAdServerVideo: Add GPP consent information to the DFP video url#13761
robin-crazygames wants to merge 7 commits into
prebid:masterfrom
robin-crazygames:gpp_string_for_video_url

Conversation

@robin-crazygames

@robin-crazygames robin-crazygames commented Aug 12, 2025

Copy link
Copy Markdown
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

When using pbjs.adServers.dfp.buildVideoUrl, the url would contain GDPR related information in GDPR countries, but not the GPP related information in GPP countries.

I cannot find any links to the official Google Ad Manager docs on how to provide this information. However, when using the googletag script for banner ads, you see it using 'gpp_string' and 'gpp_sid' to pass the GPP related information.

Inspecting the obfuscated/minimized googletag script, you can also find that it is using:

a.pingData.gppString

and

a.pingData.applicableSections.join(',')

So I added those gpp_string and gpp_sid to the buildVideoUrl function, and to the bidViewability (because that one was mentioned in issue 9364 as well)

Other information

fixes #9364

@patmmccann

patmmccann commented Aug 12, 2025

Copy link
Copy Markdown
Collaborator

Thanks for you submission! I contacted Google and they certainly do not have this feature documented at https://support.google.com/admanager/answer/10678356 yet although there may be unstable and unofficial support before a release. Does this feature solve some issue for you?

@robin-crazygames

Copy link
Copy Markdown
Contributor Author

Solving an issue might be a big word. What we do for videos is first having a Prebid auction. And then we use the URL generated by Prebid to hold another auction on GAM.

I would assume that, for the auction on GAM, passing in the GPP consent string will result in better bids

@patmmccann

Copy link
Copy Markdown
Collaborator

We contacted Google, we're not sure adding this would improve monetization in the short term. We expect them to release official support soon and we can try and merge then so we don't subject publishers to a sudden behavior change when Google announces official support.

@jimee02

jimee02 commented Aug 26, 2025

Copy link
Copy Markdown
Contributor

Per https://support.google.com/admanager/answer/10678356?sjid=8899252983232341054-NC, gpp params are not supported, but I have seen them on there before. Adjacent to this we should add us_privacy to our targeting as well. I'll create a ticket on our side to add this

@jimee02

jimee02 commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

@patmmccann ran an experiment on our side and it did not provide any revenue lift from what we can tell. I'd continue holding off on this PR

@coveralls

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 22874035328

Details

  • 7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
  • 46 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+33.2%) to 96.327%

Files with Coverage Reduction New Missed Lines %
src/adloader.js 9 76.92%
src/utils.js 37 90.51%
Totals Coverage Status
Change from base Build #3: 33.2%
Covered Lines: 217517
Relevant Lines: 225811

💛 - Coveralls

@patmmccann

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

@patmmccann

Copy link
Copy Markdown
Collaborator

@patmmccann

Copy link
Copy Markdown
Collaborator

@robin-crazygames it seems it is gpp instead of gpp_string?

@barecheck

barecheck Bot commented Jul 9, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 96.58%

Your code coverage diff: 0.00% ▴

Uncovered files and lines
FileLines
modules/gamAdServerVideo.js254, 297, 307, 324, 358-359

Comment thread libraries/dfpUtils/dfpUtils.js Outdated
@robin-crazygames

Copy link
Copy Markdown
Contributor Author

At the moment, even when only GPP consent info is present, the gamAdServerVideo code derives a value for us_privacy based on the GPP info.

Since the gpp string is still not present on https://support.google.com/admanager/answer/10678356?hl=en#zippy=%2Cquickly-navigate-to-any-vast-parameter, I wonder whether we should pass both the gpp and derived us_privacy, or just stick to gpp only.

@patmmccann

Copy link
Copy Markdown
Collaborator

hopefully our friends at gam will weigh in :) I know the tachlab team is frustrated

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.

Passing GPP data for bidViewability and dfpAdServerVideo modules follow-up

5 participants