prebid.js 7.53.0 → 8.0.0

package/modules/optidigitalBidAdapter.js

@@ -46,6 +46,7 @@ export const spec = {
       referrer: (bidderRequest.refererInfo && bidderRequest.refererInfo.page) ? bidderRequest.refererInfo.page : '',
       hb_version: '$prebid.version$',
       deviceWidth: document.documentElement.clientWidth,
+      // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
       auctionId: deepAccess(validBidRequests[0], 'auctionId'),
       bidderRequestId: deepAccess(validBidRequests[0], 'bidderRequestId'),
       publisherId: deepAccess(validBidRequests[0], 'params.publisherId'),
@@ -107,7 +108,6 @@ export const spec = {
       serverResponse.bids.forEach((bid) => {
         const bidResponse = {
           placementId: bid.placementId,
-          transactionId: bid.transactionId,
           requestId: bid.bidId,
           ttl: bid.ttl,
           creativeId: bid.creativeId,
@@ -161,7 +161,7 @@ function buildImp(bidRequest, ortb2) {
     sizes: parseSizesInput(deepAccess(bidRequest, 'mediaTypes.banner.sizes')),
     bidId: deepAccess(bidRequest, 'bidId'),
     adUnitCode: deepAccess(bidRequest, 'adUnitCode'),
-    transactionId: deepAccess(bidRequest, 'transactionId'),
+    transactionId: deepAccess(bidRequest, 'ortb2Imp.ext.tid'),
     placementId: deepAccess(bidRequest, 'params.placementId')
   };
 

package/modules/orbidderBidAdapter.js

@@ -101,7 +101,8 @@ export const spec = {
           pageUrl: referer,
           bidId: bidRequest.bidId,
           auctionId: bidRequest.auctionId,
-          transactionId: bidRequest.transactionId,
+          // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
+          transactionId: bidRequest.ortb2Imp?.ext?.tid,
           adUnitCode: bidRequest.adUnitCode,
           bidRequestCount: bidRequest.bidRequestCount,
           params: bidRequest.params,

package/modules/otmBidAdapter.js

@@ -53,7 +53,8 @@ export const spec = {
       const domain = isStr(bid.params.domain) ? bid.params.domain : topOrigin
       const cur = getValue(bid.params, 'currency') || DEFAULT_CURRENCY
       const bidid = getBidIdParameter('bidId', bid)
-      const transactionid = getBidIdParameter('transactionId', bid)
+      const transactionid = bid.ortb2Imp?.ext?.tid || '';
+      // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
       const auctionid = getBidIdParameter('auctionId', bid)
       const bidfloor = _getBidFloor(bid)
 
@@ -112,7 +113,6 @@ export const spec = {
             netRevenue: true,
             ad: bid.ad,
             ttl: bid.ttl,
-            transactionId: bid.transactionid,
             meta: {
               advertiserDomains: bid.adDomain ? [bid.adDomain] : []
             }

package/modules/outbrainBidAdapter.js

@@ -1,16 +1,14 @@
 // jshint esversion: 6, es3: false, node: true
 'use strict';
 
-import {
-  registerBidder
-} from '../src/adapters/bidderFactory.js';
-import { NATIVE, BANNER, VIDEO } from '../src/mediaTypes.js';
-import { OUTSTREAM } from '../src/video.js';
-import { deepAccess, deepSetValue, replaceAuctionPrice, _map, isArray, logWarn } from '../src/utils.js';
-import { ajax } from '../src/ajax.js';
-import { config } from '../src/config.js';
-import { convertOrtbRequestToProprietaryNative } from '../src/native.js';
-import { Renderer } from '../src/Renderer.js';
+import {registerBidder} from '../src/adapters/bidderFactory.js';
+import {BANNER, NATIVE, VIDEO} from '../src/mediaTypes.js';
+import {OUTSTREAM} from '../src/video.js';
+import {_map, deepAccess, deepSetValue, isArray, logWarn, replaceAuctionPrice} from '../src/utils.js';
+import {ajax} from '../src/ajax.js';
+import {config} from '../src/config.js';
+import {convertOrtbRequestToProprietaryNative} from '../src/native.js';
+import {Renderer} from '../src/Renderer.js';
 
 const BIDDER_CODE = 'outbrain';
 const GVLID = 164;
@@ -107,7 +105,7 @@ export const spec = {
     });
 
     const request = {
-      id: bidderRequest.auctionId,
+      id: bidderRequest.bidderRequestId,
       site: { page, publisher },
       device: { ua },
       source: { fd: 1 },

package/modules/oxxionRtdProvider.js

@@ -10,8 +10,8 @@ const allAdUnits = [];
 export const oxxionSubmodule = {
   name: 'oxxionRtd',
   init: init,
-  onAuctionEndEvent: onAuctionEnd,
   getBidRequestData: getAdUnits,
+  onBidResponseEvent: insertVideoTracking,
 };
 
 function init(config, userConsent) {
@@ -32,8 +32,10 @@ function getAdUnits(reqBidsConfigObj, callback, config, userConsent) {
   }
 }
 
-function insertVideoTracking(bidResponse, config, maxCpm) {
+function insertVideoTracking(bidResponse, config, userConsent) {
+  // this should only be do for video bids
   if (bidResponse.mediaType === 'video') {
+    let maxCpm = 0;
     const trackingUrl = getImpUrl(config, bidResponse, maxCpm);
     if (!trackingUrl) {
       return;
@@ -42,7 +44,8 @@ function insertVideoTracking(bidResponse, config, maxCpm) {
     if (bidResponse.vastUrl) {
       bidResponse.vastImpUrl = bidResponse.vastImpUrl
         ? trackingUrl + '&url=' + encodeURI(bidResponse.vastImpUrl)
-        : trackingUrl
+        : trackingUrl;
+      logInfo(LOG_PREFIX + 'insert into vastImpUrl for adId ' + bidResponse.adId);
     }
     // Vast XML document
     if (bidResponse.vastXml !== undefined) {
@@ -87,33 +90,8 @@ function getImpUrl(config, data, maxCpm) {
     }
     return acc;
   }, '');
-  const cpmIncrement = Math.round(100000 * (data.cpm - maxCpm)) / 100000;
+  const cpmIncrement = 0.0;
   return trackingImpUrl + 'cpmIncrement=' + cpmIncrement + '&context=' + context;
 }
 
-function onAuctionEnd(auctionDetails, config, userConsent) {
-  const transactionsToCheck = {}
-  auctionDetails.adUnits.forEach(adunit => {
-    if (config.params.contexts.includes(deepAccess(adunit, 'mediaTypes.video.context'))) {
-      transactionsToCheck[adunit.transactionId] = {'bids': {}, 'maxCpm': 0.0, 'secondMaxCpm': 0.0};
-    }
-  });
-  for (const key in auctionDetails.bidsReceived) {
-    if (auctionDetails.bidsReceived[key].transactionId in transactionsToCheck) {
-      transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['bids'][auctionDetails.bidsReceived[key].adId] = {'key': key, 'cpm': auctionDetails.bidsReceived[key].cpm};
-      if (auctionDetails.bidsReceived[key].cpm > transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['maxCpm']) {
-        transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['secondMaxCpm'] = transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['maxCpm'];
-        transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['maxCpm'] = auctionDetails.bidsReceived[key].cpm;
-      } else if (auctionDetails.bidsReceived[key].cpm > transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['secondMaxCpm']) {
-        transactionsToCheck[auctionDetails.bidsReceived[key].transactionId]['secondMaxCpm'] = auctionDetails.bidsReceived[key].cpm;
-      }
-    }
-  };
-  Object.keys(transactionsToCheck).forEach(transaction => {
-    Object.keys(transactionsToCheck[transaction]['bids']).forEach(bid => {
-      insertVideoTracking(auctionDetails.bidsReceived[transactionsToCheck[transaction]['bids'][bid].key], config, transactionsToCheck[transaction].secondMaxCpm);
-    });
-  });
-}
-
 submodule('realTimeData', oxxionSubmodule);

package/modules/ozoneBidAdapter.js

@@ -262,6 +262,7 @@ export const spec = {
       obj.placementId = placementId;
       deepSetValue(obj, 'ext.prebid', {'storedrequest': {'id': placementId}});
       obj.ext[whitelabelBidder] = {};
+      // TODO: fix auctionId/transactionID leak: https://github.com/prebid/Prebid.js/issues/9781
       obj.ext[whitelabelBidder].adUnitCode = ozoneBidRequest.adUnitCode; // eg. 'mpu'
       obj.ext[whitelabelBidder].transactionId = ozoneBidRequest.transactionId; // this is the transactionId PER adUnit, common across bidders for this unit
       if (ozoneBidRequest.params.hasOwnProperty('customData')) {
@@ -381,7 +382,7 @@ export const spec = {
       ozoneRequestSingle.auctionId = imp.ext[whitelabelBidder].transactionId; // not sure if this should be here?
       ozoneRequestSingle.imp = [imp];
       ozoneRequestSingle.ext = extObj;
-      deepSetValue(ozoneRequestSingle, 'source.tid', imp.ext[whitelabelBidder].transactionId);// RTB 2.5 : tid is Transaction ID that must be common across all participants in this bid request (e.g., potentially multiple exchanges).
+      deepSetValue(ozoneRequestSingle, 'source.tid', bidderRequest.auctionId);// RTB 2.5 : tid is Transaction ID that must be common across all participants in this bid request (e.g., potentially multiple exchanges).
       deepSetValue(ozoneRequestSingle, 'user.ext.eids', userExtEids);
       logInfo('buildRequests RequestSingle (for non-single) = ', ozoneRequestSingle);
       return {

package/modules/padsquadBidAdapter.js

@@ -1,4 +1,4 @@
-import { logInfo, deepAccess } from '../src/utils.js';
+import {deepAccess, logInfo} from '../src/utils.js';
 import {registerBidder} from '../src/adapters/bidderFactory.js';
 import {BANNER} from '../src/mediaTypes.js';
 
@@ -40,7 +40,7 @@ export const spec = {
     }));
 
     const openrtbRequest = {
-      id: bidderRequest.auctionId,
+      id: bidderRequest.bidderRequestId,
       imp: impressions,
       site: {
         domain: bidderRequest?.refererInfo?.domain,

package/modules/pixfutureBidAdapter.js

@@ -1,20 +1,12 @@
-import { registerBidder } from '../src/adapters/bidderFactory.js';
-import { getStorageManager } from '../src/storageManager.js';
-import { BANNER } from '../src/mediaTypes.js';
-import { config } from '../src/config.js';
-import { find, includes } from '../src/polyfill.js';
-import {
-  convertCamelToUnderscore,
-  deepAccess,
-  isArray,
-  isEmpty,
-  isFn,
-  isNumber,
-  isPlainObject,
-  transformBidderParamKeywords
-} from '../src/utils.js';
-import { auctionManager } from '../src/auctionManager.js';
+import {registerBidder} from '../src/adapters/bidderFactory.js';
+import {getStorageManager} from '../src/storageManager.js';
+import {BANNER} from '../src/mediaTypes.js';
+import {config} from '../src/config.js';
+import {find, includes} from '../src/polyfill.js';
+import {convertCamelToUnderscore, deepAccess, isArray, isFn, isNumber, isPlainObject} from '../src/utils.js';
+import {auctionManager} from '../src/auctionManager.js';
 import {getGlobal} from '../src/prebidGlobal.js';
+import {getANKeywordParam} from '../libraries/appnexusKeywords/anKeywords.js';
 
 const SOURCE = 'pbjs';
 const storageManager = getStorageManager({bidderCode: 'pixfuture'});
@@ -136,8 +128,9 @@ export const spec = {
           v: getGlobal().version,
           pageUrl: referer,
           bidId: bidRequest.bidId,
+          // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
           auctionId: bidRequest.auctionId,
-          transactionId: bidRequest.transactionId,
+          transactionId: bidRequest.ortb2Imp?.ext?.tid,
           adUnitCode: bidRequest.adUnitCode,
           bidRequestCount: bidRequest.bidRequestCount,
           sizes: bidRequest.sizes,
@@ -281,14 +274,7 @@ function bidToTag(bid) {
   if (bid.params.externalImpId) {
     tag.external_imp_id = bid.params.externalImpId;
   }
-  if (!isEmpty(bid.params.keywords)) {
-    let keywords = transformBidderParamKeywords(bid.params.keywords);
-
-    if (keywords.length > 0) {
-      keywords.forEach(deleteValues);
-    }
-    tag.keywords = keywords;
-  }
+  tag.keywords = getANKeywordParam(bid.ortb2, bid.params.keywords)
 
   let gpid = deepAccess(bid, 'ortb2Imp.ext.data.pbadslot');
   if (gpid) {
@@ -302,7 +288,7 @@ function bidToTag(bid) {
   if (bid.params.frameworks && isArray(bid.params.frameworks)) {
     tag['banner_frameworks'] = bid.params.frameworks;
   }
-
+  // TODO: why does this need to iterate through every adUnit?
   let adUnit = find(auctionManager.getAdUnits(), au => bid.transactionId === au.transactionId);
   if (adUnit && adUnit.mediaTypes && adUnit.mediaTypes.banner) {
     tag.ad_types.push(BANNER);
@@ -368,14 +354,4 @@ function getBidFloor(bid) {
   return null;
 }
 
-function deleteValues(keyPairObj) {
-  if (isPopulatedArray(keyPairObj.value) && keyPairObj.value[0] === '') {
-    delete keyPairObj.value;
-  }
-}
-
-function isPopulatedArray(arr) {
-  return !!(isArray(arr) && arr.length > 0);
-}
-
 registerBidder(spec);

package/modules/prebidServerBidAdapter/index.js

@@ -96,10 +96,8 @@ export const s2sDefaultConfig = {
   adapterOptions: {},
   syncUrlModifier: {},
   ortbNative: {
-    context: 1,
-    plcmttype: 1,
     eventtrackers: [
-      {event: 1, methods: [1]}
+      {event: 1, methods: [1, 2]}
     ],
   }
 };

package/modules/prebidServerBidAdapter/ortbConverter.js

@@ -18,6 +18,12 @@ import {setImpBidParams} from '../../libraries/pbsExtensions/processors/params.j
 import {SUPPORTED_MEDIA_TYPES} from '../../libraries/pbsExtensions/processors/mediaType.js';
 import {IMP, REQUEST, RESPONSE} from '../../src/pbjsORTB.js';
 import {beConvertCurrency} from '../../src/utils/currency.js';
+import {redactor} from '../../src/activities/redactor.js';
+import {s2sActivityParams} from '../../src/adapterManager.js';
+import {activityParams} from '../../src/activities/activityParams.js';
+import {MODULE_TYPE_BIDDER} from '../../src/activities/modules.js';
+import {isActivityAllowed} from '../../src/activities/rules.js';
+import {ACTIVITY_TRANSMIT_TID} from '../../src/activities/activities.js';
 
 const DEFAULT_S2S_TTL = 60;
 const DEFAULT_S2S_CURRENCY = 'USD';
@@ -51,7 +57,6 @@ const PBS_CONVERTER = ortbConverter({
       const request = buildRequest(imps, proxyBidderRequest, context);
 
       request.tmax = s2sBidRequest.s2sConfig.timeout;
-      deepSetValue(request, 'source.tid', proxyBidderRequest.auctionId);
 
       [request.app, request.dooh, request.site].forEach(section => {
         if (section && !section.publisher?.id) {
@@ -69,6 +74,10 @@ const PBS_CONVERTER = ortbConverter({
         deepSetValue(request, 'ext.prebid.data.eidpermissions', eidPermissions);
       }
 
+      if (!context.transmitTids) {
+        deepSetValue(request, 'ext.prebid.createtids', false);
+      }
+
       return request;
     }
   },
@@ -103,9 +112,8 @@ const PBS_CONVERTER = ortbConverter({
 
     // because core has special treatment for PBS adapter responses, we need some additional processing
     bidResponse.requestTimestamp = context.requestTimestamp;
-    const status = bid.price !== 0 ? CONSTANTS.STATUS.GOOD : CONSTANTS.STATUS.NO_BID;
     return {
-      bid: Object.assign(createBid(status, {
+      bid: Object.assign(createBid(CONSTANTS.STATUS.GOOD, {
         src: CONSTANTS.S2S.SRC,
         bidId: bidRequest ? (bidRequest.bidId || bidRequest.bid_Id) : null,
         transactionId: context.adUnit.transactionId,
@@ -167,13 +175,17 @@ const PBS_CONVERTER = ortbConverter({
           deepSetValue(ortbRequest, 'ext.prebid', mergeDeep(ortbRequest.ext?.prebid || {}, context.s2sBidRequest.s2sConfig.extPrebid));
         }
 
+        // for global FPD, check allowed activities against "prebid.pbsBidAdapter"...
+        context.getRedactor().ortb2(ortbRequest);
+
         const fpdConfigs = Object.entries(context.s2sBidRequest.ortb2Fragments?.bidder || {}).filter(([bidder]) => {
           const bidders = context.s2sBidRequest.s2sConfig.bidders;
           const allowUnknownBidderCodes = context.s2sBidRequest.s2sConfig.allowUnknownBidderCodes;
           return allowUnknownBidderCodes || (bidders && bidders.includes(bidder));
         }).map(([bidder, ortb2]) => ({
+          // ... but for bidder specific FPD we can use the actual bidder
           bidders: [bidder],
-          config: {ortb2}
+          config: {ortb2: context.getRedactor(bidder).ortb2(ortb2)}
         }));
         if (fpdConfigs.length) {
           deepSetValue(ortbRequest, 'ext.prebid.bidderconfig', fpdConfigs);
@@ -241,10 +253,25 @@ export function buildPBSRequest(s2sBidRequest, bidderRequests, adUnits, requeste
   const requestTimestamp = timestamp();
   const impIds = new Set();
   const proxyBidRequests = [];
+  const s2sParams = s2sActivityParams(s2sBidRequest.s2sConfig);
+
+  const getRedactor = (() => {
+    const global = redactor(s2sParams);
+    const bidders = {};
+    return (bidder) => {
+      if (bidder == null) return global;
+      if (!bidders.hasOwnProperty(bidder)) {
+        bidders[bidder] = redactor(activityParams(MODULE_TYPE_BIDDER, bidder));
+      }
+      return bidders[bidder]
+    }
+  })();
+
+  adUnits = adUnits.map((au) => getRedactor().bidRequest(au))
 
   adUnits.forEach(adUnit => {
     const actualBidRequests = new Map();
-
+    adUnits.bids = adUnit.bids.map(br => getRedactor(br.bidder).bidRequest(br));
     adUnit.bids.forEach((bid) => {
       if (bid.mediaTypes != null) {
         // TODO: support labels / conditional bids
@@ -286,6 +313,8 @@ export function buildPBSRequest(s2sBidRequest, bidderRequests, adUnits, requeste
       actualBidderRequests: bidderRequests,
       eidPermissions,
       nativeRequest: s2sBidRequest.s2sConfig.ortbNative,
+      getRedactor,
+      transmitTids: isActivityAllowed(ACTIVITY_TRANSMIT_TID, s2sParams),
     }
   });
 }

package/modules/priceFloors.js

@@ -151,8 +151,9 @@ export function getFirstMatchingFloor(floorData, bidObject, responseObject = {})
     matchingRule
   };
   // use adUnit floorMin as priority!
-  if (typeof deepAccess(bidObject, 'ortb2Imp.ext.prebid.floorMin') === 'number') {
-    matchingData.floorMin = bidObject.ortb2Imp.ext.prebid.floorMin;
+  const floorMin = deepAccess(bidObject, 'ortb2Imp.ext.prebid.floors.floorMin');
+  if (typeof floorMin === 'number') {
+    matchingData.floorMin = floorMin;
   }
   matchingData.matchingFloor = Math.max(matchingData.floorMin, matchingData.floorRuleValue);
   // save for later lookup if needed
@@ -632,7 +633,7 @@ export function handleSetFloorsConfig(config) {
       'bidAdjustment', bidAdjustment => bidAdjustment !== false, // defaults to true
     ]),
     'additionalSchemaFields', additionalSchemaFields => typeof additionalSchemaFields === 'object' && Object.keys(additionalSchemaFields).length > 0 ? addFieldOverrides(additionalSchemaFields) : undefined,
-    'data', data => (data && parseFloorData(data, 'setConfig')) || _floorsConfig.data // do not overwrite if passed in data not valid
+    'data', data => (data && parseFloorData(data, 'setConfig')) || undefined
   ]);
 
   // if enabled then do some stuff
@@ -744,10 +745,9 @@ export const addBidResponseHook = timedBidResponseHook('priceFloors', function a
   // now do the compare!
   if (shouldFloorBid(floorData, floorInfo, bid)) {
     // bid fails floor -> throw it out
-    // continue with a "NO_BID" bid, TODO: remove this in v8
-    const flooredBid = reject(CONSTANTS.REJECTION_REASON.FLOOR_NOT_MET);
-    logWarn(`${MODULE_NAME}: ${flooredBid.bidderCode}'s Bid Response for ${adUnitCode} was rejected due to floor not met (adjusted cpm: ${bid?.floorData?.cpmAfterAdjustments}, floor: ${floorInfo?.matchingFloor})`, bid);
-    return fn.call(this, adUnitCode, flooredBid, reject);
+    reject(CONSTANTS.REJECTION_REASON.FLOOR_NOT_MET);
+    logWarn(`${MODULE_NAME}: ${bid.bidderCode}'s Bid Response for ${adUnitCode} was rejected due to floor not met (adjusted cpm: ${bid?.floorData?.cpmAfterAdjustments}, floor: ${floorInfo?.matchingFloor})`, bid);
+    return;
   }
   return fn.call(this, adUnitCode, bid, reject);
 });

package/modules/prismaBidAdapter.js

@@ -1,8 +1,8 @@
 import {ajax} from '../src/ajax.js';
 import {config} from '../src/config.js';
-import { transformBidderParamKeywords } from '../src/utils.js';
 import {registerBidder} from '../src/adapters/bidderFactory.js';
 import {BANNER, VIDEO} from '../src/mediaTypes.js';
+import {getANKeywordParam} from '../libraries/appnexusKeywords/anKeywords.js';
 
 const BIDDER_CODE = 'prisma';
 const BIDDER_URL = 'https://prisma.nexx360.io/prebid';
@@ -70,12 +70,13 @@ export const spec = {
         tagId: adunitValue.params.tagId,
         label: adunitValue.adUnitCode,
         bidId: adunitValue.bidId,
+        // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
         auctionId: adunitValue.auctionId,
-        transactionId: adunitValue.transactionId,
+        transactionId: adunitValue.ortb2Imp?.ext?.tid,
         mediatypes: adunitValue.mediaTypes,
         bidfloor: 0,
         bidfloorCurrency: 'USD',
-        keywords: adunitValue.params.keywords ? transformBidderParamKeywords(adunitValue.params.keywords) : [],
+        keywords: getANKeywordParam(bidderRequest.ortb2, adunitValue.params.keywords)
       }
       adUnits.push(foo);
       if (adunitValue.userIdAsEids) userEids = adunitValue.userIdAsEids;

package/modules/proxistoreBidAdapter.js

@@ -24,8 +24,9 @@ function _createServerRequest(bidRequests, bidderRequest) {
     sizeIds.push(sizeId);
   });
   var payload = {
+    // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
     auctionId: bidRequests[0].auctionId,
-    transactionId: bidRequests[0].auctionId,
+    transactionId: bidRequests[0].ortb2Imp?.ext?.tid,
     bids: sizeIds,
     website: bidRequests[0].params.website,
     language: bidRequests[0].params.language,

package/modules/pubgeniusBidAdapter.js

@@ -1,7 +1,7 @@
-import { registerBidder } from '../src/adapters/bidderFactory.js';
-import { ajax } from '../src/ajax.js';
-import { config } from '../src/config.js';
-import { BANNER, VIDEO } from '../src/mediaTypes.js';
+import {registerBidder} from '../src/adapters/bidderFactory.js';
+import {ajax} from '../src/ajax.js';
+import {config} from '../src/config.js';
+import {BANNER, VIDEO} from '../src/mediaTypes.js';
 import {
   deepAccess,
   deepSetValue,
@@ -41,7 +41,7 @@ export const spec = {
 
   buildRequests: function (bidRequests, bidderRequest) {
     const data = {
-      id: bidderRequest.auctionId,
+      id: bidderRequest.bidderRequestId,
       imp: bidRequests.map(buildImp),
       tmax: bidderRequest.timeout,
       ext: {

package/modules/pubmaticAnalyticsAdapter.js

@@ -110,10 +110,6 @@ function setBidStatus(bid, args) {
       bid.status = SUCCESS;
       delete bid.error; // it's possible for this to be set by a previous timeout
       break;
-    case CONSTANTS.STATUS.NO_BID:
-      bid.status = NO_BID;
-      delete bid.error;
-      break;
     default:
       bid.status = ERROR;
       bid.error = {
@@ -256,6 +252,10 @@ export function getMetadata(meta) {
   return metaObj;
 }
 
+function isS2SBidder(bidder) {
+  return (s2sBidders.indexOf(bidder) > -1) ? 1 : 0
+}
+
 function gatherPartnerBidsForAdUnitForLogger(adUnit, adUnitId, highestBid) {
   highestBid = (highestBid && highestBid.length > 0) ? highestBid[0] : null;
   return Object.keys(adUnit.bids).reduce(function(partnerBids, bidId) {
@@ -275,7 +275,7 @@ function gatherPartnerBidsForAdUnitForLogger(adUnit, adUnitId, highestBid) {
         'l1': bid.bidResponse ? bid.clientLatencyTimeMs : 0,
         'l2': 0,
         'adv': bid.bidResponse ? getAdDomain(bid.bidResponse) || undefined : undefined,
-        'ss': (s2sBidders.indexOf(bid.bidder) > -1) ? 1 : 0,
+        'ss': isS2SBidder(bid.bidder),
         't': (bid.status == ERROR && bid.error.code == TIMEOUT_ERROR) ? 1 : 0,
         'wb': (highestBid && highestBid.adId === bid.adId ? 1 : 0),
         'mi': bid.bidResponse ? (bid.bidResponse.mi || undefined) : undefined,
@@ -311,6 +311,14 @@ function getAdUnit(adUnits, adUnitId) {
   return adUnits.filter(adUnit => (adUnit.divID && adUnit.divID == adUnitId) || (adUnit.code == adUnitId))[0];
 }
 
+function getTgId() {
+  var testGroupId = parseInt(config.getConfig('testGroupId') || 0);
+  if (testGroupId <= 15 && testGroupId >= 0) {
+    return testGroupId;
+  }
+  return 0;
+}
+
 function executeBidsLoggerCall(e, highestCpmBids) {
   let auctionId = e.auctionId;
   let referrer = config.getConfig('pageUrl') || cache.auctions[auctionId].referer || '';
@@ -337,13 +345,7 @@ function executeBidsLoggerCall(e, highestCpmBids) {
   outputObj['pid'] = '' + profileId;
   outputObj['pdvid'] = '' + profileVersionId;
   outputObj['dvc'] = {'plt': getDevicePlatform()};
-  outputObj['tgid'] = (function() {
-    var testGroupId = parseInt(config.getConfig('testGroupId') || 0);
-    if (testGroupId <= 15 && testGroupId >= 0) {
-      return testGroupId;
-    }
-    return 0;
-  })();
+  outputObj['tgid'] = getTgId();
 
   if (floorData) {
     outputObj['fmv'] = floorData.floorRequestData ? floorData.floorRequestData.modelVersion || undefined : undefined;
@@ -391,6 +393,12 @@ function executeBidWonLoggerCall(auctionId, adUnitId) {
 
   const adapterName = getAdapterNameForAlias(winningBid.adapterCode || winningBid.bidder);
   let origAdUnit = getAdUnit(cache.auctions[auctionId].origAdUnits, adUnitId) || {};
+  let auctionCache = cache.auctions[auctionId];
+  let floorData = auctionCache.floorData;
+  let referrer = config.getConfig('pageUrl') || cache.auctions[auctionId].referer || '';
+  let adv = winningBid.bidResponse ? getAdDomain(winningBid.bidResponse) || undefined : undefined;
+  let fskp = floorData ? (floorData.floorRequestData ? (floorData.floorRequestData.skipped == false ? 0 : 1) : undefined) : undefined;
+
   let pixelURL = END_POINT_WIN_BID_LOGGER;
   pixelURL += 'pubid=' + publisherId;
   pixelURL += '&purl=' + enc(config.getConfig('pageUrl') || cache.auctions[auctionId].referer || '');
@@ -407,6 +415,17 @@ function executeBidWonLoggerCall(auctionId, adUnitId) {
   pixelURL += '&eg=' + enc(winningBid.bidResponse.bidGrossCpmUSD);
   pixelURL += '&kgpv=' + enc(getValueForKgpv(winningBid, adUnitId));
   pixelURL += '&piid=' + enc(winningBid.bidResponse.partnerImpId || EMPTY_STRING);
+
+  pixelURL += '&plt=' + enc(getDevicePlatform());
+  pixelURL += '&psz=' + enc((winningBid?.bidResponse?.dimensions?.width || '0') + 'x' +
+    (winningBid?.bidResponse?.dimensions?.height || '0'));
+  pixelURL += '&tgid=' + enc(getTgId());
+  adv && (pixelURL += '&adv=' + enc(adv));
+  pixelURL += '&orig=' + enc(getDomainFromUrl(referrer));
+  pixelURL += '&ss=' + enc(isS2SBidder(winningBid.bidder));
+  (fskp != undefined) && (pixelURL += '&fskp=' + enc(fskp));
+  pixelURL += '&af=' + enc(winningBid.bidResponse ? (winningBid.bidResponse.mediaType || undefined) : undefined);
+
   ajax(
     pixelURL,
     null,

package/modules/pubmaticBidAdapter.js

@@ -1078,7 +1078,7 @@ export const spec = {
       }
       conf.pubId = conf.pubId || bid.params.publisherId;
       conf = _handleCustomParams(bid.params, conf);
-      conf.transactionId = bid.transactionId;
+      conf.transactionId = bid.ortb2Imp?.ext?.tid;
       if (bidCurrency === '') {
         bidCurrency = bid.params.currency || UNDEFINED;
       } else if (bid.params.hasOwnProperty('currency') && bidCurrency !== bid.params.currency) {
@@ -1110,6 +1110,7 @@ export const spec = {
     payload.ext.wrapper = {};
     payload.ext.wrapper.profile = parseInt(conf.profId) || UNDEFINED;
     payload.ext.wrapper.version = parseInt(conf.verId) || UNDEFINED;
+    // TODO: fix auctionId leak: https://github.com/prebid/Prebid.js/issues/9781
     payload.ext.wrapper.wiid = conf.wiid || bidderRequest.auctionId;
     // eslint-disable-next-line no-undef
     payload.ext.wrapper.wv = $$REPO_AND_VERSION$$;
@@ -1132,8 +1133,9 @@ export const spec = {
 
     payload.user.gender = (conf.gender ? conf.gender.trim() : UNDEFINED);
     payload.user.geo = {};
-    payload.user.geo.lat = _parseSlotParam('lat', conf.lat);
-    payload.user.geo.lon = _parseSlotParam('lon', conf.lon);
+    // TODO: fix lat and long to only come from request object, not params
+    payload.user.geo.lat = _parseSlotParam('lat', 0);
+    payload.user.geo.lon = _parseSlotParam('lon', 0);
     payload.user.yob = _parseSlotParam('yob', conf.yob);
     payload.device.geo = payload.user.geo;
     payload.site.page = conf.kadpageurl.trim() || payload.site.page.trim();
@@ -1153,7 +1155,7 @@ export const spec = {
     payload.device.language = payload.device.language && payload.device.language.split('-')[0];
 
     // passing transactionId in source.tid
-    deepSetValue(payload, 'source.tid', conf.transactionId);
+    deepSetValue(payload, 'source.tid', bidderRequest?.ortb2?.source?.tid);
 
     // test bids
     if (window.location.href.indexOf('pubmaticTest=true') !== -1) {

package/modules/pubwiseBidAdapter.js

@@ -194,7 +194,7 @@ export const spec = {
       _parseAdSlot(bid);
 
       conf = _handleCustomParams(bid.params, conf);
-      conf.transactionId = bid.transactionId;
+      conf.transactionId = bid.ortb2Imp?.ext?.tid;
       bidCurrency = bid.params.currency || UNDEFINED;
       bid.params.currency = bidCurrency;
 
@@ -224,8 +224,9 @@ export const spec = {
     payload.site.publisher.id = bid.params.siteId.trim();
     payload.user.gender = (conf.gender ? conf.gender.trim() : UNDEFINED);
     payload.user.geo = {};
-    payload.user.geo.lat = _parseSlotParam('lat', conf.lat);
-    payload.user.geo.lon = _parseSlotParam('lon', conf.lon);
+    // TODO: fix lat and long to only come from ortb2 object so publishers can control precise location
+    payload.user.geo.lat = _parseSlotParam('lat', 0);
+    payload.user.geo.lon = _parseSlotParam('lon', 0);
     payload.user.yob = _parseSlotParam('yob', conf.yob);
     payload.device.geo = payload.user.geo;
     payload.site.page = payload.site?.page?.trim();
@@ -242,7 +243,7 @@ export const spec = {
     }
 
     // passing transactionId in source.tid
-    deepSetValue(payload, 'source.tid', bidderRequest?.auctionId);
+    deepSetValue(payload, 'source.tid', bidderRequest?.ortb2?.source?.tid);
 
     // schain
     if (validBidRequests[0].schain) {
@@ -528,7 +529,7 @@ function _createImpressionObject(bid, conf) {
     secure: 1,
     bidfloorcur: bid.params.currency ? _parseSlotParam('currency', bid.params.currency) : DEFAULT_CURRENCY, // capitalization dicated by 3.2.4 spec
     ext: {
-      tid: (bid.transactionId ? bid.transactionId : '')
+      tid: bid.ortb2Imp?.ext?.tid || ''
     }
   };