-
Notifications
You must be signed in to change notification settings - Fork 720
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
Medianet: Add fledge support #3641
Conversation
adapters/medianet/medianet.go
Outdated
return bidResponse, errs | ||
} | ||
|
||
// Builder builds a new instance of the Medianet adapter for the given bidder with the given config. | ||
func Builder(bidderName openrtb_ext.BidderName, config config.Adapter, server config.Server) (adapters.Bidder, error) { | ||
url := buildEndpoint(config.Endpoint, config.ExtraAdapterInfo) | ||
return &adapter{ | ||
endpoint: url, | ||
return &MedianetAdapter{ |
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.
You can call this simply "adapter", the MedianetAdapter
identification is already supplied by the package name. As you have it, referencing your adapter from outside the package would be MedianetAdapter.MedianetAdapter
which looks a little redundant. See example below:
package foo
type adapter struct {
endpoint string
}
func Builder(bidderName openrtb_ext.BidderName, config config.Adapter, server config.Server) (adapters.Bidder, error) {
return &adapter{endpoint: "https://www.foo.com"}, nil
}
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.
Done.
Code coverage summaryNote:
medianetRefer here for heat map coverage report
|
Code coverage summaryNote:
medianetRefer here for heat map coverage report
|
adapters/medianet/medianet.go
Outdated
if err := json.Unmarshal(bidResp.Ext, &bidRespExt); err != nil { | ||
return nil | ||
} |
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.
any particular reason to not return error from line 140
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.
Done, took error into consideration for assigning fledge config.
Code coverage summaryNote:
medianetRefer here for heat map coverage report
|
PR tests are failing.
|
Add Fledge support for Medianet adapter. The response from bidder for fledge auction config support the ortb format mentioned in the below link.
https://github.com/InteractiveAdvertisingBureau/openrtb/blob/main/extensions/community_extensions/Protected%20Audience%20Support.md