Getting current window frames protocol, not assuming pub domain (#64)

Merging since failing circleci issue is unrelated to this code change
prebid · Jun 28, 2019 · 8f47f4a · 8f47f4a
1 parent af1739e
commit 8f47f4a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/renderingManager.js b/src/renderingManager.js
@@ -66,10 +66,11 @@ export function newRenderingManager(win, environment) {
    * @param {string} pubUrl Url of publisher page
    */
   function renderCrossDomain(adId, pubAdServerDomain = '', pubUrl) {
+    let windowLocation = window.location;
     let parsedUrl = utils.parseUrl(pubUrl);
     let publisherDomain = parsedUrl.protocol + '://' + parsedUrl.host;
-    let adServerDomain = (pubAdServerDomain !== '') ? pubAdServerDomain : GOOGLE_IFRAME_HOSTNAME;
-    let fullAdServerDomain = parsedUrl.protocol + '://' + adServerDomain;
+    let adServerDomain = pubAdServerDomain || GOOGLE_IFRAME_HOSTNAME;
+    let fullAdServerDomain = windowLocation.protocol + '//' + adServerDomain;
 
     function renderAd(ev) {
       let key = ev.message ? 'message' : 'data';