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

GDPR support for AOL adapter #2443

Merged
merged 7 commits into from
Apr 30, 2018
Merged

Conversation

vzhukovsky
Copy link
Contributor

@vzhukovsky vzhukovsky commented Apr 22, 2018

Type of change

  • Feature

Description of change

GDPR support for AOL bid adapter.

@vzhukovsky vzhukovsky changed the title GDPR support for AOL bid adapter. GDPR support for AOL bid adapter Apr 22, 2018
@vzhukovsky vzhukovsky changed the title GDPR support for AOL bid adapter GDPR support for AOL adapter Apr 22, 2018
@jsnellbaker jsnellbaker self-requested a review April 23, 2018 15:58
@jsnellbaker jsnellbaker self-assigned this Apr 23, 2018
@jsnellbaker
Copy link
Collaborator

@vzhukovsky Thanks for submitting this PR. We're actually in the middle of updating the module to match the 1.1 spec. Would you be able to back-out the changes to the consentManagement and utils files?

@vzhukovsky
Copy link
Contributor Author

@jsnellbaker Sure. I'll do it when you update main branch for 1.1.

@jsnellbaker
Copy link
Collaborator

@vzhukovsky The GDPR branch has been updated for the 1.1 CMP spec. Please let me know when you've completed making the necessary updates in this PR and I'll take a look over it. Thanks!

@vzhukovsky
Copy link
Contributor Author

@jsnellbaker

PR is updated

@jsnellbaker
Copy link
Collaborator

jsnellbaker commented Apr 27, 2018

Hi @vzhukovsky I was trying to test out the adapter with using the test data from the .md but I'm not getting a proper ad unit back in the bid response.

Below is a copy of the request that was made and the corresponding response. Is there something special I need to do in the setup or is there an alternate set of test params I should use?

request:

http://adserver-us.adtech.advertising.com/pubapi/3.0/9599.1/3611253/0/0/ADTECH;v=2;cmd=bid;cors=yes;alias=37acca965a2a92;misc=1524836817529;bidfloor=0.80;kvtest=key

response:

{
	"id": "106681266954316283",
	"seatbid": [],
	"nbr": 1,
	"ext": {
		"pixels": "<script type='text/javascript'>document.write('<img src=\"http://cm.g.doubleclick.net/pixel?google_nid=adtech_dbm&google_cm&google_sc\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://adt.pxl.ace.advertising.com/cfcm.ashx?providerId=1010&amp;extMatch=1&amp;rcode=1\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://match.adsrvr.org/track/cmf/generic?ttd_pid=aoladtech\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<iframe src=\"http://ssum.casalemedia.com/usermatch?s=175407&cb=http%3A%2F%2Fpixel.advertising.com%2Fups%2F55940%2Fsync%3F_origin%3D0%26uid%3D\" width=\"1\" height=\"1\" style=\"display: none\"></iframe>');document.write('<img src=\"http://sync.mathtag.com/sync/img?mt_exid=21&redir=http://pixel.advertising.com/ups/55938/sync?uid=[MM_UUID]&_origin=0\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<iframe src=\"http://ad.turn.com/server/pixel.htm?fpid=28\" width=\"1\" height=\"1\" style=\"display: none\"></iframe>');document.write('<img src=\"http://x.bidswitch.net/sync?ssp=aol\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://p.rfihub.com/cm?in=1&pub=758&forward=http://pixel.advertising.com/ups/55856/sync?uid={userid}&_origin=0\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://ad.afy11.net/ad?mode=10&sspid=585\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://pixel.quantserve.com/pixel/p-NcBg8UA4xqUFp.gif?idmatch=0\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<img src=\"http://Aol-match.dotomi.com/aol/match?cb=http://pixel.advertising.com/ups/55853/sync?uid=$UID&_origin=0\" width=\"1\" height=\"1\" style=\"display:none;\">');document.write('<iframe src=\"http://us-u.openx.net/w/1.0/cm?id=9e0a35ea-c8e3-4b1b-9efa-4af6f54a373e&ph=e1e12abf-0bb5-47ee-ae31-55f011a265d0&r=http://pixel.advertising.com/ups/55981/sync?_origin=0&uid=\" width=\"1\" height=\"1\" style=\"display: none\"></iframe>');</script>"
	}
}

@jsnellbaker
Copy link
Collaborator

jsnellbaker commented Apr 27, 2018

Also, would you be able to open a docs PR in the docs repo and make an update in your bidders page to add the following variable?

gdpr_supported: true

This can go directly below the variable for your 1.0 compliance. This will allow your adapter to appear in a table that shows GDPR compliant adapters. Thanks.

@kizzard
Copy link
Contributor

kizzard commented Apr 27, 2018

Hi @jsnellbaker can you try removing the bidfloor parameter, or set it lower eg 0.10? You should then see bids.

