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

uBO mode #8478

Closed
2 of 3 tasks
pes10k opened this issue Mar 1, 2020 · 12 comments
Closed
2 of 3 tasks

uBO mode #8478

pes10k opened this issue Mar 1, 2020 · 12 comments
Assignees
Labels
closed/duplicate Issue has already been reported feature/shields The overall Shields feature in Brave. priority/P4 Planned work. We expect to get to it "soon".

Comments

@pes10k
Copy link
Contributor

pes10k commented Mar 1, 2020

Some users have asked for a uBO mode, Brave applies all the same protections as uBO, and can enable them with a single button / toggle.

Right now Brave has all the same capabilities as uBO, with the following excpetions:

@pes10k pes10k added feature/shields The overall Shields feature in Brave. priority/P4 Planned work. We expect to get to it "soon". labels Mar 1, 2020
@Yuki2718
Copy link

Hi, I had a question to your comment as it seemed contracting to the filter compatibility wiki so I tested some of the most important function not mentioned here on Brave Nightly 1.8.11 running on Win10 64 bit.

  1. CSS style injection is not supported, or at least its syntax is different from uBO's one. Not only it is necessary to get scroll functioning after hiding some sort of modal overlay, it can also be used to address some of anti-adblock example.

  2. Obviously not all scriptlets of uBO is supported, or at least don't work in the same way as uBO. For example, nostif doesn't work at all. It's expected as I see no such strings in here or here. Moreover, even supposedly-supported aopr doesn't work e.g. gigaho.com##+js(aopr, onload) doesn't work on the site.

  3. It seems $inline-script modifier doesn't work. No wonder given $csp is not supported according to the wiki, but not mentioned in the wiki.

  4. As stated in the wiki, $ping is not supported.

  5. As stated in the wiki, $popup is not supported.

  6. As stated in the wiki, $generichide is not supported. I think this should be prioritized as well as the above scriptlets issue because the introduction of cosmetic filtering, even only for third-party, without support of those will cause a jump of rates that users come across anti-adblock messages.

It's not meant to be an exhaustive list, but there seems to be really many things to be done.

Finding: nice to see regex filter is now supported, please update the wiki.

It's personal opinion, but I think what should be done ASAP over all the above is to allow user to freely choose subscription filter from anywhere - uBO has been doing this from the beginning.

@pes10k
Copy link
Contributor Author

pes10k commented Mar 17, 2020

@Yuki2718 thank you for the comments and findings. We also want to get all these things done, but it's a matter of competing priorities. Some of these things are useful, but (in my best guess) just less useful than the other privacy-preserving things we're working on.

That being said, again, thank you for this. I'm going to share it internally and see where things stand.

@pes10k
Copy link
Contributor Author

pes10k commented Mar 17, 2020

One note, as @ryanbr reminded me, the $ping attribute isn't supported because Brave doesn't include ping-functionality (#7640)

@Yuki2718
Copy link

@pes10k Thank you for clarification and positive answer, I understand you have a priority for various features. My comment was meant to be potential FYI.

@antonok-edm
Copy link
Collaborator

@Yuki2718 Style injection should already be supported in exactly the same capacity as uBlock Origin.

2020-03-18-120621_936x513_scrot

Did you find a particular example of a rule whose syntax isn't supported?

@Yuki2718
Copy link

Yuki2718 commented Mar 19, 2020

Did you find a particular example of a rule whose syntax isn't supported?

Applying
google.*##.mn-dwn-arw:style(top: 27px !important;)
to Google search results in move of arrow icons as
google1
on uBO - for this screenshot I disabled uBO's other filters but keeping them enabled didn't change the results, only changed other appearance such that URL was shown in grey the same as the last screenshot below.

But the same rule on Brave Nightly 1.8.22
brave
didn't
google2
and changing the wildcard to "com" google.com##.mn-dwn-arw:style(top: 27px !important;) didn't make a difference.

@Yuki2718
Copy link

This is when uBO's other filters are enabled:
google3

@pes10k
Copy link
Contributor Author

pes10k commented Mar 19, 2020

@antonok-edm is this because custom rules are still going through the 1p vs 3p distinction (which i think is good for now)? That might explain what you're seeing @Yuki2718 (see https://brave.com/whats-brave-done-for-my-privacy-lately-episode2/ for more details)

@ryanbr
Copy link

ryanbr commented Mar 21, 2020

Supporting $generichide would be helpful (used by uBo and EL)

@RobertH1999
Copy link

Is it possible to implement something like this in the future?
ublock
@pes10k

Like an "This is for advanced users" more granule blocking

@pes10k
Copy link
Contributor Author

pes10k commented Aug 25, 2020

Hi @RobertH1999, we're interested in adding more debugging information like this, but I dont think we'll be able to get to it soon so I'll close this (this issue is more for matching uBO capabilities than debugging information / UI). I'm going to close this issue now, but I've added a separate issue to track here: #11426

@pes10k pes10k closed this as completed Aug 25, 2020
@pes10k pes10k reopened this Aug 25, 2020
@pes10k pes10k added the closed/duplicate Issue has already been reported label Aug 25, 2020
@pes10k
Copy link
Contributor Author

pes10k commented Aug 25, 2020

(marking this as duplicate since there is just one related open issue, already being tracked in #8559)

@pes10k pes10k closed this as completed Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported feature/shields The overall Shields feature in Brave. priority/P4 Planned work. We expect to get to it "soon".
Projects
None yet
Development

No branches or pull requests

5 participants