From 7a70dbef0daaf03b410b706ab71825873c49d8a9 Mon Sep 17 00:00:00 2001 From: pankivanastasiya Date: Thu, 5 Feb 2026 11:56:14 +0200 Subject: [PATCH 1/2] bidResponseFilter: read mediaType from bid, not bid.meta --- modules/bidResponseFilter/index.js | 10 ++++++---- test/spec/modules/bidResponseFilter_spec.js | 6 +++--- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/modules/bidResponseFilter/index.js b/modules/bidResponseFilter/index.js index 64026958bc6..6cdd59c9fda 100644 --- a/modules/bidResponseFilter/index.js +++ b/modules/bidResponseFilter/index.js @@ -39,11 +39,13 @@ export function addBidResponseHook(next, adUnitCode, bid, reject, index = auctio const mediaTypesConfig = {enforce: true, blockUnknown: true, ...(moduleConfig?.mediaTypes || {})}; const { - primaryCatId, secondaryCatIds = [], - advertiserDomains = [], - attr: metaAttr, mediaType: metaMediaType, - } = bid.meta || {}; + meta: { + primaryCatId, secondaryCatIds = [], + advertiserDomains = [], + attr: metaAttr + } = {} + } = bid; // checking if bid fulfills ortb2 fields rules if ((catConfig.enforce && bcat.some(category => [primaryCatId, ...secondaryCatIds].includes(category))) || diff --git a/test/spec/modules/bidResponseFilter_spec.js b/test/spec/modules/bidResponseFilter_spec.js index c37003bde50..7072d7927b1 100644 --- a/test/spec/modules/bidResponseFilter_spec.js +++ b/test/spec/modules/bidResponseFilter_spec.js @@ -65,11 +65,11 @@ describe('bidResponseFilter', () => { }) const bid = { + mediaType: 'banner', meta: { advertiserDomains: ['domain1.com', 'domain2.com'], primaryCatId: 'EXAMPLE-CAT-ID', attr: 'attr', - mediaType: 'banner' } }; @@ -144,11 +144,11 @@ describe('bidResponseFilter', () => { it('should omit the validation if the flag is set to false', () => { const call = sinon.stub(); const bid = { + mediaType: 'banner', meta: { advertiserDomains: ['validdomain1.com', 'validdomain2.com'], primaryCatId: 'BANNED_CAT1', attr: 'valid_attr', - mediaType: 'banner', } }; @@ -173,11 +173,11 @@ describe('bidResponseFilter', () => { it('should allow bid for unknown flag set to false', () => { const call = sinon.stub(); const bid = { + mediaType: 'banner', meta: { advertiserDomains: ['validdomain1.com', 'validdomain2.com'], primaryCatId: undefined, attr: 'valid_attr', - mediaType: 'banner' } }; From 8b4161786872768021569c441be535c4deb28eb2 Mon Sep 17 00:00:00 2001 From: pankivanastasiya Date: Thu, 5 Feb 2026 16:21:12 +0200 Subject: [PATCH 2/2] updated remaining tests --- test/spec/modules/bidResponseFilter_spec.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/spec/modules/bidResponseFilter_spec.js b/test/spec/modules/bidResponseFilter_spec.js index 7072d7927b1..1977bc8cc47 100644 --- a/test/spec/modules/bidResponseFilter_spec.js +++ b/test/spec/modules/bidResponseFilter_spec.js @@ -118,12 +118,12 @@ describe('bidResponseFilter', () => { const reject = sinon.stub(); const call = sinon.stub(); const bid = { + mediaType: 'video', meta: { advertiserDomains: ['validdomain1.com', 'validdomain2.com'], primaryCatId: 'VALID_CAT', attr: 'BANNED_ATTR' }, - mediaType: 'video' }; mockAuctionIndex.getOrtb2 = () => ({ badv: ['domain2.com'], bcat: ['BANNED_CAT1', 'BANNED_CAT2'] @@ -203,11 +203,11 @@ describe('bidResponseFilter', () => { const reject = sinon.stub(); const call = sinon.stub(); const bid = { + mediaType: 'audio', meta: { advertiserDomains: ['validdomain1.com', 'validdomain2.com'], primaryCatId: 'VALID_CAT', attr: 6, - mediaType: 'audio' }, };