-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
check if bidder exists before making request object #2695
Conversation
@jsnellbaker Pulled this branch and confirmed it fixes #2677. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM but why do we have 2 places we are checking this / giving an error message:
https://github.com/prebid/Prebid.js/blob/master/src/adaptermanager.js#L352
@mkendall07 I removed the redundant check in the |
@jsnellbaker this has conflicts that need to be resolved. |
@harpere the conflict should be resolved now. |
* check if bidder exists before making request object * remove redundant check
* check if bidder exists before making request object * remove redundant check
* check if bidder exists before making request object * remove redundant check
* check if bidder exists before making request object * remove redundant check
* check if bidder exists before making request object * remove redundant check
* check if bidder exists before making request object * remove redundant check
Type of change
Description of change
To fix the issue #2677
Currently when an invalid/unknown bidder is used in an auction, a request object is still made for that bidder. When the auction is running, it will recognize the bidder wasn't real and would return an error. But the
done
related code used latter in the auction would still try to wait for that bidder's request to respond causing the auction to wait for the timeout.This fix performs the check to validate the bidder earlier, and if the bidder is rejected then the request object doesn't get made. This streamlines the latter logic to only wait for bidders that are real/registered.