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

"no-large-media: * true" doesn't block all videos #2946

Closed
BenFroh opened this issue Sep 1, 2017 · 3 comments
Closed

"no-large-media: * true" doesn't block all videos #2946

BenFroh opened this issue Sep 1, 2017 · 3 comments

Comments

@BenFroh
Copy link

BenFroh commented Sep 1, 2017

Describe the issue

I set the custom filter "no-large-media: * true" to prevent Stagefright exploits.
Sadly, on some sites aren't the videos blocked.

(But besides that it would be great to have an option to allow pictures and bock only videos / audios.)

One or more specific URLs where the issue occurs

http://www.webmfiles.org/demo-files/

Steps for anyone to reproduce the issue

  1. set the custom filter: no-large-media: * true
  2. go to the site: http://www.webmfiles.org/demo-files/
  3. click on the video at the bottom of the page.

Your settings

  • OS/version: Android 5.0.1
  • Browser/version: Firefox 56.0b7
  • uBlock Origin version: 1.14.0
Your filter lists

Default filter lists + German

Your custom filters (if any)

no-large-media: * true

@gorhill
Copy link
Owner

gorhill commented Sep 2, 2017

I could not reproduce your issue at first, I had toggled on "no-large-media" before going to the site. The video was properly blocked when I clicked on it, and the logger reported it was blocked because of "no-large-media".

However, I toggled off the switch, and I could play the video -- as expected. I toggled on the switch and I still could play the video -- unexpectedly. Does that resemble to what you did? There is an issue when toggling off the switch, this will causes the large media to no longer be blocked after toggling on the switch, as long as the user does not navigate to a new site.

@BenFroh
Copy link
Author

BenFroh commented Sep 2, 2017

Oh, I'm sorry... you are right.
I think before I toggled it off/on and stand on the same site.

Thank you for your help and your great extension

@BenFroh BenFroh closed this as completed Sep 2, 2017
@gorhill gorhill reopened this Sep 2, 2017
@gorhill
Copy link
Owner

gorhill commented Sep 2, 2017

As said, there is indeed an issue uBO, I just wanted confirmation for the repro steps.

The issue is that uBO should stop allowing large media elements if the user toggles back the switch on, without needing to navigate to another site or open a new tab for the same site.

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

2 participants