prebid / Prebid.js

Setup and manage header bidding advertising partners without writing code or confusing line items. Prebid.js is open source and free.
https://docs.prebid.org
Apache License 2.0
1.28k stars 2.05k forks source link

PubMatic - Ad tracker not being fired #229

Closed mjwhaley closed 8 years ago

mjwhaley commented 8 years ago

We have recently added Pubmatic into the header and its bidding/rendering ads lovely. But Pubmatic aren't counting the impressions at all.

Has anyone seen issues with Pubmatic's ad tracker not being fired?

mkendall07 commented 8 years ago

@Heray Anyone using PubMatic that could chime in?

@jdelhommeau

Is this related to https://github.com/prebid/Prebid.js/issues/214 ?

mjwhaley commented 8 years ago

@jdelhommeau @Heray I'm in contact with Pubmatic and they believe that everything from PreBid 5.0 isn't working correctly. It's looking like it's not completely sending the whole URL string of the bid and thus the won impression isn't being tracked by Pubmatic. Pubmatic are seeing every ad request, the winning ad is being delivered and rendered to the page, but Pubmatic are counting nothing.

mkendall07 commented 8 years ago

@mjwhaley I checked the code and it looks like Pubmatic is returning a tracking_url that is not being fired. Can you send me your configuration details for Pubmatic so i can test this quickly?

mjwhaley commented 8 years ago

@mkendall07 I have a test page running here. Thanks for the help. http://streamamp.co.uk/advertising_github_prebid_6/

mkendall07 commented 8 years ago

@mjwhaley

Can you try with this build of prebid.js and see if it resolves the issue? prebid.js.zip

mjwhaley commented 8 years ago

@mkendall07 I have amended the follow page with the new build of prebid.js http://streamamp.co.uk/advertising_github_prebid_6/

mjwhaley commented 8 years ago

@mkendall07 Pubmatic and Myself are running some tests and will confirm back soon. Initial tests are looking positive though. Thanks for the help.

mkendall07 commented 8 years ago

@mjwhaley Good to hear. Please keep me posted.

mjwhaley commented 8 years ago

@mkendall07 This is now working and tracking paid impressions in Pubmatic. Thanks for all of you help. When do you think you will be updating the normal pull of prebid.js to reflect this change?

mkendall07 commented 8 years ago

@mjwhaley I merged the fix to master.