-
Notifications
You must be signed in to change notification settings - Fork 769
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
OpenRTB 2.6 For Internal Model #2280
Conversation
@@ -16,7 +16,8 @@ require ( | |||
github.com/julienschmidt/httprouter v1.3.0 | |||
github.com/lib/pq v1.10.4 | |||
github.com/mitchellh/copystructure v1.2.0 | |||
github.com/mxmCherry/openrtb/v15 v15.0.1 | |||
github.com/mxmCherry/openrtb v13.0.0+incompatible | |||
github.com/mxmCherry/openrtb/v16 v16.0.0-alpha.2 |
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.
Don't worry about the alpha status. The library is being kept in alpha until it's implemented Prebid Server to prove it works well.
} | ||
|
||
// ExtRequestPrebidSChainSChain defines the contract for bidrequest.ext.prebid.schains[i].schain | ||
type ExtRequestPrebidSChainSChain struct { |
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.
Goodbye SChainSchain
!
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.
LGTM, just need to resolve the conflict.
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.
LGTM
* 'master' of https://github.com/wwwyyy/prebid-server: AMP targeting query string (prebid#2293) Fix typo (prebid#2275) Sovrn: Update endpoint (prebid#2305) Added native support (prebid#2299) Bidders Should Be Listed Alphabetically (prebid#2295) Kargo Adapter: Add userSync to bidder-info (prebid#2294) OpenRTB 2.6 For Internal Model (prebid#2280)
* OpenRTB 2.6 Initial Commit * Fix Compilation Issues - Move To v16alpha2 * Replace ExtUserEid With MXMCherry Type * Use SChain Model From MXMCherry * Review Pass
* OpenRTB 2.6 Initial Commit * Fix Compilation Issues - Move To v16alpha2 * Replace ExtUserEid With MXMCherry Type * Use SChain Model From MXMCherry * Review Pass
* OpenRTB 2.6 Initial Commit * Fix Compilation Issues - Move To v16alpha2 * Replace ExtUserEid With MXMCherry Type * Use SChain Model From MXMCherry * Review Pass
* OpenRTB 2.6 Initial Commit * Fix Compilation Issues - Move To v16alpha2 * Replace ExtUserEid With MXMCherry Type * Use SChain Model From MXMCherry * Review Pass
* OpenRTB 2.6 Initial Commit * Fix Compilation Issues - Move To v16alpha2 * Replace ExtUserEid With MXMCherry Type * Use SChain Model From MXMCherry * Review Pass
Upgrades the internal OpenRTB model to support 2.6. This will allow the core code and adapters to access OpenRTB 2.6 fields, but features (like schains, eids, gdpr, ccpa) will still be supported at 2.5 locations only until a future PR adds up convert and down convert support.
This PR changes adapter code. The changes are just simple type replacements except for the Rubicon adapter which relied upon EID.ID, an non-standard field. Removed with their approval and flagging them here (@bretg) to review the changes to their tests.