From 48710caf031b2c00e008137eb67005f780c62851 Mon Sep 17 00:00:00 2001 From: Hasan Kanjee <55110940+hasan-kanjee@users.noreply.github.com> Date: Tue, 22 Aug 2023 20:01:13 -0400 Subject: [PATCH] OFF-657 remove ip as a bidder request parameter (#6) --- adapters/flipp/flipp.go | 4 +--- adapters/flipp/flipptest/exemplary/simple-banner-native.json | 3 +-- openrtb_ext/imp_flipp.go | 1 - 3 files changed, 2 insertions(+), 6 deletions(-) diff --git a/adapters/flipp/flipp.go b/adapters/flipp/flipp.go index 9705fa574ba..14f31a91efb 100644 --- a/adapters/flipp/flipp.go +++ b/adapters/flipp/flipp.go @@ -114,9 +114,7 @@ func (a *adapter) processImp(request *openrtb2.BidRequest, imp openrtb2.Imp) (*a } var userIP string - if flippExtParams.IP != "" { - userIP = flippExtParams.IP - } else if request.Device != nil && request.Device.IP != "" { + if request.Device != nil && request.Device.IP != "" { userIP = request.Device.IP } else { return nil, fmt.Errorf("no IP set in flipp bidder params or request device") diff --git a/adapters/flipp/flipptest/exemplary/simple-banner-native.json b/adapters/flipp/flipptest/exemplary/simple-banner-native.json index c242c323003..d8a24131c77 100644 --- a/adapters/flipp/flipptest/exemplary/simple-banner-native.json +++ b/adapters/flipp/flipptest/exemplary/simple-banner-native.json @@ -30,7 +30,6 @@ "creativeType": "NativeX", "siteId": 1243066, "zoneIds": [285431], - "ip": "123.123.123.124", "options": { "startCompact": true, "contentCode": "publisher-test-2" @@ -46,7 +45,7 @@ "expectedRequest": { "uri": "http://example.com/pserver", "body": { - "ip":"123.123.123.124", + "ip":"123.123.123.123", "keywords":[ "" ], diff --git a/openrtb_ext/imp_flipp.go b/openrtb_ext/imp_flipp.go index 029e2882652..56874c024c6 100644 --- a/openrtb_ext/imp_flipp.go +++ b/openrtb_ext/imp_flipp.go @@ -6,7 +6,6 @@ type ImpExtFlipp struct { SiteID int64 `json:"siteId"` ZoneIds []int64 `json:"zoneIds,omitempty"` UserKey string `json:"userKey,omitempty"` - IP string `json:"ip,omitempty"` Options ImpExtFlippOptions `json:"options,omitempty"` }