Align Missena adapter request payload with PBS Go - #4509
Conversation
bb332db to
a5365df
Compare
The adapter built bids without w/h, so every bid tripped creative size validation with size 'nullxnull' and produced no hb_size targeting. The Missena ad server already returns width and height in the response body; parse them and map them onto the bid.
Resolves conflict in test-missena-bid-request.json: upstream reformatted the fixture and added ext.prebid.server.http_method; reapplied the apiKey param and the test -> sample rename on top.
|
@CTMBNara friendly ping for a re-review when you get a chance 🙏 The requested passthrough test was added back in June: Some context on why the size passthrough matters to us in production. With On Happy to make any further changes needed to get this over the line. |
Send the version as `prebid-server-java@<version>` to follow the `name@version` convention the bid server already receives from Prebid.js (`prebid.js@<version>`) and Prebid Server Go (`prebid-server@<version>`), while keeping Go and Java traffic distinguishable. Falls back to `prebid-server-java@unknown` when the version record is unavailable. Drop the JSON body assertions in makeHttpRequestsShouldMakeRequestForFirstValidImp; the preceding containsExactly(expectedPayload) already compares the whole payload, so they were redundant. Add givenBidRequest(UnaryOperator<BidRequest.BidRequestBuilder>, Imp...) and use it in place of givenBidRequest(...).toBuilder(). The existing imp-customizer overload is replaced by givenBidRequest(Imp...) as in VisibleMeasuresBidderTest, since two overloads both taking a lambda first are ambiguous at the call site.
Summary
testwithsampleapiKeyinparams, plus top-leveldebuganduserEidsTesting
git diff --check -- src/main/java/org/prebid/server/bidder/missena src/main/java/org/prebid/server/proto/openrtb/ext/request/missena src/main/resources/static/bidder-params/missena.json src/test/java/org/prebid/server/bidder/missena src/test/resources/org/prebid/server/it/openrtb2/missenajq empty src/main/resources/static/bidder-params/missena.json src/test/resources/org/prebid/server/it/openrtb2/missena/test-auction-missena-request.json src/test/resources/org/prebid/server/it/openrtb2/missena/test-missena-bid-request.json