Videos on nhl.com are not played while ETP is set to STRICT
Categories
(Web Compatibility :: Desktop, defect, P3)
Tracking
(Not tracked)
People
(Reporter: sergiu, Unassigned)
References
(Blocks 2 open bugs, )
Details
(Whiteboard: [tp-content][tp-ads][tp-yellowlist-passive][tp-shim-complex][tp-embedded-media])
User Story
gstatic.com googleapis.com adsafeprotected.com
Reporter | ||
Comment 1•6 years ago
|
||
Reporter | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Comment 2•5 years ago
|
||
In strict mode, I see this in the console:
TypeError: can't access property "publish", this.dispatcher is undefined index.js.gz:40:6346
onerror https://www-league.nhlstatic.com/nhl.com/sections/video/builds/6042dcbaf08f9f019e2af3cdbf3bddd48df62e13_1581528057/scripts/index.js.gz:40
It's failing here, hitting b.onerror
because the script fails to load and triggers an error:
function h() {
var a = this,
b = document.createElement('script'),
c = _.get(a.config, 'ias.imaUrl');
return c ? (b.src = c, b.onload = function () {
i.call(a)
}, b.onerror = function (a) {
this.dispatcher.publish(AdEvents.AD_PLUGIN_ERROR)
}, b.type = 'text/javascript', void document.getElementsByTagName('body') [0].appendChild(b)) : (_.set(a.config, 'ias.iasImaAdapterEnabled', !1), !1)
}
That's because the script https://static.adsafeprotected.com/vans-adapter-google-ima.js
is being blocked, triggering the error callback.
If I prevent that error by "allowing" the request, but replacing its contents with a blank string, then the pre-video ads play and the video plays just fine. So we may be able to fix this just by acting as though the script loaded.
Comment 3•5 years ago
|
||
I just confirmed with urlclassifier.trackingSkipURLs
that the above spoof should indeed work, though we will also need to whitelist https://pubads.g.doubleclick.net/gampad/ads
.
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 5•5 years ago
|
||
After some testing with shims, it turns out that both the AdSafeProtected Google IMA adapter and Google Publisher Tags should be shimmed here for best effect. This allows an ad to still play when the user clicks on the video, so we may want to warn the user in advance.
Updated•4 years ago
|
Comment 6•4 years ago
|
||
This does not reproduce with ETP set to STANDARD.
Environment:
Operating System: Windows 10 PRO x64
Firefox Version: Firefox Nightly 86.0a1 (2021-01-20) (64-bit)
Comment 7•4 years ago
|
||
(In reply to Thomas Wisniewski [:twisniewski] from comment #5)
After some testing with shims, it turns out that both the AdSafeProtected Google IMA adapter and Google Publisher Tags should be shimmed here for best effect. This allows an ad to still play when the user clicks on the video, so we may want to warn the user in advance.
These two shims were added in bug 1637329. The Publisher Tags shim was enabled in bug 1693386.
The Ad Safe shim is enabled in Nightly only. Does the issue still happen with ETP Strict in Nightly?
Comment 8•4 years ago
|
||
Videos now also play on nhl.com in strict mode/private browsing as of bug 1705111, which will be shipping on desktop Firefox 89.
Updated•3 years ago
|
Description
•