-
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
Azerion / Improve: does not properly support currency module #12238
Comments
Ideally bid adapters should know what currencies they can bid on and wouldn't ask their exchanges for a currency they can't deal with; adapters that do should be fixed individually. If we accept that publishers are responsible for the fix (I'm not sure we should), some options are:
|
cc @jbartek25 please see rixEngine as an example on how to handle |
My take is that this is a bidder issue. Prebid.js should enhance the bid adapter developer documentation to better explain currency - maybe something similar to what's in the PBS developer doc? That said, publisher frustration is understandable, and we should help communicate with bid adapter maintainers on this issue. |
OP: setBidderConfig({bidders: ['improvedigital'], config: {ortb2: {cur: ['USD']}}) works; we do need something cleaner |
There are 4 bidder currency scenarios that I'm aware of:
|
Bidder's perspective:
They don't even understand where the "currency" is coming from. |
Hi @dmitriyshashkin. Please give me a test page where I can check the issue because if Improve ad server ignores any unknown currencies, it responds with bids in USD that will then get converted to the adServerCurrency by Prebid. Test for example with ZAR. |
That's what we get when we request bids. We're using alternative currency conversion file with additional currencies: https://cdn.membrana.media/currency_file.json And both our account manager and your tech support told us the following:
They even sent us a report that says that our requests are being rejected because "No supported currency is specified" |
@dmitriyshashkin I put together a test page with your publisher id and you currency file + adServerCurrency "UAH". Improve Digital bidder handles such request correctly. See for yourself: https://test-hb.improvedigital.com/pbw/examples/improve-cur-test.html |
I took your example and replaced the placement id. Now I get 400 error https://js-debug-2.b-cdn.net/improve.html With your placement id it produces bid every time. Not sure what should I make of this. I mean, above I provided an excerpt from the report provided by our account manager at improvedigital. It explicitly says that the main reason why our requests were rejected is the lack of support for the currency. And improvedigital's tech support even checked our setup and told us that the unsupported currency is the problem. Perhaps you're using some debug placement id that circumvents the standard logic and always bids no matter the currency? |
@dmitriyshashkin the placement you use at https://js-debug-2.b-cdn.net/improve.html is a video placement and the request is for banner ads - that's why the request gets rejected with error 400. I spoke with the Improve support team to get to the bottom of when "No supported currency is specified" error was seen. It turns out you were sending bidfloorcur=UAH. In this case I'm not sure what else the ad server can do other than rejecting the request as the clients wouldn't appreciate the ad server ignoring the floors in (for the ad server) unknown currencies. |
@jbartek25 - bid adapters have a function available to change the floor currency to something they can handle. See https://docs.prebid.org/prebid-server/developers/add-new-bidder-go.html#currency - maybe this would help this situation? |
thanks @bretg . I'll look into that |
We’ll let publishers hide the currency as ‘sharedCurrency’, and it will override adserverCurrency in the request object. |
I accidentally unassigned. Doesn't matter, @lyubomirshishkov will take care of the fix so the issue can be assigned to him. |
Type of issue
Feature request
Description
The currency module is supposed to convert bids to the ad server currency on the client. Instead it leaks it to the bid requests (e.g. through openrtb converter https://github.com/prebid/Prebid.js/blob/master/libraries/ortbConverter/README.md?plain=1#L45).
This is problematic, cause some of the bidders do not understand the purpose of the currency module (e.g. improvedigital, previously adform). And if they receive bid request with a currency they do not recognise, they don't bid.
The only workaround for me I see currently that does not include patching Prebid is to disable the currency module completely and implement it separately from Prebid code.
Steps to reproduce
Set currency to something like "UAH" or "ZWD". Add improvedigital. Request bids
Expected results
Bid request does not contain information from the currency module. Bidder responds with a bid in USD and currency module converts it to whatever currency is used on ad server.
Actual results
Bidder responds with error.
Other information
Same problem I had with adform previously #8566
The text was updated successfully, but these errors were encountered: