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

OpenRTB 2.6 For Internal Model #2280

Merged
merged 7 commits into from
Jul 11, 2022
Merged

OpenRTB 2.6 For Internal Model #2280

merged 7 commits into from
Jul 11, 2022

Conversation

SyntaxNode
Copy link
Contributor

@SyntaxNode SyntaxNode commented Jun 21, 2022

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.

@SyntaxNode SyntaxNode marked this pull request as ready for review July 5, 2022 21:34
@@ -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
Copy link
Contributor Author

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 {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goodbye SChainSchain!

Copy link
Collaborator

@bsardo bsardo left a 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.

Copy link
Collaborator

@hhhjort hhhjort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hhhjort hhhjort merged commit 4dfe9e3 into prebid:master Jul 11, 2022
wwwyyy pushed a commit to wwwyyy/prebid-server that referenced this pull request Jul 19, 2022
* '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)
@SyntaxNode SyntaxNode deleted the openrtb26 branch July 21, 2022 16:36
pm-aadit-patil pushed a commit to PubMatic-OpenWrap/prebid-server that referenced this pull request Sep 22, 2022
* OpenRTB 2.6 Initial Commit

* Fix Compilation Issues - Move To v16alpha2

* Replace ExtUserEid With MXMCherry Type

* Use SChain Model From MXMCherry

* Review Pass
pm-aadit-patil pushed a commit to PubMatic-OpenWrap/prebid-server that referenced this pull request Sep 22, 2022
* OpenRTB 2.6 Initial Commit

* Fix Compilation Issues - Move To v16alpha2

* Replace ExtUserEid With MXMCherry Type

* Use SChain Model From MXMCherry

* Review Pass
pm-aadit-patil pushed a commit to PubMatic-OpenWrap/prebid-server that referenced this pull request Sep 22, 2022
* OpenRTB 2.6 Initial Commit

* Fix Compilation Issues - Move To v16alpha2

* Replace ExtUserEid With MXMCherry Type

* Use SChain Model From MXMCherry

* Review Pass
jorgeluisrocha pushed a commit to jwplayer/prebid-server that referenced this pull request Sep 28, 2022
* OpenRTB 2.6 Initial Commit

* Fix Compilation Issues - Move To v16alpha2

* Replace ExtUserEid With MXMCherry Type

* Use SChain Model From MXMCherry

* Review Pass
shunj-nb pushed a commit to ParticleMedia/prebid-server that referenced this pull request Nov 8, 2022
* OpenRTB 2.6 Initial Commit

* Fix Compilation Issues - Move To v16alpha2

* Replace ExtUserEid With MXMCherry Type

* Use SChain Model From MXMCherry

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

Successfully merging this pull request may close these issues.

3 participants