Update ReadPeak adapter #2369
Update ReadPeak adapter #2369
Conversation
modules/readpeakBidAdapter.js
Outdated
| return undefined; | ||
| } | ||
|
|
||
| function referrer() { |
There was a problem hiding this comment.
This duplicates the utils function getTopWindowReferrer(). Please use that function
There was a problem hiding this comment.
Thanks, I missed that function. Fixed.
|
When I went to test these changes using the test params from the original adapter submission, I'm getting back 400 Bad Request request error |
|
I'm not sure what's going on. I see your tests in the log but they have the old format request. The old adapter code stopped working once we moved to tighter openrtb compliance in the back end. |
|
I don't mean to spam, but what is the status of this PR? The problem above @mike-chowla saw is most likely a caching issue or accidentally not having the latest code as the format of the request was different from what this PR has and the change requested was made. |
* Update ReadPeak adapter * Replace $$PREBID_GLOBAL$$.version with $prebid.version$ * Fix test * Use utils.getTopWindowReferrer() instead of custom function
Type of change
Description of change
Updates to the ReadPeak adapter to match changes to the ReadPeak system as well as some minor refactoring to clean up the adapter and make it a better citizen in Prebid.
kurre.stahlberg@readpeak.com