@jsnellbaker
Copy link
Collaborator

Hi @kizzard, I tried your suggestions for the bidfloor (removing it and setting it to 0.10), but it didn't seem to make any difference. The response only contained pixels as before.

request (without bidfloor):
http://adserver-us.adtech.advertising.com/pubapi/3.0/9599.1/3611253/0/0/ADTECH;v=2;cmd=bid;cors=yes;alias=3b7616291068d6;misc=1524853374278;kvtest=key

@jsnellbaker
Copy link
Collaborator

@kizzard @vzhukovsky As an update, I just retried the test page in Firefox and I was seeing a bid come back successfully. I was initially testing in Chrome. Is there any targeting in place like that on the placement?

When you have the chance, please submit a doc PR and we should be good to merge.

Thanks!

@vzhukovsky
Copy link
Contributor Author

vzhukovsky commented Apr 28, 2018

@jsnellbaker docs added
prebid/prebid.github.io#730

@jsnellbaker jsnellbaker merged commit f042d7c into prebid:gdpr Apr 30, 2018
mkendall07 pushed a commit that referenced this pull request May 1, 2018
* initial commit

* wip update 2

* wip update 3

* example

* clean up

* wip update 3

* hook setup for callBids

* wip update 4

* changed gdpr code to be async-like

* cleaned up the callback chain

* added iab cmp detection logic

* moved hook, reverted unit test changes, and restructed gdpr module

* renaming module from gdpr to consentManagement

* prebidserver adatper update, additional changes in module

* updated unit tests for all areas, updates to module logic and structure of consent data

* adding missing default value

* removing accidentally committed load time testing code

* changes to layout of consentManagement code and other items based on feedback

* moved unit test to different location

* finished incomplete unit test in appnexusBidAdapter_spec file

* altered CMP function call logic

* refactored consentManagement AN lookup function and added gdprDataHandler to help transfer data in auction

* some minor cleanup from previous commit

* change spacing to try to fix travis issue

* added scenario to support consentTimeout=0 skip setTimeout

* updated some comments

* refactored exit logic for module

* added support for consentRequired field in config

* remove internal consentRequired default

* minor comment fixes

* comment fixes that should be have part of last commit

* fix includes issue and added gdprConsent to getUserSyncs function

* renamed default CMP and config field to cmpApi

* wip - using postmessage to call cmp

* postMessage workflow added, removed CMP eventlistener check

* removed if statement

* cleanup; removed variable and unneeded comments

* add gdpr tests pages

* updates for 1.1 CMP spec

* remove rogue debugger in unit test

* restructured 1.1 CMP iframe code, renamed utils function, cleaned up unit tests

