gamAdServerVideo: Add GPP consent information to the DFP video url#13761
gamAdServerVideo: Add GPP consent information to the DFP video url#13761robin-crazygames wants to merge 7 commits into
Conversation
|
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? |
|
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 |
|
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. |
|
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 |
|
@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 |
Pull Request Test Coverage Report for Build 22874035328Details
💛 - Coveralls |
|
@copilot resolve the merge conflicts in this pull request |
|
https://developers.google.com/ad-manager/dynamic-ad-insertion/sdk/html5/ccpa and https://developers.google.com/ad-manager/dynamic-ad-insertion/sdk/roku/ccpa seem to officially document this parameter I would say this is unblocked |
|
@robin-crazygames it seems it is gpp instead of gpp_string? |
|
At the moment, even when only GPP consent info is present, the Since the |
|
hopefully our friends at gam will weigh in :) I know the tachlab team is frustrated |
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
googletagscript for banner ads, you see it using'gpp_string'and'gpp_sid'to pass the GPP related information.Inspecting the obfuscated/minimized
googletagscript, you can also find that it is using:and
So I added those
gpp_stringandgpp_sidto thebuildVideoUrlfunction, and to thebidViewability(because that one was mentioned in issue 9364 as well)Other information
fixes #9364