Skip to content
This repository was archived by the owner on Nov 20, 2025. It is now read-only.

[pull] master from prebid:master#61

Merged
pull[bot] merged 4 commits intoadhese:masterfrom
prebid:master
Sep 22, 2025
Merged

[pull] master from prebid:master#61
pull[bot] merged 4 commits intoadhese:masterfrom
prebid:master

Conversation

@pull
Copy link

@pull pull bot commented Sep 22, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

andreafassina and others added 4 commits September 22, 2025 12:38
Co-authored-by: Andrea Fassina <fasenderos@gmail.com>
Co-authored-by: nicoladellamorte <nicola.dellamorte@nativery.com>
Co-authored-by: QuentinGallard <quentin.gallard@digitalnolimit.com>
@pull pull bot locked and limited conversation to collaborators Sep 22, 2025
@pull pull bot added the ⤵️ pull label Sep 22, 2025
@pull pull bot merged commit a42ab4d into adhese:master Sep 22, 2025
1 check passed
}

func getMediaTypeForBid(bid *bidExt) (openrtb_ext.BidType, error) {
switch bid.Nativery.BidType {

Choose a reason for hiding this comment

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

Consider this as a suggestion. The current implementation follows an anti-pattern, assumes that if there is a multi-format request, the media type defaults to openrtb_ext.BidTypeNative. Prebid server expects the media type to be explicitly set in the adapter response. Therefore, we strongly recommend implementing a pattern where the adapter server sets the MType field in the response to accurately determine the media type for the impression.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments