added missing dfpAdServerVideo.js dfp vast tag generation uri compone… (

#2440)

* added missing dfpAdServerVideo.js dfp vast tag generation uri component encoding

* fixed buildDfpVideoUrl not returning empty targeting

* reverted last commits change because it broke display targeting

modules/dfpAdServerVideo.js

@@ -80,7 +80,7 @@ export default function buildDfpVideoUrl(options) {
   const derivedParams = {
     correlator: Date.now(),
     sz: parseSizesInput(adUnit.sizes).join('|'),
-    url: location.href,
+    url: encodeURIComponent(location.href),
   };
   const encodedCustomParams = getCustParams(bid, options);
 
@@ -152,7 +152,7 @@ function getCustParams(bid, options) {
     adserverTargeting,
     { hb_uuid: bid && bid.videoCacheKey },
     // hb_uuid will be deprecated and replaced by hb_cache_id
-    {hb_cache_id: bid && bid.videoCacheKey},
+    { hb_cache_id: bid && bid.videoCacheKey },
     optCustParams,
   );
   return encodeURIComponent(formatQS(customParams));

src/targeting.js

@@ -22,7 +22,7 @@ export const isBidExpired = (bid) => (bid.responseTimestamp + bid.ttl * 1000 + T
 const isUnusedBid = (bid) => bid && ((bid.status && !includes([BID_TARGETING_SET, RENDERED], bid.status)) || !bid.status);
 
 // If two bids are found for same adUnitCode, we will use the latest one to take part in auction
-// This can happen in case of concurrent autions
+// This can happen in case of concurrent auctions
 export const getOldestBid = function(bid, i, arr) {
   let oldestBid = true;
   arr.forEach((val, j) => {