search

openrtb / OpenRTB

Documentation and issue tracking for the OpenRTB Project
BSD 2-Clause "Simplified" License
854 stars 143 forks source link

Conflict of "protocol" field in Video object between the definition and example #6

Closed yiyeguhu closed 10 years ago

yiyeguhu commented 10 years ago

On page 21 in "OPENRTB API Specification Version 2.2", it defines the type of "protocol" as integer, while on page 53, the Video bid request example gives "protocol" field as an array of integers. So, which type is the field supposed to be?

drjbutler commented 10 years ago

Both are defined. In 2.1 and prior, it was just the single integer “protocol”. In 2.2, it was agreed that an array of possible values was required. Changing the type would not have been backward compatible, however, so the “protocols” array was added and “protocol” was deprecated (i.e., still supported, but no longer recommended for use). When the spec turns 3.0 some fine day, “protocol” will likely be removed.

:JB

From: yiyeguhu [mailto:notifications@github.com] Sent: Thursday, July 10, 2014 3:12 PM To: openrtb/OpenRTB Subject: [OpenRTB] Conflict of "protocol" field in Video object between the definition and example (#6)

On page 21 in "OPENRTB API Specification Version 2.2", it defines the type of "protocol" as integer, while on page 53, the Video bid request example gives "protocol" field as an array of integers. So, which type is the field supposed to be?

— Reply to this email directly or view it on GitHubhttps://github.com/openrtb/OpenRTB/issues/6.

yiyeguhu commented 10 years ago

So on page 53, it should be "protocols": [2,3] instead of "protocol": [2,3]

drjbutler commented 10 years ago

Yes.

From: yiyeguhu [mailto:notifications@github.com] Sent: Thursday, July 10, 2014 4:44 PM To: openrtb/OpenRTB Cc: Jim Butler Subject: Re: [OpenRTB] Conflict of "protocol" field in Video object between the definition and example (#6)

So on page 53, it should be "protocols": [2,3] instead of "protocol": [2,3]

— Reply to this email directly or view it on GitHubhttps://github.com/openrtb/OpenRTB/issues/6#issuecomment-48661412.

yiyeguhu commented 10 years ago

Is this fixed in latest spec? In which spec version? Where is the latest version?

drjbutler commented 10 years ago

Yes, 2.3, it is currently in draft form being edited; not ready for distribution.

From: yiyeguhu [mailto:notifications@github.com] Sent: Thursday, July 10, 2014 5:11 PM To: openrtb/OpenRTB Cc: Jim Butler Subject: Re: [OpenRTB] Conflict of "protocol" field in Video object between the definition and example (#6)

Is this fixed in latest spec? In which spec version? Where is the latest version?

— Reply to this email directly or view it on GitHubhttps://github.com/openrtb/OpenRTB/issues/6#issuecomment-48664964.