Skip to content

Commit de3b797

Browse files
Bill Newmanvladis-teqblazeAiholkinMykhailo Yaremchuk
authored
Colossus Bid Adapter: fix validator (prebid#7926)
* 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 Co-authored-by: Vladislav Isaiko <[email protected]> Co-authored-by: Aiholkin <[email protected]> Co-authored-by: Mykhailo Yaremchuk <[email protected]>
1 parent c9dc39c commit de3b797

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

modules/colossussspBidAdapter.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,10 @@ export const spec = {
4747
* @return boolean True if this is a valid bid, and false otherwise.
4848
*/
4949
isBidRequestValid: (bid) => {
50-
return Boolean(bid.bidId && bid.params && !isNaN(bid.params.placement_id));
50+
const validPlacamentId = bid.params && !isNaN(bid.params.placement_id);
51+
const validGroupId = bid.params && !isNaN(bid.params.group_id);
52+
53+
return Boolean(bid.bidId && (validPlacamentId || validGroupId));
5154
},
5255

5356
/**

test/spec/modules/colossussspBidAdapter_spec.js

+1
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ describe('ColossussspAdapter', function () {
6161
});
6262
it('Should return false when placement_id is not a number', function () {
6363
bid.params.placement_id = 'aaa';
64+
delete bid.params.group_id;
6465
expect(spec.isBidRequestValid(bid)).to.be.false;
6566
});
6667
});

0 commit comments

Comments
 (0)