From 461957284018af906d32afb3360f205cb6d04827 Mon Sep 17 00:00:00 2001 From: kapil-tuptewar Date: Thu, 24 Aug 2023 16:34:45 +0530 Subject: [PATCH 1/6] Logging timetorespond field in l1 --- modules/pubmaticAnalyticsAdapter.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/modules/pubmaticAnalyticsAdapter.js b/modules/pubmaticAnalyticsAdapter.js index f0f4bd6aa71..fbfe08413c8 100755 --- a/modules/pubmaticAnalyticsAdapter.js +++ b/modules/pubmaticAnalyticsAdapter.js @@ -312,7 +312,8 @@ function gatherPartnerBidsForAdUnitForLogger(adUnit, adUnitId, highestBid) { 'en': bid.bidResponse ? bid.bidResponse.bidPriceUSD : 0, 'di': bid.bidResponse ? (bid.bidResponse.dealId || EMPTY_STRING) : EMPTY_STRING, 'dc': bid.bidResponse ? (bid.bidResponse.dealChannel || EMPTY_STRING) : EMPTY_STRING, - 'l1': bid.bidResponse ? bid.clientLatencyTimeMs : 0, + 'l1': bid.bidResponse ? bid.partnerTimeToRespond : 0, + 'ol1': bid.bidResponse ? bid.clientLatencyTimeMs : 0, 'l2': 0, 'adv': bid.bidResponse ? getAdDomain(bid.bidResponse) || undefined : undefined, 'ss': isS2SBidder(bid.bidder), @@ -578,6 +579,7 @@ function bidResponseHandler(args) { bid.adId = args.adId; bid.source = formatSource(bid.source || args.source); setBidStatus(bid, args); + bid.partnerTimeToRespond = (args?.timeToRespond ? args?.timeToRespond : (window.PWT?.versionDetails ? (Date.now() - cache.auctions[args.auctionId].timestamp > window.PWT?.versionDetails.timeout ? (window.PWT?.versionDetails.timeout + 100) : Date.now() - cache.auctions[args.auctionId].timestamp) : Date.now() - cache.auctions[args.auctionId].timestamp)); bid.clientLatencyTimeMs = Date.now() - cache.auctions[args.auctionId].timestamp; bid.bidResponse = parseBidResponse(args); } From 10ed89e126271a70a5861cdc15555dcd62519628 Mon Sep 17 00:00:00 2001 From: kapil-tuptewar Date: Thu, 24 Aug 2023 16:44:03 +0530 Subject: [PATCH 2/6] Logging timetorespond field in l1 --- modules/pubmaticAnalyticsAdapter.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/modules/pubmaticAnalyticsAdapter.js b/modules/pubmaticAnalyticsAdapter.js index fbfe08413c8..5328a157670 100755 --- a/modules/pubmaticAnalyticsAdapter.js +++ b/modules/pubmaticAnalyticsAdapter.js @@ -579,7 +579,8 @@ function bidResponseHandler(args) { bid.adId = args.adId; bid.source = formatSource(bid.source || args.source); setBidStatus(bid, args); - bid.partnerTimeToRespond = (args?.timeToRespond ? args?.timeToRespond : (window.PWT?.versionDetails ? (Date.now() - cache.auctions[args.auctionId].timestamp > window.PWT?.versionDetails.timeout ? (window.PWT?.versionDetails.timeout + 100) : Date.now() - cache.auctions[args.auctionId].timestamp) : Date.now() - cache.auctions[args.auctionId].timestamp)); + const latency = args?.timeToRespond || Date.now() - cache.auctions[args.auctionId].timestamp; + bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > window.PWT.versionDetails.timeout ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; bid.clientLatencyTimeMs = Date.now() - cache.auctions[args.auctionId].timestamp; bid.bidResponse = parseBidResponse(args); } From c105eb38fb6e1ba711b3e7afd2e121fe48791f9f Mon Sep 17 00:00:00 2001 From: kapil-tuptewar Date: Thu, 24 Aug 2023 18:09:02 +0530 Subject: [PATCH 3/6] Logging timetorespond field in l1 --- modules/pubmaticAnalyticsAdapter.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/pubmaticAnalyticsAdapter.js b/modules/pubmaticAnalyticsAdapter.js index 5328a157670..5d5d19cbcd8 100755 --- a/modules/pubmaticAnalyticsAdapter.js +++ b/modules/pubmaticAnalyticsAdapter.js @@ -580,7 +580,7 @@ function bidResponseHandler(args) { bid.source = formatSource(bid.source || args.source); setBidStatus(bid, args); const latency = args?.timeToRespond || Date.now() - cache.auctions[args.auctionId].timestamp; - bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > window.PWT.versionDetails.timeout ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; + bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > (window.PWT.versionDetails.timeout + 100) ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; bid.clientLatencyTimeMs = Date.now() - cache.auctions[args.auctionId].timestamp; bid.bidResponse = parseBidResponse(args); } From 6c0a4f7deeaa6a71f805a193a02e002a8a0044eb Mon Sep 17 00:00:00 2001 From: kapil-tuptewar Date: Fri, 25 Aug 2023 16:55:07 +0530 Subject: [PATCH 4/6] Updated test cases for logger field l1 --- modules/pubmaticAnalyticsAdapter.js | 4 +-- .../modules/pubmaticAnalyticsAdapter_spec.js | 36 ++++++++++++------- 2 files changed, 26 insertions(+), 14 deletions(-) diff --git a/modules/pubmaticAnalyticsAdapter.js b/modules/pubmaticAnalyticsAdapter.js index 5d5d19cbcd8..6af0824f28e 100755 --- a/modules/pubmaticAnalyticsAdapter.js +++ b/modules/pubmaticAnalyticsAdapter.js @@ -313,7 +313,7 @@ function gatherPartnerBidsForAdUnitForLogger(adUnit, adUnitId, highestBid) { 'di': bid.bidResponse ? (bid.bidResponse.dealId || EMPTY_STRING) : EMPTY_STRING, 'dc': bid.bidResponse ? (bid.bidResponse.dealChannel || EMPTY_STRING) : EMPTY_STRING, 'l1': bid.bidResponse ? bid.partnerTimeToRespond : 0, - 'ol1': bid.bidResponse ? bid.clientLatencyTimeMs : 0, + 'ol1': bid.bidResponse ? bid.clientLatencyTimeMs : 0, 'l2': 0, 'adv': bid.bidResponse ? getAdDomain(bid.bidResponse) || undefined : undefined, 'ss': isS2SBidder(bid.bidder), @@ -580,7 +580,7 @@ function bidResponseHandler(args) { bid.source = formatSource(bid.source || args.source); setBidStatus(bid, args); const latency = args?.timeToRespond || Date.now() - cache.auctions[args.auctionId].timestamp; - bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > (window.PWT.versionDetails.timeout + 100) ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; + bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > (window.PWT.versionDetails.timeout + 100) ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; bid.clientLatencyTimeMs = Date.now() - cache.auctions[args.auctionId].timestamp; bid.bidResponse = parseBidResponse(args); } diff --git a/test/spec/modules/pubmaticAnalyticsAdapter_spec.js b/test/spec/modules/pubmaticAnalyticsAdapter_spec.js index 65ca250ade3..f9d87f03f70 100755 --- a/test/spec/modules/pubmaticAnalyticsAdapter_spec.js +++ b/test/spec/modules/pubmaticAnalyticsAdapter_spec.js @@ -431,7 +431,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[0].ps[0].en).to.equal(1.23); expect(data.s[0].ps[0].di).to.equal(''); expect(data.s[0].ps[0].dc).to.equal(''); - expect(data.s[0].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[0].ps[0].l2).to.equal(0); expect(data.s[0].ps[0].ss).to.equal(1); expect(data.s[0].ps[0].t).to.equal(0); @@ -465,7 +466,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -864,7 +866,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(0); + expect(data.s[0].ps[0].ol1).to.equal(0); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(1); @@ -927,7 +930,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -979,7 +983,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -1043,7 +1048,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -1099,7 +1105,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -1154,7 +1161,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -1236,7 +1244,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[0].ps[0].en).to.equal(1.23); expect(data.s[0].ps[0].di).to.equal(''); expect(data.s[0].ps[0].dc).to.equal(''); - expect(data.s[0].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[0].ps[0].l2).to.equal(0); expect(data.s[0].ps[0].ss).to.equal(0); expect(data.s[0].ps[0].t).to.equal(0); @@ -1271,7 +1280,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); @@ -1367,7 +1377,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[0].ps[0].en).to.equal(1.23); expect(data.s[0].ps[0].di).to.equal(''); expect(data.s[0].ps[0].dc).to.equal(''); - expect(data.s[0].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[0].ps[0].l2).to.equal(0); expect(data.s[0].ps[0].ss).to.equal(0); expect(data.s[0].ps[0].t).to.equal(0); @@ -1396,7 +1407,8 @@ describe('pubmatic analytics adapter', function () { expect(data.s[1].ps[0].dc).to.equal('PMP'); expect(data.s[1].ps[0].mi).to.equal('matched-impression'); expect(data.s[1].ps[0].adv).to.equal('example.com'); - expect(data.s[1].ps[0].l1).to.equal(3214); + expect(data.s[0].ps[0].l1).to.equal(944); + expect(data.s[0].ps[0].ol1).to.equal(3214); expect(data.s[1].ps[0].l2).to.equal(0); expect(data.s[1].ps[0].ss).to.equal(1); expect(data.s[1].ps[0].t).to.equal(0); From 32bce7567c8fba9e4353ee367c882e104106389d Mon Sep 17 00:00:00 2001 From: pm-nikhil-lakare <138765774+pm-nikhil-lakare@users.noreply.github.com> Date: Tue, 29 Aug 2023 14:23:23 +0530 Subject: [PATCH 5/6] Added missing modules in module_meta.json --- modules/module_meta.json | 463 ++++++++++++++++++++------------------- 1 file changed, 241 insertions(+), 222 deletions(-) diff --git a/modules/module_meta.json b/modules/module_meta.json index 1b0b3dcab7d..acb560c8770 100644 --- a/modules/module_meta.json +++ b/modules/module_meta.json @@ -9,6 +9,11 @@ "name": "dfpAdServerVideo", "path": "/modules/dfpAdServerVideo.js", "module": "dfpAdServerVideo" + }, + { + "name": "adpod", + "path": "/modules/adpod.js", + "module": "adpod" } ], "RTD Module": [ @@ -22,56 +27,56 @@ "path": "/modules/browsiRtdProvider.js", "module": "browsiRtdProvider" }, - { - "name": "1plusXRtdProvider", - "path": "/modules/1plusXRtdProvider.js", - "module": "1plusXRtdProvider" - }, - { - "name": "adnuntiusRtdProvider", - "path": "/modules/adnuntiusRtdProvider.js", - "module": "adnuntiusRtdProvider" - }, - { - "name": "airgridRtdProvider", - "path": "/modules/airgridRtdProvider.js", - "module": "airgridRtdProvider" - }, - { - "name": "akamaiDapRtdProvider", - "path": "/modules/akamaiDapRtdProvider.js", - "module": "akamaiDapRtdProvider" - }, - { - "name": "blueconicRtdProvider", - "path": "/modules/blueconicRtdProvider.js", - "module": "blueconicRtdProvider" - }, - { - "name": "brandmetricsRtdProvider", - "path": "/modules/brandmetricsRtdProvider.js", - "module": "brandmetricsRtdProvider" - }, - { - "name": "cleanioRtdProvider", - "path": "/modules/cleanioRtdProvider.js", - "module": "cleanioRtdProvider" - }, - { - "name": "idWardRtdProvider", - "path": "/modules/idWardRtdProvider.js", - "module": "idWardRtdProvider" - }, + { + "name": "1plusXRtdProvider", + "path": "/modules/1plusXRtdProvider.js", + "module": "1plusXRtdProvider" + }, + { + "name": "adnuntiusRtdProvider", + "path": "/modules/adnuntiusRtdProvider.js", + "module": "adnuntiusRtdProvider" + }, + { + "name": "airgridRtdProvider", + "path": "/modules/airgridRtdProvider.js", + "module": "airgridRtdProvider" + }, + { + "name": "akamaiDapRtdProvider", + "path": "/modules/akamaiDapRtdProvider.js", + "module": "akamaiDapRtdProvider" + }, + { + "name": "blueconicRtdProvider", + "path": "/modules/blueconicRtdProvider.js", + "module": "blueconicRtdProvider" + }, + { + "name": "brandmetricsRtdProvider", + "path": "/modules/brandmetricsRtdProvider.js", + "module": "brandmetricsRtdProvider" + }, + { + "name": "cleanioRtdProvider", + "path": "/modules/cleanioRtdProvider.js", + "module": "cleanioRtdProvider" + }, + { + "name": "idWardRtdProvider", + "path": "/modules/idWardRtdProvider.js", + "module": "idWardRtdProvider" + }, { "name": "dgkeywordRtdProvider", "path": "/modules/dgkeywordRtdProvider.js", "module": "dgkeywordRtdProvider" }, - { - "name": "imRtdProvider", - "path": "/modules/imRtdProvider.js", - "module": "imRtdProvider" - }, + { + "name": "imRtdProvider", + "path": "/modules/imRtdProvider.js", + "module": "imRtdProvider" + }, { "name": "geoedgeRtdProvider", "path": "/modules/geoedgeRtdProvider.js", @@ -135,17 +140,17 @@ { "name": "mgidRtdProvider", "path": "/modules/mgidRtdProvider.js", - "module":"mgidRtdProvider" + "module": "mgidRtdProvider" }, { "name": "aaxBlockmeterRtdProvider", "path": "/modules/aaxBlockmeterRtdProvider.js", - "module":"aaxBlockmeterRtdProvider" + "module": "aaxBlockmeterRtdProvider" }, { "name": "oneKeyRtdProvider", "path": "/modules/oneKeyRtdProvider.js", - "module":"oneKeyRtdProvider" + "module": "oneKeyRtdProvider" } ], "FPD Module": [ @@ -421,180 +426,194 @@ "path": "/modules/yuktamediaAnalyticsAdapter.js", "module": "yuktamediaAnalyticsAdapter" }, - { - "name": "intersectionRtdProvider", - "path": "/modules/intersectionRtdProvider.js", - "module": "intersectionRtdProvider" - }, - { - "name": "bidwatchAnalyticsAdapter", - "path": "/modules/bidwatchAnalyticsAdapter.js", - "module": "bidwatchAnalyticsAdapter" - }, - { - "name": "conversantAnalyticsAdapter", - "path": "/modules/conversantAnalyticsAdapter.js", - "module": "conversantAnalyticsAdapter" - }, - { - "name": "hadronAnalyticsAdapter", - "path": "/modules/hadronAnalyticsAdapter.js", - "module": "hadronAnalyticsAdapter" - }, - { - "name": "liveIntentAnalyticsAdapter", - "path": "/modules/liveIntentAnalyticsAdapter.js", - "module": "liveIntentAnalyticsAdapter" - }, - { - "name": "magniteAnalyticsAdapter", - "path": "/modules/magniteAnalyticsAdapter.js", - "module": "magniteAnalyticsAdapter" - }, - { - "name": "pianoDmpAnalyticsAdapter", - "path": "/modules/pianoDmpAnalyticsAdapter.js", - "module": "pianoDmpAnalyticsAdapter" - } + { + "name": "intersectionRtdProvider", + "path": "/modules/intersectionRtdProvider.js", + "module": "intersectionRtdProvider" + }, + { + "name": "bidwatchAnalyticsAdapter", + "path": "/modules/bidwatchAnalyticsAdapter.js", + "module": "bidwatchAnalyticsAdapter" + }, + { + "name": "conversantAnalyticsAdapter", + "path": "/modules/conversantAnalyticsAdapter.js", + "module": "conversantAnalyticsAdapter" + }, + { + "name": "hadronAnalyticsAdapter", + "path": "/modules/hadronAnalyticsAdapter.js", + "module": "hadronAnalyticsAdapter" + }, + { + "name": "liveIntentAnalyticsAdapter", + "path": "/modules/liveIntentAnalyticsAdapter.js", + "module": "liveIntentAnalyticsAdapter" + }, + { + "name": "magniteAnalyticsAdapter", + "path": "/modules/magniteAnalyticsAdapter.js", + "module": "magniteAnalyticsAdapter" + }, + { + "name": "pianoDmpAnalyticsAdapter", + "path": "/modules/pianoDmpAnalyticsAdapter.js", + "module": "pianoDmpAnalyticsAdapter" + } ], "Others": [ - { - "name": "consentManagement", - "path": "/modules/consentManagement.js", - "module": "consentManagement" - }, - { - "name": "CCPA", - "path": "/modules/consentManagementUsp.js", - "module": "consentManagementUsp" - }, - { - "name": "gdprEnforcement", - "path": "/modules/gdprEnforcement.js", - "module": "gdprEnforcement" - }, - { - "name": "gptPreAuction", - "path": "/modules/gptPreAuction.js", - "module": "gptPreAuction" - }, - { - "name": "bidViewability", - "path": "/modules/bidViewability.js", - "module": "bidViewability" - }, - { - "name": "iABCategoryTranslation", - "path": "/modules/categoryTranslation.js", - "module": "categoryTranslation" - }, - { - "name": "Currency", - "path": "/modules/currency.js", - "module": "currency" - }, - { - "name": "dataControllerModule", - "path": "/modules/dataControllerModule.js", - "module": "dataControllerModule" - }, - { - "name": "dchain", - "path": "/modules/dchain.js", - "module": "dchain" - }, - { - "name": "debugging", - "path": "/modules/debugging.js", - "module": "debugging" - }, - { - "name": "priceFloors", - "path": "/modules/priceFloors.js", - "module": "priceFloors" - }, - { - "name": "idImportLibrary", - "path": "/modules/idImportLibrary.js", - "module": "idImportLibrary" - }, - { - "name": "instreamTracking", - "path": "/modules/instreamTracking.js", - "module": "instreamTracking" - }, - { - "name": "mass", - "path": "/modules/mass.js", - "module": "mass" - }, - { - "name": "multibid", - "path": "/modules/multibid/index.js", - "module": "multibid" - }, - { - "name": "Server-to-Server Testing", - "path": "/modules/ssTesting.js", - "module": "ssTesting" - }, - { - "name": "publisherCommonId", - "path": "/modules/pubCommonId.js", - "module": "pubCommonId" - }, - { - "name": "supplyChainObject", - "path": "/modules/schain.js", - "module": "schain" - }, - { - "name": "userId", - "path": "/modules/userId/index.js", - "module": "userId" - }, - { - "name": "viewability", - "path": "/modules/viewability.js", - "module": "viewability" - }, - { - "name": "gAMExpress", - "path": "/modules/express.js", - "module": "express" - }, - { - "name": "konduitAccelerate", - "path": "/modules/konduitWrapper.js", - "module": "konduitWrapper" - }, - { - "name": "yieldmoSyntheticInventoryModule", - "path": "/modules/yieldmoSyntheticInventoryModule.js", - "module": "yieldmoSyntheticInventoryModule" - }, - { - "name": "sizeMappingV2", - "path": "/modules/sizeMappingV2.js", - "module": "sizeMappingV2" - }, - { - "name": "fledgeForGpt", - "path": "/modules/fledgeForGpt.js", - "module": "fledgeForGpt" - } -], -"Video": [ - { - "name": "jwplayerVideoProvider", - "path": "/modules/jwplayerVideoProvider.js", - "module": "jwplayerVideoProvider" - }, - { - "name": "videojsVideoProvider", - "path": "/modules/videojsVideoProvider.js", - "module": "videojsVideoProvider" - } - -] + { + "name": "consentManagement", + "path": "/modules/consentManagement.js", + "module": "consentManagement" + }, + { + "name": "CCPA", + "path": "/modules/consentManagementUsp.js", + "module": "consentManagementUsp" + }, + { + "name": "gdprEnforcement", + "path": "/modules/gdprEnforcement.js", + "module": "gdprEnforcement" + }, + { + "name": "gptPreAuction", + "path": "/modules/gptPreAuction.js", + "module": "gptPreAuction" + }, + { + "name": "bidViewability", + "path": "/modules/bidViewability.js", + "module": "bidViewability" + }, + { + "name": "iABCategoryTranslation", + "path": "/modules/categoryTranslation.js", + "module": "categoryTranslation" + }, + { + "name": "Currency", + "path": "/modules/currency.js", + "module": "currency" + }, + { + "name": "dataControllerModule", + "path": "/modules/dataControllerModule.js", + "module": "dataControllerModule" + }, + { + "name": "dchain", + "path": "/modules/dchain.js", + "module": "dchain" + }, + { + "name": "debugging", + "path": "/modules/debugging.js", + "module": "debugging" + }, + { + "name": "priceFloors", + "path": "/modules/priceFloors.js", + "module": "priceFloors" + }, + { + "name": "idImportLibrary", + "path": "/modules/idImportLibrary.js", + "module": "idImportLibrary" + }, + { + "name": "instreamTracking", + "path": "/modules/instreamTracking.js", + "module": "instreamTracking" + }, + { + "name": "mass", + "path": "/modules/mass.js", + "module": "mass" + }, + { + "name": "multibid", + "path": "/modules/multibid/index.js", + "module": "multibid" + }, + { + "name": "Server-to-Server Testing", + "path": "/modules/ssTesting.js", + "module": "ssTesting" + }, + { + "name": "publisherCommonId", + "path": "/modules/pubCommonId.js", + "module": "pubCommonId" + }, + { + "name": "supplyChainObject", + "path": "/modules/schain.js", + "module": "schain" + }, + { + "name": "userId", + "path": "/modules/userId/index.js", + "module": "userId" + }, + { + "name": "viewability", + "path": "/modules/viewability.js", + "module": "viewability" + }, + { + "name": "gAMExpress", + "path": "/modules/express.js", + "module": "express" + }, + { + "name": "konduitAccelerate", + "path": "/modules/konduitWrapper.js", + "module": "konduitWrapper" + }, + { + "name": "yieldmoSyntheticInventoryModule", + "path": "/modules/yieldmoSyntheticInventoryModule.js", + "module": "yieldmoSyntheticInventoryModule" + }, + { + "name": "sizeMappingV2", + "path": "/modules/sizeMappingV2.js", + "module": "sizeMappingV2" + }, + { + "name": "trustpidSystem", + "path": "/modules/trustpidSystem.js", + "module": "trustpidSystem" + }, + { + "name": "idLibrary", + "path": "/modules/idLibrary.js", + "module": "idLibrary" + }, + { + "name": "fledgeForGpt", + "path": "/modules/fledgeForGpt.js", + "module": "fledgeForGpt" + }, + { + "name": "bidViewabilityIO", + "path": "/modules/bidViewabilityIO.js", + "module": "bidViewabilityIO" + } + ], + "Video": [ + { + "name": "jwplayerVideoProvider", + "path": "/modules/jwplayerVideoProvider.js", + "module": "jwplayerVideoProvider" + }, + { + "name": "videojsVideoProvider", + "path": "/modules/videojsVideoProvider.js", + "module": "videojsVideoProvider" + } + ] } From 44c80f66b06c59d98cf2238daa4e9ff3f48d4e0b Mon Sep 17 00:00:00 2001 From: kapil-tuptewar Date: Tue, 29 Aug 2023 16:21:42 +0530 Subject: [PATCH 6/6] Added comments for future reference --- modules/pubmaticAnalyticsAdapter.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/pubmaticAnalyticsAdapter.js b/modules/pubmaticAnalyticsAdapter.js index 6af0824f28e..3f80c5dcfac 100755 --- a/modules/pubmaticAnalyticsAdapter.js +++ b/modules/pubmaticAnalyticsAdapter.js @@ -580,6 +580,8 @@ function bidResponseHandler(args) { bid.source = formatSource(bid.source || args.source); setBidStatus(bid, args); const latency = args?.timeToRespond || Date.now() - cache.auctions[args.auctionId].timestamp; + // Checking if latency is greater than auctiontime+100, if yes instead of logging actual latency log + // auctiontime+100 to keep actual values and to keep avarage latency in expected range. bid.partnerTimeToRespond = window.PWT?.versionDetails?.timeout ? (latency > (window.PWT.versionDetails.timeout + 100) ? (window.PWT.versionDetails.timeout + 100) : latency) : latency; bid.clientLatencyTimeMs = Date.now() - cache.auctions[args.auctionId].timestamp; bid.bidResponse = parseBidResponse(args);