Remove additional markup from ad markup - just pass Content.Body through

adapters/consumable/consumable.go

@@ -218,13 +218,12 @@ func (a *ConsumableAdapter) MakeBids(
 		}
 
 		if decision.Pricing != nil && decision.Pricing.ClearPrice != nil {
-			_, consumableExt, _ := extractExtensions(*imp)
 
 			bid := openrtb.Bid{}
 			bid.ID = internalRequest.ID
 			bid.ImpID = impID
 			bid.Price = *decision.Pricing.ClearPrice
-			bid.AdM = retrieveAd(decision, consumableExt.UnitId, consumableExt.UnitName, a.clock.Now())
+			bid.AdM = retrieveAd(decision)
 			bid.W = imp.Banner.Format[0].W // TODO: Review to check if this is correct behaviour
 			bid.H = imp.Banner.Format[0].H
 			bid.CrID = strconv.FormatInt(decision.AdID, 10)

adapters/consumable/consumable/exemplary/simple-banner.json

@@ -105,7 +105,7 @@
             "id": "test-request-id",
             "impid": "test-imp-id",
             "price": 0.5,
-            "adm": "<script type=\"text/javascript\">document.write(\"<div id=\"fakeName42-42\">\");</script><blink>Remember this: https://www.google.com/search?q=blink+tag ?</blink><div style=\"position:absolute;left:0px;top:0px;visibility:hidden;\"><img src=\"http%3A%2F%2Flocalhost%3A8080%2Fshown\"></div><script type=\"text/javascript\">document.write(\"</div>\");</script><script type=\"text/javascript\">document.write(\"<div class=\"fakeName42\"></div>\");</script><script type=\"text/javascript\">document.write(\"<scr\"+\"ipt type=\"text/javascript\" src=\"https://yummy.consumable.com/42/fakeName42/widget/unit.js?cb=1451651415\" charset=\"utf-8\" async></scr\"+\"ipt>\");</script>",
+            "adm": "<blink>Remember this: https://www.google.com/search?q=blink+tag ?</blink>",
             "crid": "1234567890",
             "exp": 30,
             "w": 728,

adapters/consumable/consumable/supplemental/simple-banner-no-impressionUrl.json

@@ -119,7 +119,7 @@
             "crid": "1234567890",
             "impid": "test-no-impUrl-id",
             "price": 0.5,
-            "adm": "<script type=\"text/javascript\">document.write(\"<div id=\"the-answer-42\">\");</script><blink>Remember this: https://www.google.com/search?q=blink+tag ?</blink><script type=\"text/javascript\">document.write(\"</div>\");</script><script type=\"text/javascript\">document.write(\"<div class=\"the-answer\"></div>\");</script><script type=\"text/javascript\">document.write(\"<scr\"+\"ipt type=\"text/javascript\" src=\"https://yummy.consumable.com/42/the-answer/widget/unit.js?cb=1451651415\" charset=\"utf-8\" async></scr\"+\"ipt>\");</script>",
+            "adm": "<blink>Remember this: https://www.google.com/search?q=blink+tag ?</blink>",
             "exp": 30,
             "w": 300,
             "h": 250

adapters/consumable/retrieveAd.go

@@ -1,33 +1,10 @@
 package consumable
 
-import (
-	"strconv"
-	"time"
-)
+func retrieveAd(decision decision) string {
 
-func retrieveAd(decision decision, unitId int, unitName string, now time.Time) string {
-
-	oad := ""
 	if decision.Contents != nil && len(decision.Contents) > 0 {
-		// When testing against e.serverbid.com, Contents seems to have a fair bit of usersync code both as img and iframe
-		oad = decision.Contents[0].Body + createTrackPixelHtml(decision.ImpressionUrl)
+		return decision.Contents[0].Body
 	}
 
-	cb := strconv.FormatInt(now.Unix(), 10)
-	sUnitId := strconv.Itoa(unitId)
-
-	// Some observed systems omit unitName and still get bids, so we'll make sure we attempt something in that case
-	// we can remove this later if needed.  Note: This is an addition compared to prebid.js implementation
-	if len(unitName) == 0 {
-		unitName = "fakeName" + sUnitId
-	}
-
-	ad := "<script type=\"text/javascript\">document.write(\"<div id=\"" +
-		unitName + "-" + sUnitId + "\">\");</script>" + oad +
-		"<script type=\"text/javascript\">document.write(\"</div>\");</script>" +
-		"<script type=\"text/javascript\">document.write(\"<div class=\"" + unitName + "\"></div>\");</script>" +
-		"<script type=\"text/javascript\">document.write(\"<scr\"+\"ipt type=\"text/javascript\" src=\"https://yummy.consumable.com/" +
-		sUnitId + "/" + unitName + "/widget/unit.js?cb=" + cb + "\" charset=\"utf-8\" async></scr\"+\"ipt>\");</script>"
-
-	return ad
+	return ""
 }