Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inline script tag filtering - Concrete examples of usefulness #795

Closed
flyingzebras opened this issue Oct 7, 2015 · 35 comments
Closed

Inline script tag filtering - Concrete examples of usefulness #795

flyingzebras opened this issue Oct 7, 2015 · 35 comments

Comments

@flyingzebras
Copy link

Pornhub.com have Global Alexa Rank 65 at this writing.

Can you please add how to block the popunder for pornhub.com in https://github.com/gorhill/uBlock/wiki/Inline-script-tag-filtering#concrete-examples-of-usefulness ?

What do you think about adding https://adblockplus.org/forum/viewtopic.php?t=27067 as reference to the text: "http://dayt.se/forum/activity.php: dayt.se##script:contains(adBlockDetected) (see "Blocking FuckAdblock")" in https://github.com/gorhill/uBlock/wiki/Inline-script-tag-filtering#concrete-examples-of-usefulness ?

@gorhill
Copy link
Owner

gorhill commented Oct 7, 2015

Can you please add how to block the popunder for pornhub.com in https://github.com/gorhill/uBlock/wiki/Inline-script-tag-filtering#concrete-examples-of-usefulness ?

Interesting challenge, will see what can be done -- if something can be done with the new filter. Can you give me a specific URL on that site for which the issue occur for sure, I don't feel like hunting for a test case -- I just need one URL.

@flyingzebras
Copy link
Author

@gorhill Just click on any video on pornhub.com to trigger a popunder. In fact, this mechanism is used for all popunder in sites of the the whole Pornhub NETWORK (pornhub.com, redtube.com, etc)!

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

Does this work?

www.pornhub.com##script:contains(FastPopSessionRequestNumber)

@flyingzebras
Copy link
Author

@gorhill Amazing, that worked in uBlock Origin 1.2.0 with over 400k daily users on AMO at this writing! Congratulations to be the first developer to ever have blocked inline scripting AFAIK!

Can you please add ||pornhub.com##script:contains(FastPopSessionRequestNumber) to any uBlock filter and confirm?

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

Ok, added pornhub.com##script:contains(FastPopSessionRequestNumber) (leading double pipes are not a valid cosmetic filter syntax).

Even with this new filter, just a reminder this it is still a cat and mouse game between sites/blockers -- the new inline script tag filter is just one more tool on the blocker side.

@flyingzebras
Copy link
Author

@gorhill

Also, can you please write about this in

@flyingzebras
Copy link
Author

@gorhill Thank you.

Where did you add pornhub.com##script:contains(FastPopSessionRequestNumber)

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

can you please write about this in

Not sure I want to "advertise" this on EasyList forum, I wouldn't be surprised if the site's owners have people monitoring the latest solutions in there against their pop-under "feature".

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

@flyingzebras
Copy link
Author

@gorhill Thank you.

Is it ok if I give you code to block popunders for the rest of the Pornhub NETWORK in https://github.com/gorhill/uBlock/blob/master/assets/ublock/filters.txt ?

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

Is it ok if I give you code to block popunders for the rest of the Pornhub NETWORK

If you can confirm it works for all of them -- that will save me having to check.

@flyingzebras
Copy link
Author

@gorhill MindGeek domains using FastPopSessionRequestNumber

extremetube.com|gaytube.com|mofosex.com|pornhub.com|redtube.com|spankwire.com|thumbzilla.com|tube8.*|xtube.com|youporn.com|youporngay.com##script:contains(FastPopSessionRequestNumber)

I recommend that you add them to https://github.com/gorhill/uBlock/blob/master/assets/ublock/badware.txt per https://en.wikipedia.org/wiki/MindGeek#Malvertising. What do you think about it?

gorhill added a commit that referenced this issue Oct 9, 2015
@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

Added suggested filter. FYI, I had to edit your suggested filter:

  • | is not a proper domain separator for cosmetic filters, a comma , is what needs to be used.
  • I used a single entity specifier tube8.* in place of tube8.com, tube8.es, and tube8.fr.

@flyingzebras
Copy link
Author

@gorhill Peter Lowe ignored adding these to his List:

[Aftermath: Wrong filter list]

@gorhill
Copy link
Owner

gorhill commented Oct 9, 2015

I see most of them in Peter Lowe's list. To be clear, I do not intend to become a filter list maintainer -- whatever I put in uBlock's filter lists is completely at my own discretion, I do not intend to take requests to add stuff in these lists, these requests should be directed to known filter list maintainers -- except for filters based on syntax extension (like the new inlien script tag filter).

@flyingzebras
Copy link
Author

@gorhill Out of curiosity, why didn't you move extremetube.com,gaytube.com,keezmovies.com,mofosex.com,pornhub.com,redtube.com,spankwire.com,thumbzilla.com,tube8.*,xtube.com,youporn.com,youporngay.com##script:contains(FastPopSessionRequestNumber) to uBlock filters – Badware risks‎ (https://github.com/gorhill/uBlock/blob/master/assets/ublock/badware.txt). It matches the requirement documented in https://github.com/gorhill/uBlock/wiki/Badware-risks so it should be the appropriate file to put it in.

If you add the domains to https://github.com/gorhill/uBlock/wiki/Badware-risks then write the titles like this, "Pornhub NETWORK: Blank", to be able to keep track of what belongs to the Pornhub NETWORK and not. See https://github.com/lejenome/html5-video-everywhere/issues/41 for example.

@gorhill
Copy link
Owner

gorhill commented Oct 10, 2015

why didn't you move

Because this filter is to prevent spurious popunders, not warn of risk of badware ahead.

@Hrxn
Copy link

Hrxn commented Oct 11, 2015

