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

Update First Party Data workflow and introduce Bidder Config #573

Merged
merged 26 commits into from
Jun 3, 2020

Conversation

RossGoncharuk
Copy link
Contributor

@RossGoncharuk RossGoncharuk commented Dec 23, 2019

Updated First Party Data processing workflow and introduced Bidder Config support to match PBS behavior to what is required in prebid issue 879

AMP and Auction.

Fix bug when we enforced fpd for request without bidderconfig list.
(Now this behavior can be reproduced when this parameter set to {})

Implement PR

# Conflicts:
#	src/test/resources/org/prebid/server/it/cache/update/test-auction-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/adform/test-auction-adform-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/adkernel/test-auction-adkernel-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/adkerneladn/test-auction-adkerneladn-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/adpone/test-auction-adpone-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/adtelligent/test-auction-adtelligent-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/advangelists/test-auction-advangelists-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/beachfront/test-auction-beachfront-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/brightroll/test-auction-brightroll-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/conversant/alias/test-auction-conversant-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/conversant/test-auction-conversant-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/engagebdr/test-auction-engagebdr-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/facebook/test-auction-facebook-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/gamoshi/test-auction-gamoshi-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/grid/test-auction-grid-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/gumgum/test-auction-gumgum-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/improvedigital/test-auction-improvedigital-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/ix/test-auction-ix-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/lifestreet/test-auction-lifestreet-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/lockerdome/test-auction-lockerdome-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/mgid/test-auction-mgid-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/openx/test-auction-openx-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/pubmatic/test-auction-pubmatic-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/pulsepoint/test-auction-pulsepoint-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/rhythmone/test-auction-rhythmone-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/rtbhouse/test-auction-rtbhouse-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/somoaudience/test-auction-somoaudience-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/sonobi/test-auction-sonobi-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/sovrn/test-auction-sovrn-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/synacormedia/test-auction-synacormedia-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/tappx/test-auction-tappx-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/triplelift/test-auction-triplelift-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/ttx/test-auction-ttx-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/unruly/test-auction-unruly-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/verizonmedia/test-auction-verizonmedia-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/verizonmedia/test-verizonmedia-bid-request-1.json
#	src/test/resources/org/prebid/server/it/openrtb2/visx/test-auction-visx-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/vrtcal/test-auction-vrtcal-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/yieldmo/test-auction-yieldmo-response.json
@lgtm-com
Copy link

lgtm-com bot commented Dec 23, 2019

This pull request introduces 1 alert when merging a4395f8 into dde2146 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

# Conflicts:
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-auction-rubicon-appnexus-response.json
@lgtm-com
Copy link

lgtm-com bot commented Jan 16, 2020

This pull request introduces 1 alert when merging c8d62ef into 6acad88 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com
Copy link

lgtm-com bot commented Jan 16, 2020

This pull request introduces 1 alert when merging 92887aa into 6acad88 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

# Conflicts:
#	src/test/resources/org/prebid/server/it/openrtb2/facebook/test-facebook-bid-request-1.json
@lgtm-com
Copy link

lgtm-com bot commented Jan 16, 2020

This pull request introduces 1 alert when merging 7e59386 into f9b0c17 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@rpanchyk rpanchyk requested review from schernysh and DGarbar and removed request for rpanchyk February 5, 2020 10:54
DGarbar and others added 9 commits February 5, 2020 14:03
# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
#	src/main/java/org/prebid/server/proto/openrtb/ext/request/ExtRequestPrebid.java
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-appnexus-bid-request-1.json
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-auction-rubicon-appnexus-response.json
# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
#	src/test/resources/org/prebid/server/it/openrtb2/datablocks/test-auction-datablocks-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/emxdigital/test-auction-emxdigital-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/kubient/test-auction-kubient-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/kubient/test-kubient-bid-request-1.json
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-appnexus-bid-request-1.json
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-appnexus-bid-request-2.json
#	src/test/resources/org/prebid/server/it/openrtb2/rubicon_appnexus/test-rubicon-bid-request-2.json
#	src/test/resources/org/prebid/server/it/openrtb2/synacormedia/test-synacormedia-bid-request.json
#	src/test/resources/org/prebid/server/it/openrtb2/tripleliftnative/test-auction-triplelift-native-response.json
# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
DGarbar
DGarbar previously approved these changes Feb 21, 2020
@lgtm-com
Copy link

lgtm-com bot commented Feb 21, 2020

This pull request introduces 1 alert when merging 90ebb6b into 3a982b4 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@DGarbar DGarbar requested a review from rpanchyk February 21, 2020 14:26
@lgtm-com
Copy link

lgtm-com bot commented Mar 5, 2020

This pull request introduces 1 alert when merging 6399f6a into c2f6bb1 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com
Copy link

lgtm-com bot commented Mar 13, 2020

This pull request introduces 1 alert when merging d2de2e5 into 2b53488 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
#	src/main/java/org/prebid/server/proto/openrtb/ext/request/ExtRequestPrebid.java
#	src/test/java/org/prebid/server/auction/ExchangeServiceTest.java
#	src/test/resources/org/prebid/server/it/openrtb2/brightroll/test-brightroll-bid-request-1.json
#	src/test/resources/org/prebid/server/it/openrtb2/datablocks/test-auction-datablocks-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/kubient/test-auction-kubient-response.json
#	src/test/resources/org/prebid/server/it/openrtb2/smartrtb/test-smartrtb-bid-request.json
#	src/test/resources/org/prebid/server/it/openrtb2/synacormedia/test-synacormedia-bid-request.json
#	src/test/resources/org/prebid/server/it/openrtb2/tripleliftnative/test-auction-triplelift-native-response.json
@lgtm-com
Copy link

lgtm-com bot commented May 6, 2020

This pull request introduces 1 alert when merging 6fd26e1 into 0ac5a89 - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

rpanchyk added 2 commits May 22, 2020 15:37
# Conflicts:
#	src/main/java/org/prebid/server/auction/AmpRequestFactory.java
@lgtm-com
Copy link

lgtm-com bot commented Jun 2, 2020

This pull request introduces 1 alert when merging 5db231d into fa0429c - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 1 alert when merging 815e083 into fa0429c - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@rpanchyk rpanchyk merged commit c9cfd47 into master Jun 3, 2020
@rpanchyk rpanchyk deleted the fpd-update branch June 3, 2020 13:41
@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 1 alert when merging 9647883 into fa0429c - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

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