* GDPR support in adform adapter (#2396)

* bid response adId same as bidId

* test

* update adform bid adapter

* update unit tests

* Added adform adapter description file

* updated tests

* Another tests update

* Add auctionId

* Update adapter for auctionId

* add auctionId to adformBidAdapter

* Final updates to fit 1.0 version

* update docs and integration example

* Do not mutate original validBidRequests

* use atob and btoa instead of custom made module

* Renaming one query string parameter

* XDomainRequest.send exception fix (#1942)

* Added YIELDONE Bid Adapter for Prebid.js 1.0 (#1900)

* Added YIELDONE Bid Adapter for Prebid.js 1.0

* Update yieldoneBidAdapter.md

change placementId to 44082

* Changed to get size from bid.sizes

* fix sizes array

* Add user-facing docs reminder to PR template (#1956)

* allow non-mappable sizes to be passed and used in rubicon adapter (#1893)

* Typo correction of YIELDONE md file (#1954)

* Added YIELDONE Bid Adapter for Prebid.js 1.0

* Update yieldoneBidAdapter.md

change placementId to 44082

* Changed to get size from bid.sizes

* fix sizes array

* Fix a typo

* Serverbid bid adapter: update alias config (#1963)

* use auctionId instead of requestId (#1968)

* Add freewheel ssp bidder adapter for prebid 1.0 (#1793)

* add stickyadsTV bidder adapter

* init unit test file

* ad some unit tests

* fix unit test on ad format with parameters

* add some unit tests

* add unit tests on getBid method

* add some test cases in unit tests

* minor fix on component id tag.

* remove adapters-sticky.json test file

* use top most accessible window instead of window.top

* Pass in the bid request in the createBid call.

* use top most accessible window instead of window.top

* add unit tests

* update unit tests

* fix unit test.

* fix CI build

* add alias freewheel-ssp

* update unit tests on bidderCode value

* fix component id values and add unit tests

* allws to use any outstream format.

* fix ASLoader on futur outstream format versions

* minor: fix code format.

* update unit tests

* minor fix code format

* minor: add missing new line at eof

* replace StickyAdsTVAdapter by freewheel ssp bd adapter (for prebid 1.0)

* remove old stickyadstv unittest spec.

* fix server response parsing if sent as object with 'body' field

* use the vastXml field for video mediatype

* add user sync pixel in freewheel ssp adapter

* remove all console log calls (replaced using util helper)

* remove useless bidderCode (automatically added by the bidderFactory)

* Return the SYNC pixel to be added in the page by Prebid.js

* remove instance level properties to enable concurrent bids with the same adapter instance.

* fix the request apss through and corresponding unit tests

* fix 'freeheelssp' typo

* + fixed endpoint request data property names - width to w and height to h (#1955)

+ updated unit test for the adapter to comply with the property name changes

* Added iQM Bid Adapter for Prebid.js 1.0 (#1880)

* Added iQM Bid Adapter for Prebid.js 1.0

* Modified URL from http to https

* Removed geo function which was fetching user location.

* Remove stray console.log (#1975)

* Remove duplicate request id and fix empty response from getHighesCpmBids, getAdserverTargeting (#1970)

* Removed requestId and added auctionId

* Updated module fixtures to use auctionId and not requestId

* remove request id from external bid object and fix bug for empty result in public api

* use auctionId instead of requestId

* fixed lint errors

* [Add BidAdapter] rxrtb adapter for Perbid.js 1.0 (#1950)

* Add: rxrtb prebidAdapter

* Update: params for test

* Update: code format

* Update: code format

* Update: code format

* ServerBid Server BidAdapter (#1819)

* ServerBid Server BidAdapter

Allow S2S configuration with ServerBid.

* Updates to meet 1.0 callBids/config changes.

* Fix linting issues.

* added hb_source to default keys (#1969)

* added hb_source

* dropped function to add hb_source since it is now default key

* fixed lint error

* Prebid 1.1.0 Release

* Increment pre version

* S2s defaults fix in serverbidServerBidAdapter (#1986)

* removed s2s defaults

* start timestamp was missing on s2s requests

* remove hardcoded localhost port for tests (#1988)

* Fixes unit tests in browsers other than chrome (#1987)

* Fixes unit tests in browsers other than chrome

* fixed lint errors

* Prebid 1.1.1 Release

* Add note about docs needed before merge (#1959)

* Add note about docs needed before merge

* Update pr_review.md

* Update pr_review.md

* Update pr_review.md

* Adding optional width and height to display parameters  (#1998)

* adding optional size

* no tabs

* TrustX adapter update (#1979)

* Add trustx adapter and tests for it

* update integration example

* Update trustx adapter

* Post-review fixes of Trustx adapter

* Code improvement for trustx adapter: changed default price type from gross to net

* Update TrustX adapter to support the 1.0 version

* Make requested changes for TrustX adapter

* Updated markdown file for TrustX adapter

* Fix TrustX adapter and spec file

* Update TrustX adapter: r parameter was added to ad request as cache buster

* Serverbid Bid Adapter: Add new ad sizes (#1983)

* Added dynamic ttl property for One Display and One Mobile. (#2004)

* pin gulp-connect at non-broken version (#2008)

* pin gulp-connect at non-broken version

* updated yarn.lock to specify pinned gulp-connect

* Gjirafa Bidder Adapter (#1944)

* Added Gjirafa adapter

* Add gjirafa adapter unit test

* adapter update

* New line

* Requested changes

* change hello_world.html to one bid

* change hello_world.html to one bid

* Dropping changes in gitignore and hello_world example

* hello_world changes

* Drop hello_world and gitignore

* multiformat size validation checks (#1964)

* initial commit for multiformat size validation checks

* adding unit tests and changes to checkBidRequestSizes function

* updates to appnexusBidAdapter

* Upgrade Admixer adapter for Prebid 1.0 (#1755)

* Migrating to Prebid 1.0

* Migrating to Prebid 1.0

* Fix spec

* Add NasmediaAdmixer adapter for Perbid.js 1.0 (#1937)

* add NasmediaAdmixer adapter for Perbid.js 1.0

* add NasmediaAdmixer adapter for Perbid.js 1.0

* add NasmediaAdmixer adapter for Perbid.js 1.0

* add NasmediaAdmixer adapter for Perbid.js 1.0

* add NasmediaAdmixer adapter for Perbid.js 1.0

* add NasmediaAdmixer adapter for Perbid.js 1.0

* Added gdpr to adform adapter

* Added unit tests

* Updated spacing

* Update gdprConsent object due to changes in spec

* Add gdpr support for PubMaticBidAdapter (#2469)

* GDPR support for AOL adapter (#2443)

* Added GDPR support for AOL adapter.

* Added unit tests for AOL GDPR changes.

* Added utils for resolving object type and undefined.

* Fixed issues caused by merge.

* Made changes in AOL adapter to support gdprApplies flag.

* Removed bid floor value from test bid config.

* removing iframe example pages

* comment updates
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.

None yet

4 participants