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

OTT-1704: Bidder Response mapping framework for ortb bidder #806

Merged
merged 53 commits into from
Jun 17, 2024

Conversation

Pubmatic-Dhruv-Sonone
Copy link
Collaborator

@Pubmatic-Dhruv-Sonone Pubmatic-Dhruv-Sonone commented Jun 3, 2024

Description

  1. Ortb bidder response changes
  2. Bidder param resolver framework
  3. Code refactor

Checklist:

  • PR commit list is unique (rebase/pull with the origin branch to keep master clean).
  • JIRA number is added in the PR title and the commit message.
  • Updated the header-bidding repo with appropiate commit id.
  • Documented the new changes.

For Prebid upgrade, refer: https://inside.pubmatic.com:8443/confluence/display/Products/Prebid-server+upgrade

adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/param_mapper.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Outdated Show resolved Hide resolved
@Pubmatic-Dhruv-Sonone Pubmatic-Dhruv-Sonone marked this pull request as ready for review June 6, 2024 19:44
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Repository owner deleted a comment from github-actions bot Jun 16, 2024
Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, c2d28f5

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	parseResponse			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:38:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:93:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.2%

Copy link
Collaborator

@ashishshinde-pubm ashishshinde-pubm left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@ShriprasadM ShriprasadM left a comment

Choose a reason for hiding this comment

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

@Pubmatic-Dhruv-Sonone : There are minor comments.

adapters/ortbbidder/response_builder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/response_builder.go Outdated Show resolved Hide resolved
adapters/ortbbidder/ortbbidder.go Show resolved Hide resolved
adapters/ortbbidder/resolver/bidtype_resolver.go Outdated Show resolved Hide resolved
adapters/ortbbidder/resolver/param_resolver.go Outdated Show resolved Hide resolved
ShriprasadM
ShriprasadM previously approved these changes Jun 17, 2024
Copy link
Collaborator

@ShriprasadM ShriprasadM left a comment

Choose a reason for hiding this comment

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

@Pubmatic-Dhruv-Sonone : Please address minor comments. Otherwise LGTM

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, de38c94

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:90:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.1%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, cfd9047

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:90:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.1%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 58168e3

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:90:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.1%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 583e23f

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:90:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.1%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, f931f65

ortbbidder

Refer here for heat map coverage report

github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:17:			newBadInputError		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/errors.go:23:			newBadServerResponseError	100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:16:	parseRequest			75.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/multi_request_builder.go:43:	makeRequest			82.4%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:38:		InitBidderParamsConfig		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:44:		Builder				100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:63:		MakeRequests			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:82:		MakeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/ortbbidder.go:102:		makeBids			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:32:	newRequestBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:50:	getEndpoint			100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:62:	cloneRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/request_builder.go:72:	appendRequestData		80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:12:	setRequestParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:32:	addIndicesInPath		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/requestparam_mapper.go:48:	getImpExtBidderParams		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:21:	newResponseBuilder		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:30:	setPrebidBidderResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/response_builder.go:90:	buildAdapterResponse		100.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:19:	parseRequest			80.0%
github.com/PubMatic-OpenWrap/prebid-server/v2/adapters/ortbbidder/single_request_builder.go:46:	makeRequest			91.7%
total:												(statements)			93.1%

@Pubmatic-Dhruv-Sonone Pubmatic-Dhruv-Sonone merged commit 61416e2 into ci Jun 17, 2024
4 checks passed
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.

5 participants