Recent vaft/video-swap-new update breaks script injection

[younesaassila]

In the meantime, please only use 1 Twitch adblocker at the same time.

[ketcham1009]

Also not working in Firefox

[pixeltris]

Fixed by pixeltris/TwitchAdSolutions@144bc99?w=1

[younesaassila]

Fixed by pixeltris/TwitchAdSolutions@144bc99?w=1

Thank you very much!

[pixeltris]

I had to break it again due to pixeltris/TwitchAdSolutions#277

I essentially changed it to if (window.Worker.toString().includes('twitch')) { return; } which is checked when the script loads rather than when DOM loads as background web pages would load workers before the DOM.

You'd need to do something similar in https://github.com/younesaassila/ttv-lol-pro/blob/v2/src/page/page.ts

[younesaassila]

@pixeltris No worries, any specific reason as to why check for "twitch" specifically? Is this what AdGuard uses?
I'm dumb, I thought it would test against the Twitch worker file which has a high probability of containing the word "twitch". Of course the built-in object wouldn't ><'

[pixeltris]

Yea in this case it's checking the string representation of the Worker constructor/function for twitch as that text is inside the Worker implementation of vaft, video-swap-new, AdGuard Extra, and TTV LOL PRO

I preferred AdGuard Extra's way of doing it by comparing the window Worker to an iframe Worker but under Firefox when opening streams in a new tab this lets ads leak through as workers can be created before the check