Skip to content

Commit 665e9f5

Browse files
Change Targting Key Prefix Constant
1 parent fdb8998 commit 665e9f5

File tree

4 files changed

+10
-9
lines changed

4 files changed

+10
-9
lines changed

endpoints/openrtb2/ctv/util/util.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,9 @@ import (
1818
"github.com/prebid/prebid-server/v3/openrtb_ext"
1919
)
2020

21+
// default targeting key prefix
22+
const DefaultKeyPrefix = "hb"
23+
2124
var (
2225
//prebid_ctv_errors
2326
UnableToGenerateImpressionsError = &errortypes.AdpodPrefiltering{Message: `prebid_ctv unable to generate impressions for adpod`}
@@ -106,8 +109,7 @@ func TimeTrack(start time.Time, name string) {
106109
// if value not present or any error occured empty value will be returned
107110
// along with error.
108111
func GetTargeting(key openrtb_ext.TargetingKey, bidder openrtb_ext.BidderName, bid openrtb2.Bid) (string, error) {
109-
hbtargetingPrefix := "hb"
110-
bidderSpecificKey := key.BidderKey(hbtargetingPrefix, openrtb_ext.BidderName(bidder), 20)
112+
bidderSpecificKey := key.BidderKey(DefaultKeyPrefix, openrtb_ext.BidderName(bidder), 20)
111113
return jsonparser.GetString(bid.Ext, "prebid", "targeting", bidderSpecificKey)
112114
}
113115

modules/pubmatic/openwrap/adpod/auction/auction.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,7 @@ func FormAdpodBidsAndPerformExclusion(response *openrtb2.BidResponse, rctx model
6161
// if value not present or any error occured empty value will be returned
6262
// along with error.
6363
func GetTargeting(key openrtb_ext.TargetingKey, bidder openrtb_ext.BidderName, bid openrtb2.Bid) (string, error) {
64-
hbtargetingPrefix := "hb"
65-
bidderSpecificKey := key.BidderKey(hbtargetingPrefix, openrtb_ext.BidderName(bidder), 20)
64+
bidderSpecificKey := key.BidderKey(models.DefaultTargetingKeyPrefix, openrtb_ext.BidderName(bidder), 20)
6665
return jsonparser.GetString(bid.Ext, "prebid", "targeting", bidderSpecificKey)
6766
}
6867

@@ -71,7 +70,7 @@ func addTargetingKey(bid *openrtb2.Bid, key openrtb_ext.TargetingKey, value stri
7170
return errors.New("Invalid bid")
7271
}
7372

74-
key = "hb" + key
73+
key = models.DefaultTargetingKeyPrefix + key
7574
raw, err := jsonparser.Set(bid.Ext, []byte(strconv.Quote(value)), "prebid", "targeting", string(key))
7675
if err == nil {
7776
bid.Ext = raw

modules/pubmatic/openwrap/models/constants.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ const (
293293
HbBuyIdPrefix = "hb_buyid_"
294294
HbBuyIdPubmaticConstantKey = "hb_buyid_pubmatic"
295295
PwtBuyIdPubmaticConstantKey = "pwtbuyid_pubmatic"
296-
HBTargetingKeyPrefix = "hb"
296+
DefaultTargetingKeyPrefix = "hb"
297297

298298
SChainDBKey = "sChain"
299299
SChainObjectDBKey = "sChainObj"

modules/pubmatic/openwrap/targeting.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ import (
1515

1616
// whitelist of prebid targeting keys
1717
var prebidTargetingKeysWhitelist = map[string]struct{}{
18-
models.HBTargetingKeyPrefix + string(openrtb_ext.PbKey): {},
19-
models.HBTargetingKeyPrefix + models.HbBuyIdPubmaticConstantKey: {},
18+
models.DefaultTargetingKeyPrefix + string(openrtb_ext.PbKey): {},
19+
models.HbBuyIdPubmaticConstantKey: {},
2020
// OTT - 18 Deal priortization support
2121
// this key required to send deal prefix and priority
22-
models.HBTargetingKeyPrefix + string(openrtb_ext.CategoryDurationKey): {},
22+
models.DefaultTargetingKeyPrefix + string(openrtb_ext.CategoryDurationKey): {},
2323
}
2424

2525
// check if prebid targeting keys are whitelisted

0 commit comments

Comments
 (0)