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

Auction ends prematurely [1.24.0] #3091

Closed
dugwood opened this issue Sep 13, 2018 · 8 comments · Fixed by #3096
Closed

Auction ends prematurely [1.24.0] #3091

dugwood opened this issue Sep 13, 2018 · 8 comments · Fixed by #3096
Labels

Comments

@dugwood
Copy link
Contributor

dugwood commented Sep 13, 2018

Type of issue

Bug

Description

After updating from 1.23.0 to 1.24.0 a drop in revenues occurred.

Steps to reproduce

Upgrade to 1.24.0 and look for the auction's end. If you have some debug, you'll see that auction ends before some valid bids.
Note that reading the releases' page didn't show any new parameter involved.

Test page

No test page as I've reverted back to 1.23.0 in production.

Expected results

All bids should arrive before auction ends.

Actual results

Auction ends in the middle of bids. Most probably Criteo (the fastest bid response) cut all other (but Criteo is also cut by other bidders sometimes).

Platform details

Compiled on Linux Debian, Node 8.11.4, npm 5.6.0

Other information

Change to 1.23.0 at 11:45, reverted at 17:00.

Sent bids (for AppNexus) vs returned bids (darker green):
image

Revenues (still only AppNexus):
image

Sent bids for Criteo, less impacted as it's the fastest:
image

@jaiminpanchal27
Copy link
Collaborator

@dugwood Can you share the url of the page ?

@dugwood
Copy link
Contributor Author

dugwood commented Sep 13, 2018

@jaiminpanchal27 I've reverted back... I'll try to setup one, I'll let you know.

@dugwood
Copy link
Contributor Author

dugwood commented Sep 13, 2018

@jaiminpanchal27 for a limited period of time:
https://www.lacoccinelle.net/prebid.html?20190916#debug-ads-for-skyboard-670168053

You should see some debugs, hide the «Zone 17980» and just focus on «Zone 438».

For example I can see:
image

=> here AppNexus answers within the 2000ms timeout, but rubicon wins. The «Winner» is a message sent in the bidsBackHandler... so the auction is already ended.

@dugwood
Copy link
Contributor Author

dugwood commented Sep 13, 2018

Numbers in gray are the time in milliseconds.

To enable Prebid debug, use the following url:
https://www.lacoccinelle.net/prebid.html?20190916#debug-ads-for-skyboard-670168053&pbjs_debug=true
(as GET parameters are forbidden)

@jaiminpanchal27
Copy link
Collaborator

@dugwood Thanks. Checking now

@jaiminpanchal27
Copy link
Collaborator

@dugwood Found the bug. We will do the patch release tomorrow. Thanks for contribution.

@jaiminpanchal27
Copy link
Collaborator

@dugwood Can you test with #3096 branch.

@dugwood
Copy link
Contributor Author

dugwood commented Sep 15, 2018

jaiminpanchal27 added a commit that referenced this issue Sep 17, 2018
* auction closing early fix

* minor updates after code review
janua pushed a commit to guardian/Prebid.js that referenced this issue Sep 24, 2018
* auction closing early fix

* minor updates after code review
ptomasroos pushed a commit to happypancake/Prebid.js that referenced this issue Sep 25, 2018
* auction closing early fix

* minor updates after code review
StefanWallin pushed a commit to mittmedia/Prebid.js that referenced this issue Sep 28, 2018
* auction closing early fix

* minor updates after code review
SublimeJeremy pushed a commit to SublimeSkinz/Prebid.js that referenced this issue Oct 1, 2018
* auction closing early fix

* minor updates after code review
ghost pushed a commit to devunrulymedia/Prebid.js that referenced this issue Jan 30, 2019
* auction closing early fix

* minor updates after code review
pedrolopezmrf pushed a commit to Marfeel/Prebid.js that referenced this issue Mar 18, 2019
* auction closing early fix

* minor updates after code review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants