Repository navigation
New Adapter: GoAdserver - #4648
Open
goadserver wants to merge 2 commits into
Open
goadserver wants to merge 2 commits into
goadserver wants to merge 2 commits into
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Logb1f3FNVSzPCTMtqNc8A
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Logb1f3FNVSzPCTMtqNc8A
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔧 Type of changes
✨ What's the context?
GoAdserver is a self-hosted, multi-tenant ad server. The Prebid.js adapter (
goadserver) is already in Prebid.js, with docs at https://docs.prebid.org/dev-docs/bidders/goadserver.html. This adds the Prebid Server Java adapter. The Go counterpart is prebid/prebid-server#5023.🧠 Rationale behind the change
https://pbs.goadserver.com/openrtb2/auction. GoAdserver's gateway routes each request to the publisher's deployment by the publisher token, so the adapter never builds a hostname from request data.token(required), plus optionalfloor(USD CPM, applied when the imp has no bidfloor) andsubid(string or integer, sent asimp.ext.goadserver.subid).site.publisher.id. Other publisher fields are kept.bid.mtype, falling back tobid.ext.prebid.type, which the endpoint sets on every bid. The endpoint's Prebid.js targeting ext is dropped andbid.ext.dsais kept forDsaEnforcer.site.page.🔎 New Bid Adapter Checklist
Bidder<BidRequest>and usesBidderUtil.defaultRequestsite.publisher.idandimp.ext, applies the floor param, and filters the response ext.GoadserverTest)🧪 Test plan
GoadserverBidderTest: 18 unit tests covering site-only, grouping by token, publisher override and creation, floor precedence, subid (string and number), missing token, invalid ext, the OpenRTB version header, mtype and ext-type resolution, DSA kept and targeting stripped, unsupported types, and the default currency.GoadserverTestintegration test against WireMock.BidderParamValidatorTestandBidderCatalogTestpass, and checkstyle reports 0 violations.🏎 Quality check
🤖 Generated with Claude Code
https://claude.ai/code/session_01Logb1f3FNVSzPCTMtqNc8A