Short question:
This list https://github.com/gorhill/uBlock/blob/master/assets/ublock/filters.txt
is always active, with default settings, isn't it?

Because this could be a bit misleading, as I just read these Examples.

Try without, then with the respective inline script tag filter:

I guess this won't work then for users with current Firefox and uBlock0 versions.

I also noticed that these filters don't work in Chrome/Chromium, but this is correct, according to the same page (caveats).
I didn't notice that first, maybe don't put it at the end of the page, so this doesn't get overlooked

@gorhill
Copy link
Owner

gorhill commented Oct 11, 2015

Good points, I fixed the doc.

@flyingzebras
Copy link
Author

@gorhill I guess it would be good to state in https://github.com/gorhill/uBlock/wiki/Badware-risks that "*.exe" files are the most common practice to spread malware files. What do you think?

@Hrxn
Copy link

Hrxn commented Oct 12, 2015

That is true, I guess.

But, in my humble opinion, that is fundamental common sense of using the Internet. The Badware warning is to remind users to be cautious, it's not an exhaustive guide of using the Web safely.

uBlock0 is not Internet Basics 101, so this is out of scope for this project.
As I said, just my humble opinion on the matter.

@flyingzebras
Copy link
Author

@gorhill pornhub.com popunders have been resurrected, despite that pornhub pages still use FastPopSessionRequestNumber. Can you fix this? Mabybe by modifying extremetube.com,gaytube.com,keezmovies.com,mofosex.com,pornhub.com,redtube.com,spankwire.com,thumbzilla.com,tube8.*,xtube.com,youporn.com,youporngay.com##script:contains(FastPopSessionRequestNumber) in ublock0/content/asset-viewer.html?url=assets/ublock/filters.txt

@gorhill
Copy link
Owner

gorhill commented Nov 12, 2015

I can't reproduce, works from here. Anything out of the ordinary with your environment?

@flyingzebras
Copy link
Author

@gorhill Yes, I upgraded from IceCat 31 to IceCat 38. Nothing more than that.

@flyingzebras
Copy link
Author

@gorhill However, the popunders are catched by strict blocking:

**
uBlock Origin has prevented the following page from loading:
http://ebocornac.com/fp.eng?id=7129f320-ce21-4b40-9d7e-37b1df8cf8f0&rand=95669&ver=asynch&time=-60&referrerUrl=&subId=&abr=true&res=1600x900&stdTime=60&fpe=1&curl=http%3A%2F%2Fwww.pornhub.com%2F&hosted=true

Because of the following filter

||ebocornac.com^
Found in: Peter Lowe’s Ad server list
**

Still annoying with the opened tab though.

@gorhill
Copy link
Owner

gorhill commented Nov 12, 2015

IceCat 38

What is the result of this test for IceCat 38?

@flyingzebras
Copy link
Author

@gorhill

Summary

Harness status: OK

Found 4 tests
4 Pass

@flyingzebras
Copy link
Author

@gorhill I downloaded and installed Trisquel 7 from trisquel.info and installed icecat with: apt-get update && apt-get install icecat. This is the absolutely safest way to make sure that one get the latest version of icecat as the maintainer of IceCat also is the lead developer of Trisquel.

Trisquel GNU/Linux can be used live from USB.

@flyingzebras
Copy link
Author

@gorhill IceCat 38.3.0 is based on Firefox ESR. You can download and try the latest FF ESR here: https://ftp.mozilla.org/pub/firefox/releases/latest-esr/

@gorhill
Copy link
Owner

gorhill commented Nov 14, 2015

Ok I installed and investigated. The reason is simply that FF 38 API does not support process message managers, which is required in the current implementation of inline script tag filtering.

@flyingzebras
Copy link
Author

@gorhill Thank you. I opened a new issue for that: #940

@flyingzebras
Copy link
Author

@gorhill You introduced the popunder filter type in uBO 1.4.0.

In assets/ublock/filters.txt, should extremetube.com,gaytube.com,keezmovies.com,mofosex.com,pornhub.com,redtube.com,spankwire.com,thumbzilla.com,tube8.*,xtube.com,youporn.com,youporngay.com##script:contains(FastPopSessionRequestNumber) be merged to |http://$popunder,third-party,domain=xhamster.com?

Also, https://github.com/gorhill/uBlock/wiki/popunder is empty. I need it as a reference, should I look elsewhere?

@gorhill
Copy link
Owner

gorhill commented Dec 13, 2015

I can't see any popunders anymore on these sites, I may have to remove the corresponding script:contains filter. If you can confirm there is no more popunders, I just verified three of them.

@flyingzebras
Copy link
Author

@gorhill FastPopSessionRequestNumber was removed from some sites including pornhub.com, but not from these sites: extremetube.com,gaytube.com,keezmovies.com,redtube.com,spankwire.com,tube8.*,youporn.com,youporngay.com##script:contains(FastPopSessionRequestNumber) However, keeping FastPopSessionRequestNumber on all sites as currently done ensures that you do not have to update the filter lists all the time.

The questions is, do the popunder filter type block FastPopSessionRequestNumber as well? Then extremetube.com,gaytube.com,keezmovies.com,mofosex.com,pornhub.com,redtube.com,spankwire.com,thumbzilla.com,tube8.*,xtube.com,youporn.com,youporngay.com##script:contains(FastPopSessionRequestNumber) should be merged to |http://$popunder,third-party,domain=xhamster.com.

@flyingzebras
Copy link
Author

@gorhill Can you please add redtube.com.br to https://github.com/gorhill/uBlock/blob/master/assets/ublock/filters.txt, a new redtube.com site for Brazil.

gorhill added a commit to uBlockOrigin/uAssets that referenced this issue Apr 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants