prebid.js 6.7.0 → 6.11.0

package/src/bidfactory.js

@@ -14,16 +14,18 @@ import { getUniqueIdentifierStr } from './utils.js';
  dealId,
  priceKeyString;
  */
-function Bid(statusCode, bidRequest) {
-  var _bidSrc = (bidRequest && bidRequest.src) || 'client';
+function Bid(statusCode, {src = 'client', bidder = '', bidId, transactionId, auctionId} = {}) {
+  var _bidSrc = src;
   var _statusCode = statusCode || 0;
 
-  this.bidderCode = (bidRequest && bidRequest.bidder) || '';
+  this.bidderCode = bidder;
   this.width = 0;
   this.height = 0;
   this.statusMessage = _getStatus();
   this.adId = getUniqueIdentifierStr();
-  this.requestId = bidRequest && bidRequest.bidId;
+  this.requestId = bidId;
+  this.transactionId = transactionId;
+  this.auctionId = auctionId;
   this.mediaType = 'banner';
   this.source = _bidSrc;
 
@@ -48,9 +50,19 @@ function Bid(statusCode, bidRequest) {
   this.getSize = function () {
     return this.width + 'x' + this.height;
   };
+
+  this.getIdentifiers = function () {
+    return {
+      src: this.source,
+      bidder: this.bidderCode,
+      bidId: this.requestId,
+      transactionId: this.transactionId,
+      auctionId: this.auctionId
+    }
+  }
 }
 
 // Bid factory function.
-export function createBid(statusCode, bidRequest) {
-  return new Bid(statusCode, bidRequest);
+export function createBid(statusCode, identifiers) {
+  return new Bid(statusCode, identifiers);
 }

package/src/native.js

@@ -1,5 +1,6 @@
-import { deepAccess, getBidRequest, getKeyByValue, insertHtmlIntoIframe, logError, triggerPixel } from './utils.js';
+import { deepAccess, getKeyByValue, insertHtmlIntoIframe, logError, triggerPixel } from './utils.js';
 import includes from 'core-js-pure/features/array/includes.js';
+import {auctionManager} from './auctionManager.js';
 
 const CONSTANTS = require('./constants.json');
 
@@ -35,6 +36,16 @@ export function processNativeAdUnitParams(params) {
   return params;
 }
 
+export function decorateAdUnitsWithNativeParams(adUnits) {
+  adUnits.forEach(adUnit => {
+    const nativeParams =
+      adUnit.nativeParams || deepAccess(adUnit, 'mediaTypes.native');
+    if (nativeParams) {
+      adUnit.nativeParams = processNativeAdUnitParams(nativeParams);
+    }
+  });
+}
+
 /**
  * Check if the native type specified in the adUnit is supported by Prebid.
  */
@@ -68,16 +79,13 @@ export const hasNonNativeBidder = adUnit =>
  * @param {BidRequest[]} bidRequests All bid requests for an auction
  * @return {Boolean} If object is valid
  */
-export function nativeBidIsValid(bid, bidRequests) {
-  const bidRequest = getBidRequest(bid.requestId, bidRequests);
-  if (!bidRequest) { return false; }
-
+export function nativeBidIsValid(bid, {index = auctionManager.index} = {}) {
   // all native bid responses must define a landing page url
   if (!deepAccess(bid, 'native.clickUrl')) {
     return false;
   }
 
-  const requestedAssets = bidRequest.nativeParams;
+  const requestedAssets = index.getAdUnit(bid).nativeParams;
   if (!requestedAssets) {
     return true;
   }
@@ -139,21 +147,21 @@ export function fireNativeTrackers(message, adObject) {
  * @param {Object} bid
  * @return {Object} targeting
  */
-export function getNativeTargeting(bid, bidReq) {
+export function getNativeTargeting(bid, {index = auctionManager.index} = {}) {
   let keyValues = {};
-
-  if (deepAccess(bidReq, 'nativeParams.rendererUrl')) {
-    bid['native']['rendererUrl'] = getAssetValue(bidReq.nativeParams['rendererUrl']);
-  } else if (deepAccess(bidReq, 'nativeParams.adTemplate')) {
-    bid['native']['adTemplate'] = getAssetValue(bidReq.nativeParams['adTemplate']);
+  const adUnit = index.getAdUnit(bid);
+  if (deepAccess(adUnit, 'nativeParams.rendererUrl')) {
+    bid['native']['rendererUrl'] = getAssetValue(adUnit.nativeParams['rendererUrl']);
+  } else if (deepAccess(adUnit, 'nativeParams.adTemplate')) {
+    bid['native']['adTemplate'] = getAssetValue(adUnit.nativeParams['adTemplate']);
   }
 
   const globalSendTargetingKeys = deepAccess(
-    bidReq,
+    adUnit,
     `nativeParams.sendTargetingKeys`
   ) !== false;
 
-  const nativeKeys = getNativeKeys(bidReq);
+  const nativeKeys = getNativeKeys(adUnit);
 
   const flatBidNativeKeys = { ...bid.native, ...bid.native.ext };
   delete flatBidNativeKeys.ext;
@@ -166,9 +174,9 @@ export function getNativeTargeting(bid, bidReq) {
       return;
     }
 
-    let sendPlaceholder = deepAccess(bidReq, `nativeParams.${asset}.sendId`);
+    let sendPlaceholder = deepAccess(adUnit, `nativeParams.${asset}.sendId`);
     if (typeof sendPlaceholder !== 'boolean') {
-      sendPlaceholder = deepAccess(bidReq, `nativeParams.ext.${asset}.sendId`);
+      sendPlaceholder = deepAccess(adUnit, `nativeParams.ext.${asset}.sendId`);
     }
 
     if (sendPlaceholder) {
@@ -176,9 +184,9 @@ export function getNativeTargeting(bid, bidReq) {
       value = placeholder;
     }
 
-    let assetSendTargetingKeys = deepAccess(bidReq, `nativeParams.${asset}.sendTargetingKeys`)
+    let assetSendTargetingKeys = deepAccess(adUnit, `nativeParams.${asset}.sendTargetingKeys`)
     if (typeof assetSendTargetingKeys !== 'boolean') {
-      assetSendTargetingKeys = deepAccess(bidReq, `nativeParams.ext.${asset}.sendTargetingKeys`);
+      assetSendTargetingKeys = deepAccess(adUnit, `nativeParams.ext.${asset}.sendTargetingKeys`);
     }
 
     const sendTargeting = typeof assetSendTargetingKeys === 'boolean' ? assetSendTargetingKeys : globalSendTargetingKeys;
@@ -259,11 +267,11 @@ function getAssetValue(value) {
   return value;
 }
 
-function getNativeKeys(bidReq) {
+function getNativeKeys(adUnit) {
   const extraNativeKeys = {}
 
-  if (deepAccess(bidReq, 'nativeParams.ext')) {
-    Object.keys(bidReq.nativeParams.ext).forEach(extKey => {
+  if (deepAccess(adUnit, 'nativeParams.ext')) {
+    Object.keys(adUnit.nativeParams.ext).forEach(extKey => {
       extraNativeKeys[extKey] = `hb_native_${extKey}`;
     })
   }

package/src/prebid.js

@@ -19,6 +19,7 @@ import { adunitCounter } from './adUnits.js';
 import { executeRenderer, isRendererRequired } from './Renderer.js';
 import { createBid } from './bidfactory.js';
 import { storageCallbacks } from './storageManager.js';
+import { emitAdRenderSucceeded, emitAdRenderFail } from './adRendering.js';
 
 const $$PREBID_GLOBAL$$ = getGlobal();
 const CONSTANTS = require('./constants.json');
@@ -27,7 +28,7 @@ const events = require('./events.js');
 const { triggerUserSyncs } = userSync;
 
 /* private variables */
-const { ADD_AD_UNITS, BID_WON, REQUEST_BIDS, SET_TARGETING, AD_RENDER_FAILED, AD_RENDER_SUCCEEDED, STALE_RENDER } = CONSTANTS.EVENTS;
+const { ADD_AD_UNITS, BID_WON, REQUEST_BIDS, SET_TARGETING, STALE_RENDER } = CONSTANTS.EVENTS;
 const { PREVENT_WRITING_ON_MAIN_DOCUMENT, NO_AD, EXCEPTION, CANNOT_FIND_AD, MISSING_DOC_OR_ADID } = CONSTANTS.AD_RENDER_FAILED_REASON;
 
 const eventValidators = {
@@ -146,7 +147,7 @@ function validateNativeMediaType(adUnit) {
 function validateAdUnitPos(adUnit, mediaType) {
   let pos = deepAccess(adUnit, `mediaTypes.${mediaType}.pos`);
 
-  if (!pos || !isNumber(pos) || !isFinite(pos)) {
+  if (!isNumber(pos) || isNaN(pos) || !isFinite(pos)) {
     let warning = `Value of property 'pos' on ad unit ${adUnit.code} should be of type: Number`;
 
     logWarn(warning);
@@ -385,23 +386,6 @@ $$PREBID_GLOBAL$$.setTargetingForAst = function (adUnitCodes) {
   events.emit(SET_TARGETING, targeting.getAllTargeting());
 };
 
-function emitAdRenderFail({ reason, message, bid, id }) {
-  const data = { reason, message };
-  if (bid) data.bid = bid;
-  if (id) data.adId = id;
-
-  logError(message);
-  events.emit(AD_RENDER_FAILED, data);
-}
-
-function emitAdRenderSucceeded({ doc, bid, id }) {
-  const data = { doc };
-  if (bid) data.bid = bid;
-  if (id) data.adId = id;
-
-  events.emit(AD_RENDER_SUCCEEDED, data);
-}
-
 /**
  * This function will check for presence of given node in given parent. If not present - will inject it.
  * @param {Node} node node, whose existance is in question

package/src/secureCreatives.js

@@ -4,22 +4,47 @@
  */
 
 import events from './events.js';
-import { fireNativeTrackers, getAssetMessage, getAllAssetsMessage } from './native.js';
+import {fireNativeTrackers, getAllAssetsMessage, getAssetMessage} from './native.js';
 import constants from './constants.json';
-import { logWarn, replaceAuctionPrice, deepAccess, isGptPubadsDefined, isApnGetTagDefined } from './utils.js';
-import { auctionManager } from './auctionManager.js';
+import {deepAccess, isApnGetTagDefined, isGptPubadsDefined, logError, logWarn, replaceAuctionPrice} from './utils.js';
+import {auctionManager} from './auctionManager.js';
 import find from 'core-js-pure/features/array/find.js';
-import { isRendererRequired, executeRenderer } from './Renderer.js';
+import {executeRenderer, isRendererRequired} from './Renderer.js';
 import includes from 'core-js-pure/features/array/includes.js';
-import { config } from './config.js';
+import {config} from './config.js';
+import {emitAdRenderFail, emitAdRenderSucceeded} from './adRendering.js';
 
 const BID_WON = constants.EVENTS.BID_WON;
 const STALE_RENDER = constants.EVENTS.STALE_RENDER;
 
+const HANDLER_MAP = {
+  'Prebid Request': handleRenderRequest,
+  'Prebid Native': handleNativeRequest,
+  'Prebid Event': handleEventRequest,
+}
+
 export function listenMessagesFromCreative() {
   window.addEventListener('message', receiveMessage, false);
 }
 
+export function getReplier(ev) {
+  if (ev.origin == null && ev.ports.length === 0) {
+    return function () {
+      const msg = 'Cannot post message to a frame with null origin. Please update creatives to use MessageChannel, see https://github.com/prebid/Prebid.js/issues/7870'
+      logError(msg)
+      throw new Error(msg);
+    }
+  } else if (ev.ports.length > 0) {
+    return function (message) {
+      ev.ports[0].postMessage(JSON.stringify(message));
+    }
+  } else {
+    return function (message) {
+      ev.source.postMessage(JSON.stringify(message), ev.origin);
+    }
+  }
+}
+
 export function receiveMessage(ev) {
   var key = ev.message ? 'message' : 'data';
   var data = {};
@@ -29,70 +54,128 @@ export function receiveMessage(ev) {
     return;
   }
 
-  if (data && data.adId) {
+  if (data && data.adId && data.message) {
     const adObject = find(auctionManager.getBidsReceived(), function (bid) {
       return bid.adId === data.adId;
     });
+    if (HANDLER_MAP.hasOwnProperty(data.message)) {
+      HANDLER_MAP[data.message](getReplier(ev), data, adObject);
+    }
+  }
+}
 
-    if (adObject && data.message === 'Prebid Request') {
-      if (adObject.status === constants.BID_STATUS.RENDERED) {
-        logWarn(`Ad id ${adObject.adId} has been rendered before`);
-        events.emit(STALE_RENDER, adObject);
-        if (deepAccess(config.getConfig('auctionOptions'), 'suppressStaleRender')) {
-          return;
-        }
-      }
+function handleRenderRequest(reply, data, adObject) {
+  if (adObject == null) {
+    emitAdRenderFail({
+      reason: constants.AD_RENDER_FAILED_REASON.CANNOT_FIND_AD,
+      message: `Cannot find ad '${data.adId}' for cross-origin render request`,
+      id: data.adId
+    });
+    return;
+  }
+  if (adObject.status === constants.BID_STATUS.RENDERED) {
+    logWarn(`Ad id ${adObject.adId} has been rendered before`);
+    events.emit(STALE_RENDER, adObject);
+    if (deepAccess(config.getConfig('auctionOptions'), 'suppressStaleRender')) {
+      return;
+    }
+  }
 
-      _sendAdToCreative(adObject, ev);
+  try {
+    _sendAdToCreative(adObject, reply);
+  } catch (e) {
+    emitAdRenderFail({
+      reason: constants.AD_RENDER_FAILED_REASON.EXCEPTION,
+      message: e.message,
+      id: data.adId,
+      bid: adObject
+    });
+    return;
+  }
 
-      // save winning bids
-      auctionManager.addWinningBid(adObject);
+  // save winning bids
+  auctionManager.addWinningBid(adObject);
 
-      events.emit(BID_WON, adObject);
-    }
+  events.emit(BID_WON, adObject);
+}
 
-    // handle this script from native template in an ad server
-    // window.parent.postMessage(JSON.stringify({
-    //   message: 'Prebid Native',
-    //   adId: '%%PATTERN:hb_adid%%'
-    // }), '*');
-    if (adObject && data.message === 'Prebid Native') {
-      if (data.action === 'assetRequest') {
-        const message = getAssetMessage(data, adObject);
-        ev.source.postMessage(JSON.stringify(message), ev.origin);
-      } else if (data.action === 'allAssetRequest') {
-        const message = getAllAssetsMessage(data, adObject);
-        ev.source.postMessage(JSON.stringify(message), ev.origin);
-      } else if (data.action === 'resizeNativeHeight') {
-        adObject.height = data.height;
-        adObject.width = data.width;
-        resizeRemoteCreative(adObject);
-      } else {
-        const trackerType = fireNativeTrackers(data, adObject);
-        if (trackerType === 'click') { return; }
-
-        auctionManager.addWinningBid(adObject);
-        events.emit(BID_WON, adObject);
+function handleNativeRequest(reply, data, adObject) {
+  // handle this script from native template in an ad server
+  // window.parent.postMessage(JSON.stringify({
+  //   message: 'Prebid Native',
+  //   adId: '%%PATTERN:hb_adid%%'
+  // }), '*');
+  if (adObject == null) {
+    logError(`Cannot find ad '${data.adId}' for x-origin event request`);
+    return;
+  }
+  switch (data.action) {
+    case 'assetRequest':
+      reply(getAssetMessage(data, adObject));
+      break;
+    case 'allAssetRequest':
+      reply(getAllAssetsMessage(data, adObject));
+      break;
+    case 'resizeNativeHeight':
+      adObject.height = data.height;
+      adObject.width = data.width;
+      resizeRemoteCreative(adObject);
+      break;
+    default:
+      const trackerType = fireNativeTrackers(data, adObject);
+      if (trackerType === 'click') {
+        return;
       }
-    }
+      auctionManager.addWinningBid(adObject);
+      events.emit(BID_WON, adObject);
   }
 }
 
-export function _sendAdToCreative(adObject, ev) {
+function handleEventRequest(reply, data, adObject) {
+  if (adObject == null) {
+    logError(`Cannot find ad '${data.adId}' for x-origin event request`);
+    return;
+  }
+  if (adObject.status !== constants.BID_STATUS.RENDERED) {
+    logWarn(`Received x-origin event request without corresponding render request for ad '${data.adId}'`);
+    return;
+  }
+  switch (data.event) {
+    case constants.EVENTS.AD_RENDER_FAILED:
+      emitAdRenderFail({
+        bid: adObject,
+        id: data.adId,
+        reason: data.info.reason,
+        message: data.info.message
+      });
+      break;
+    case constants.EVENTS.AD_RENDER_SUCCEEDED:
+      emitAdRenderSucceeded({
+        doc: null,
+        bid: adObject,
+        id: data.adId
+      });
+      break;
+    default:
+      logError(`Received x-origin event request for unsupported event: '${data.event}' (adId: '${data.adId}')`)
+  }
+}
+
+export function _sendAdToCreative(adObject, reply) {
   const { adId, ad, adUrl, width, height, renderer, cpm } = adObject;
   // rendering for outstream safeframe
   if (isRendererRequired(renderer)) {
     executeRenderer(renderer, adObject);
   } else if (adId) {
     resizeRemoteCreative(adObject);
-    ev.source.postMessage(JSON.stringify({
+    reply({
       message: 'Prebid Response',
       ad: replaceAuctionPrice(ad, cpm),
       adUrl: replaceAuctionPrice(adUrl, cpm),
       adId,
       width,
       height
-    }), ev.origin);
+    });
   }
 }
 

package/src/targeting.js

@@ -1,6 +1,6 @@
 import {
   uniques, isGptPubadsDefined, getHighestCpm, getOldestHighestCpmBid, groupBy, isAdUnitCodeMatchingSlot, timestamp,
-  deepAccess, deepClone, logError, logWarn, logInfo, isFn, isArray, logMessage, isStr, isAllowZeroCpmBidsEnabled
+  deepAccess, deepClone, logError, logWarn, logInfo, isFn, isArray, logMessage, isStr,
 } from './utils.js';
 import { config } from './config.js';
 import { NATIVE_TARGETING_KEYS } from './native.js';
@@ -8,6 +8,7 @@ import { auctionManager } from './auctionManager.js';
 import { sizeSupported } from './sizeMapping.js';
 import { ADPOD } from './mediaTypes.js';
 import { hook } from './hook.js';
+import { bidderSettings } from './bidderSettings.js';
 import includes from 'core-js-pure/features/array/includes.js';
 import find from 'core-js-pure/features/array/find.js';
 
@@ -437,7 +438,15 @@ export function newTargeting(auctionManager) {
     let bidsReceived = auctionManager.getBidsReceived();
 
     if (!config.getConfig('useBidCache')) {
+      // don't use bid cache (i.e. filter out bids not in the latest auction)
       bidsReceived = bidsReceived.filter(bid => latestAuctionForAdUnit[bid.adUnitCode] === bid.auctionId)
+    } else {
+      // if custom bid cache filter function exists, run for each bid from
+      // previous auctions. If it returns true, include bid in bid pool
+      const filterFunction = config.getConfig('bidCacheFilterFunction');
+      if (typeof filterFunction === 'function') {
+        bidsReceived = bidsReceived.filter(bid => latestAuctionForAdUnit[bid.adUnitCode] === bid.auctionId || !!filterFunction(bid))
+      }
     }
 
     bidsReceived = bidsReceived
@@ -459,7 +468,7 @@ export function newTargeting(auctionManager) {
     const adUnitCodes = getAdUnitCodes(adUnitCode);
     return bidsReceived
       .filter(bid => includes(adUnitCodes, bid.adUnitCode))
-      .filter(bid => (isAllowZeroCpmBidsEnabled(bid.bidderCode)) ? bid.cpm >= 0 : bid.cpm > 0)
+      .filter(bid => (bidderSettings.get(bid.bidderCode, 'allowZeroCpmBids') === true) ? bid.cpm >= 0 : bid.cpm > 0)
       .map(bid => bid.adUnitCode)
       .filter(uniques)
       .map(adUnitCode => bidsReceived

package/src/utils.js

@@ -1,6 +1,5 @@
 /* eslint-disable no-console */
 import { config } from './config.js';
-import { getGlobal } from './prebidGlobal.js';
 import clone from 'just-clone';
 import find from 'core-js-pure/features/array/find.js';
 import includes from 'core-js-pure/features/array/includes.js';
@@ -22,7 +21,17 @@ let consoleLogExists = Boolean(consoleExists && window.console.log);
 let consoleInfoExists = Boolean(consoleExists && window.console.info);
 let consoleWarnExists = Boolean(consoleExists && window.console.warn);
 let consoleErrorExists = Boolean(consoleExists && window.console.error);
-var events = require('./events.js');
+
+const emitEvent = (function () {
+  // lazy load events to avoid circular import
+  let ev;
+  return function() {
+    if (ev == null) {
+      ev = require('./events.js');
+    }
+    return ev.emit.apply(ev, arguments);
+  }
+})();
 
 // this allows stubbing of utility functions that are used internally by other utility functions
 export const internal = {
@@ -265,14 +274,14 @@ export function logWarn() {
   if (debugTurnedOn() && consoleWarnExists) {
     console.warn.apply(console, decorateLog(arguments, 'WARNING:'));
   }
-  events.emit(CONSTANTS.EVENTS.AUCTION_DEBUG, {type: 'WARNING', arguments: arguments});
+  emitEvent(CONSTANTS.EVENTS.AUCTION_DEBUG, {type: 'WARNING', arguments: arguments});
 }
 
 export function logError() {
   if (debugTurnedOn() && consoleErrorExists) {
     console.error.apply(console, decorateLog(arguments, 'ERROR:'));
   }
-  events.emit(CONSTANTS.EVENTS.AUCTION_DEBUG, {type: 'ERROR', arguments: arguments});
+  emitEvent(CONSTANTS.EVENTS.AUCTION_DEBUG, {type: 'ERROR', arguments: arguments});
 }
 
 function decorateLog(args, prefix) {
@@ -885,12 +894,6 @@ export function isValidMediaTypes(mediaTypes) {
   return true;
 }
 
-export function getBidderRequest(bidRequests, bidder, adUnitCode) {
-  return find(bidRequests, request => {
-    return request.bids
-      .filter(bid => bid.bidder === bidder && bid.adUnitCode === adUnitCode).length > 0;
-  }) || { start: null, auctionId: null };
-}
 /**
  * Returns user configured bidder params from adunit
  * @param {Object} adUnits
@@ -1270,7 +1273,7 @@ export function mergeDeep(target, ...sources) {
         mergeDeep(target[key], source[key]);
       } else if (isArray(source[key])) {
         if (!target[key]) {
-          Object.assign(target, { [key]: source[key] });
+          Object.assign(target, { [key]: [...source[key]] });
         } else if (isArray(target[key])) {
           source[key].forEach(obj => {
             let addItFlag = 1;
@@ -1333,9 +1336,3 @@ export function cyrb53Hash(str, seed = 0) {
   h2 = imul(h2 ^ (h2 >>> 16), 2246822507) ^ imul(h1 ^ (h1 >>> 13), 3266489909);
   return (4294967296 * (2097151 & h2) + (h1 >>> 0)).toString();
 }
-
-export function isAllowZeroCpmBidsEnabled(bidderCode) {
-  const bidderSettings = getGlobal().bidderSettings;
-  return ((bidderSettings[bidderCode] && bidderSettings[bidderCode].allowZeroCpmBids === true) ||
-   (bidderSettings.standard && bidderSettings.standard.allowZeroCpmBids === true));
-}

package/src/video.js

@@ -1,8 +1,9 @@
 import adapterManager from './adapterManager.js';
-import { getBidRequest, deepAccess, logError } from './utils.js';
+import { deepAccess, logError } from './utils.js';
 import { config } from '../src/config.js';
 import includes from 'core-js-pure/features/array/includes.js';
 import { hook } from './hook.js';
+import {auctionManager} from './auctionManager.js';
 
 const VIDEO_MEDIA_TYPE = 'video';
 export const OUTSTREAM = 'outstream';
@@ -28,23 +29,21 @@ export const hasNonVideoBidder = adUnit =>
 /**
  * Validate that the assets required for video context are present on the bid
  * @param {VideoBid} bid Video bid to validate
- * @param {BidRequest[]} bidRequests All bid requests for an auction
+ * @param index
  * @return {Boolean} If object is valid
  */
-export function isValidVideoBid(bid, bidRequests) {
-  const bidRequest = getBidRequest(bid.requestId, bidRequests);
-
-  const videoMediaType =
-    bidRequest && deepAccess(bidRequest, 'mediaTypes.video');
+export function isValidVideoBid(bid, {index = auctionManager.index} = {}) {
+  const videoMediaType = deepAccess(index.getMediaTypes(bid), 'video');
   const context = videoMediaType && deepAccess(videoMediaType, 'context');
+  const adUnit = index.getAdUnit(bid);
 
   // if context not defined assume default 'instream' for video bids
   // instream bids require a vast url or vast xml content
-  return checkVideoBidSetup(bid, bidRequest, videoMediaType, context);
+  return checkVideoBidSetup(bid, adUnit, videoMediaType, context);
 }
 
-export const checkVideoBidSetup = hook('sync', function(bid, bidRequest, videoMediaType, context) {
-  if (!bidRequest || (videoMediaType && context !== OUTSTREAM)) {
+export const checkVideoBidSetup = hook('sync', function(bid, adUnit, videoMediaType, context) {
+  if (videoMediaType && context !== OUTSTREAM) {
     // xml-only video bids require a prebid cache url
     if (!config.getConfig('cache.url') && bid.vastXml && !bid.vastUrl) {
       logError(`
@@ -59,7 +58,7 @@ export const checkVideoBidSetup = hook('sync', function(bid, bidRequest, videoMe
 
   // outstream bids require a renderer on the bid or pub-defined on adunit
   if (context === OUTSTREAM) {
-    return !!(bid.renderer || bidRequest.renderer || videoMediaType.renderer);
+    return !!(bid.renderer || (adUnit && adUnit.renderer) || videoMediaType.renderer);
   }
 
   return true;

package/src/videoCache.js

@@ -11,7 +11,7 @@
 
 import { ajax } from './ajax.js';
 import { config } from './config.js';
-import { isPlainObject } from './utils.js';
+import {auctionManager} from './auctionManager.js';
 
 /**
  * @typedef {object} CacheableUrlBid
@@ -58,9 +58,11 @@ function wrapURI(uri, impUrl) {
  * the bid can't be converted cleanly.
  *
  * @param {CacheableBid} bid
+ * @param index
  */
-function toStorageRequest(bid) {
+function toStorageRequest(bid, {index = auctionManager.index} = {}) {
   const vastValue = bid.vastXml ? bid.vastXml : wrapURI(bid.vastUrl, bid.vastImpUrl);
+  const auction = index.getAuction(bid);
 
   let payload = {
     type: 'xml',
@@ -72,10 +74,10 @@ function toStorageRequest(bid) {
     payload.bidder = bid.bidder;
     payload.bidid = bid.requestId;
     payload.aid = bid.auctionId;
-    // function has a thisArg set to bidderRequest for accessing the auctionStart
-    if (isPlainObject(this) && this.hasOwnProperty('auctionStart')) {
-      payload.timestamp = this.auctionStart;
-    }
+  }
+
+  if (auction != null) {
+    payload.timestamp = auction.getAuctionStart();
   }
 
   if (typeof bid.customCacheKey === 'string' && bid.customCacheKey !== '') {
@@ -132,12 +134,11 @@ function shimStorageCallback(done) {
  *
  * @param {CacheableBid[]} bids A list of bid objects which should be cached.
  * @param {videoCacheStoreCallback} [done] An optional callback which should be executed after
- * @param {BidderRequest} [bidderRequest]
  * the data has been stored in the cache.
  */
-export function store(bids, done, bidderRequest) {
+export function store(bids, done) {
   const requestData = {
-    puts: bids.map(toStorageRequest, bidderRequest)
+    puts: bids.map(toStorageRequest)
   };
 
   ajax(config.getConfig('cache.url'), shimStorageCallback(done), JSON.stringify(requestData), {

package/test/fixtures/fixtures.js

@@ -1231,7 +1231,7 @@ export function getCurrencyRates() {
   };
 }
 
-export function createBidReceived({bidder, cpm, auctionId, responseTimestamp, adUnitCode, adId, status, ttl, requestId}) {
+export function createBidReceived({bidder, cpm, auctionId, responseTimestamp, adUnitCode, adId, status, ttl, requestId, mediaType}) {
   let bid = {
     'bidderCode': bidder,
     'width': '300',
@@ -1259,6 +1259,7 @@ export function createBidReceived({bidder, cpm, auctionId, responseTimestamp, ad
       'hb_pb': cpm,
       'foobar': '300x250'
     }),
+    'mediaType': mediaType,
     'netRevenue': true,
     'currency': 'USD',
     'ttl': (!ttl) ? 300 : ttl