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

OpenX adapter does not pass verizonmediaid in bid request. #6930

Closed
nanek opened this issue Jun 3, 2021 · 4 comments
Closed

OpenX adapter does not pass verizonmediaid in bid request. #6930

nanek opened this issue Jun 3, 2021 · 4 comments
Labels

Comments

@nanek
Copy link
Contributor

nanek commented Jun 3, 2021

Type of issue

Bug

Description

OpenX adapter does not pass verizonmediaid in bid request.

Steps to reproduce

Enable verizon media connectid https://docs.prebid.org/dev-docs/modules/userId.html#verizon-media-connectid

pbjs.getUserIds() returns { connectid: "tOwehSrOwhAUJc5zgdsmXE3sTbHfJtBseTSwB-tBW2K9ghF6S0LZ1oE1MMwmG5d4MUfE2oRTeO4e33goY-crQA" } as expected, and the AOL adapter passes the connectid, however OpenX does not include this value in their bid requests.

https://github.com/prebid/Prebid.js/pull/6490/files#diff-1b7b8529cd571697ee8f92b2d0214da8e41c7ab8a4cb1e87be5db4083dc03ab5R34 I believe should be connectid:'verizonmediaid'

Test page

Expected results

Actual results

Platform details

All

Other information

#6490

@patmmccann
Copy link
Collaborator

@slimkrazy fyi

@patmmccann patmmccann added the bug label Jun 3, 2021
@slimkrazy
Copy link
Contributor

Hey @patmmccann - I'm not really sure why this bid adapter is attempting to consume the ConnectID as connectID is not designed currently to be used by any other adapter than Verizon Media adapters. The ID is encrypted and can't be used by other DSPs/SSPs.

In short: Verizon Media had no knowledge of this change and there is no benefit to a publisher if this feature works or not for this bid adapter.

@patmmccann
Copy link
Collaborator

@bwschmidt fyi

@bwschmidt
Copy link
Contributor

@patmmccann thanks for the heads up, i'll check internally and likely have a PR to remove shortly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants