billing_id in openrtb json is String and not Long about openrtb-doubleclick HOT 6 CLOSED

Thanks for this report -- this is a bug indeed, what happens is that DoubleClick's billing_id is a long so this code needs to make a conversion but it's doing it wrong, it should be using getValueAsLong() instead of getLongValue(). Will fix this immediately.

from openrtb-doubleclick.

Thank you. There are also other fields in the Imp Ext that have the same issue, publisher list id, etc.

from openrtb-doubleclick.

Argh, I see now why this bug wasn't caught by the tests. The billing_id is a long and real-world values are often above JSON/JavasScript's limit of maximum representable integer (2^53 - 1), so JSON serializers often have an extension to quote large integer numbers. We use that in AdX, but we use it only for values above this limit, 12345678901 is not big enough, and we shouldn't have billing_ids that exceed 2^53 so this wouldn't even be a bug technically. But I suppose you maye have some intermediary processing between AdX and your bidder, and some JSON serializers have different behaviors, for one thing even our own public Protobuf 3 will always quote longs. So I'm still going to improve the library to allow quoted longs there, but let me know how are you getting this bid request message, if it's coming straight from AdX I may need to chase that too.

from openrtb-doubleclick.

yes, making the same change for publisher_settings_list_id.

from openrtb-doubleclick.

This issue still persists. Steps to reproduce:

• Receive bid request via PROTOCOL_OPENRTB_PROTOBUF_2_3 protocol.
• Read received protobuff request to BidRequest object via parseFrom:
  ExtensionRegistry dcExtension = ExtensionRegistry.newInstance(); AdxExt.registerAllExtensions(dcExtension); OpenRtb.BidRequest openRtbRequest = OpenRtb.BidRequest.parseFrom(data, dcExtension);
• Print received JSON:
  OpenRtbJsonFactory dcJsonFactory = AdxExtUtils.registerAdxExt(OpenRtbJsonFactory.create()); String jsonOutput = dcJsonFactory.newWriter().writeBidRequest(openRtbRequest);

from openrtb-doubleclick.

Sorry for the confusion, I had closed the bug but not released the fix yet. I have just pushed the fix to the master repository here, please test again after pulling the latest code from head.

from openrtb-doubleclick.