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

Appnexus BidAdapter: It may not be possible to use currencies other USD #11994

Closed
saitoukun opened this issue Jul 18, 2024 · 5 comments · Fixed by #11995
Closed

Appnexus BidAdapter: It may not be possible to use currencies other USD #11994

saitoukun opened this issue Jul 18, 2024 · 5 comments · Fixed by #11995
Assignees

Comments

@saitoukun
Copy link
Contributor

saitoukun commented Jul 18, 2024

Type of issue

  • Adapter and server mismatch

Description

  • We use JPY currency.
  • currency will use publisher_currency_codename from Server's Bid. This case is JPY
  • But cpm is always seems to be USD
  • As a result, the currency exchange process was not executed and was regarded as a very low cpm.

This means that the server may only support USD

Platform details

There was no problem up to 8.50.0
I got problems since 8.51.0

Appnexus Bid Adapter: parse the currency from the bid if specified
https://github.com/prebid/Prebid.js/releases/tag/8.51.0

Other information

related PR
#11581

@saitoukun
Copy link
Contributor Author

I'm contacting appnexus for specs.

@saitoukun saitoukun changed the title Appnexus BidAdapter: Possibility to not able to use any currency other USD Appnexus BidAdapter: Possibility to be not able to use any currency other USD Jul 18, 2024
@saitoukun saitoukun changed the title Appnexus BidAdapter: Possibility to be not able to use any currency other USD Appnexus BidAdapter: It may not be possible to use currencies other USD Jul 18, 2024
@saitoukun
Copy link
Contributor Author

Here is sample of bid responce.

{
  "cpm": 0.477453,
  "cpm_publisher_currency": 74.943749,
  "publisher_currency_code": "¥",
  "publisher_currency_codename": "JPY",
  "content_source": "rtb",
  "ad_type": "banner",
  "media_type_id": 1,
  "media_subtype_id": 1,
  "brand_category_id": 180,
  "is_sov": false,
  "is_roadblock": false,
  "bidder_id": 440,
  "client_initiated_ad_counting": true,
  "is_managed": false,
  "no_consent": false,
}
  • publisher_currency_codename is JPY
  • It seems to be cpm is always USD
  • I found cpm_publisher_currency. It seems to be JPY (Now $0.477 is around 74JPN)
  • Maybe we should use this param, If we use currency other USD.

@saitoukun
Copy link
Contributor Author

saitoukun commented Jul 18, 2024

I am just a partner, and this is just a guess from the Bid response.
So I think It should be revert once. #11995
If we revert this, currency module works. There is no problem.

And then we should ask Appnexus engineers for implement newBid function with the exact specifications.

@jsnellbaker
Copy link
Collaborator

Hi all,

So I contacted an engineer who works on the appnexus endpoint and confirmed some details about the cpm and currency functionality (as it changed a little).

The cpm field in our endpoint's bid response will always be USD.
The cpm_publisher_currency field in the bid response will reflect and convert to the currency that was set at the publisher entity in the ad server system.

I confirmed this by setting up a new test publisher that was set to JPY, and in the test bid response I saw the following:

{
    "cpm": 1.5,
    "cpm_publisher_currency": 235.44858,
    "publisher_currency_code": "¥",
    "publisher_currency_codename": "JPY",
    "content_source": "rtb",
    "ad_type": "banner",
    ...
}

The bid is set to $1.5 USD normally and for the JPY test publisher the cpm_publisher_currency field had the JPY equivalent.

Currently in the adapter, the cpm_publisher_currency is not included in the prebid bid object. The prebid currency module (tmk) will convert the cpm value depending on the publisher setup. So unless I'm mistaken, I believe we could just leave things as is (post the revert) and let publisher make the changes with the module (since that will update the primary cpm value that prebid operates off of).

Please share any feedback/concerns about the above.

@saitoukun
Copy link
Contributor Author

Thank you!

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

Successfully merging a pull request may close this issue.

2 participants