Skip to content

Commit 852d0ee

Browse files
Bill Newmanvladis-teqblazeAiholkinMykhailo Yaremchuk
authored
Colossusssp Bid Adapter: fix traffic param (prebid#8724)
* add video&native traffic colossus ssp * Native obj validation * Native obj validation #2 * Added size field in requests * fixed test * fix merge conflicts * move to 3.0 * move to 3.0 * fix IE11 new URL issue * fix IE11 new URL issue * fix IE11 new URL issue * https for 3.0 * add https test * add ccp and schain features * fix test * sync with upstream, fix conflicts * Update colossussspBidAdapter.js remove commented code * Update colossussspBidAdapter.js lint fix * identity extensions * identity extensions * fix * fix * fix * fix * fix * add tests for user ids * fix * fix * fix * fix * fix * fix * fix * add gdpr support * add gdpr support * id5id support * Update colossussspBidAdapter.js add bidfloor parameter * Update colossussspBidAdapter.js check bidfloor * Update colossussspBidAdapter.js * Update colossussspBidAdapter.js * Update colossussspBidAdapter.js * Update colossussspBidAdapter_spec.js * use floor module * Revert "use floor module" This reverts commit f0c5c24. * use floor module * update to 5v * fix * add uid2 and bidFloor support * fix * add pbadslot support * fix conflicts * add onBidWon * refactor * add test for onBidWon() * fix * add group_id * Trigger circleci * fix * update user sync * fix window.location * fix test * updates * fix conflict * fix * updates * remove traffic param Co-authored-by: Vladislav Isaiko <[email protected]> Co-authored-by: Aiholkin <[email protected]> Co-authored-by: Mykhailo Yaremchuk <[email protected]>
1 parent 18306dc commit 852d0ee

File tree

3 files changed

+57
-29
lines changed

3 files changed

+57
-29
lines changed

modules/colossussspBidAdapter.js

+28-22
Original file line numberDiff line numberDiff line change
@@ -108,12 +108,11 @@ export const spec = {
108108

109109
for (let i = 0; i < validBidRequests.length; i++) {
110110
let bid = validBidRequests[i];
111-
let traff = bid.params.traffic || BANNER;
111+
const { mediaTypes } = bid;
112112
let placement = {
113113
placementId: bid.params.placement_id,
114114
groupId: bid.params.group_id,
115115
bidId: bid.bidId,
116-
traffic: traff,
117116
eids: [],
118117
floor: {}
119118
};
@@ -134,32 +133,39 @@ export const spec = {
134133
rtiPartner: 'TDID'
135134
});
136135
}
137-
if (traff === BANNER) {
138-
placement.sizes = bid.mediaTypes[BANNER].sizes
139-
} else if (traff === VIDEO) {
140-
placement.sizes = bid.mediaTypes[VIDEO].playerSize;
141-
placement.playerSize = bid.mediaTypes[VIDEO].playerSize;
142-
placement.minduration = bid.mediaTypes[VIDEO].minduration;
143-
placement.maxduration = bid.mediaTypes[VIDEO].maxduration;
144-
placement.mimes = bid.mediaTypes[VIDEO].mimes;
145-
placement.protocols = bid.mediaTypes[VIDEO].protocols;
146-
placement.startdelay = bid.mediaTypes[VIDEO].startdelay;
147-
placement.placement = bid.mediaTypes[VIDEO].placement;
148-
placement.skip = bid.mediaTypes[VIDEO].skip;
149-
placement.skipafter = bid.mediaTypes[VIDEO].skipafter;
150-
placement.minbitrate = bid.mediaTypes[VIDEO].minbitrate;
151-
placement.maxbitrate = bid.mediaTypes[VIDEO].maxbitrate;
152-
placement.delivery = bid.mediaTypes[VIDEO].delivery;
153-
placement.playbackmethod = bid.mediaTypes[VIDEO].playbackmethod;
154-
placement.api = bid.mediaTypes[VIDEO].api;
155-
placement.linearity = bid.mediaTypes[VIDEO].linearity;
136+
137+
if (mediaTypes && mediaTypes[BANNER]) {
138+
placement.traffic = BANNER;
139+
placement.sizes = mediaTypes[BANNER].sizes;
140+
} else if (mediaTypes && mediaTypes[VIDEO]) {
141+
placement.traffic = VIDEO;
142+
placement.sizes = mediaTypes[VIDEO].playerSize;
143+
placement.playerSize = mediaTypes[VIDEO].playerSize;
144+
placement.minduration = mediaTypes[VIDEO].minduration;
145+
placement.maxduration = mediaTypes[VIDEO].maxduration;
146+
placement.mimes = mediaTypes[VIDEO].mimes;
147+
placement.protocols = mediaTypes[VIDEO].protocols;
148+
placement.startdelay = mediaTypes[VIDEO].startdelay;
149+
placement.placement = mediaTypes[VIDEO].placement;
150+
placement.skip = mediaTypes[VIDEO].skip;
151+
placement.skipafter = mediaTypes[VIDEO].skipafter;
152+
placement.minbitrate = mediaTypes[VIDEO].minbitrate;
153+
placement.maxbitrate = mediaTypes[VIDEO].maxbitrate;
154+
placement.delivery = mediaTypes[VIDEO].delivery;
155+
placement.playbackmethod = mediaTypes[VIDEO].playbackmethod;
156+
placement.api = mediaTypes[VIDEO].api;
157+
placement.linearity = mediaTypes[VIDEO].linearity;
158+
} else if (mediaTypes && mediaTypes[NATIVE]) {
159+
placement.traffic = NATIVE;
160+
placement.native = mediaTypes[NATIVE];
156161
}
162+
157163
if (typeof bid.getFloor === 'function') {
158164
let tmpFloor = {};
159165
for (let size of placement.sizes) {
160166
tmpFloor = bid.getFloor({
161167
currency: 'USD',
162-
mediaType: traff,
168+
mediaType: placement.traffic,
163169
size: size
164170
});
165171
if (tmpFloor) {

modules/colossussspBidAdapter.md

+29-6
Original file line numberDiff line numberDiff line change
@@ -22,22 +22,45 @@ Module that connects to Colossus SSP demand sources
2222
bids: [{
2323
bidder: 'colossusssp',
2424
params: {
25-
placement_id: 0,
26-
traffic: 'banner'
25+
placement_id: 0
2726
}
2827
}]
2928
}, {
3029
code: 'placementid_1',
3130
mediaTypes: {
32-
banner: {
33-
sizes: [[300, 250], [300,600]]
31+
video: {
32+
playerSize: [ [640, 480] ],
33+
context: 'instream',
34+
minduration: 5,
35+
maxduration: 60,
3436
}
3537
},
3638
bids: [{
3739
bidder: 'colossusssp',
3840
params: {
39-
group_id: 0,
40-
traffic: 'banner'
41+
group_id: 0
42+
}
43+
}]
44+
}, {
45+
code: 'placementid_2',
46+
mediaTypes: {
47+
native: {
48+
title: {
49+
required: true
50+
},
51+
body: {
52+
required: true
53+
},
54+
icon: {
55+
required: true,
56+
size: [64, 64]
57+
}
58+
}
59+
},
60+
bids: [{
61+
bidder: 'colossusssp',
62+
params: {
63+
placement_id: 0,
4164
}
4265
}]
4366
}];

test/spec/modules/colossussspBidAdapter_spec.js

-1
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,6 @@ describe('ColossussspAdapter', function () {
118118
...bid,
119119
params: {
120120
placement_id: 0,
121-
traffic: 'video',
122121
},
123122
mediaTypes: {
124123
video: {

0 commit comments

Comments
 (0)