brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.47k stars 2.26k forks source link

Exception rules in custom filters don't work. #5240

Closed scorpion1201 closed 4 years ago

scorpion1201 commented 5 years ago

Description

Exception rules in custom filters don't work.

Steps to Reproduce

  1. Install Brave Browser.
  2. Add a exception rules in a custom filter box on brave://adblock
  3. Visit the site for which the custom filter was added, be still blocked!

Actual result:

Exception rules in custom filters don't work.

Expected result:

Exception rules work well.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.66.99 Chromium: 75.0.3770.100 (Official Build) (64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}

Version/Channel Information:

Other Additional Information:

Miscellaneous Information:

bsclifton commented 5 years ago

@scorpion1201 do you have an example of the rule you tried?

cc: @ryanbr

scorpion1201 commented 5 years ago

@bsclifton

Yes, the exception rule to blocked filters on 'tv.kakao.com' is below. @@||t1.daumcdn.net/tiara/js/v1/tiara.min.js

It's not working!

ggirard07 commented 4 years ago

Same issue here with an exception rule... But as this is my first custom filter, I am not sure if this is actually an issue or a bad usage from myself...

ggirard07 commented 4 years ago

Wondering if this issue could be linked in any way to https://github.com/brave/adblock-lists/pull/45

bsclifton commented 4 years ago

cc: @ryanbr @pes10k

pes10k commented 4 years ago

thank you for the issue. Closing in favor of https://github.com/brave/brave-browser/issues/5440, which covers the same issue

efolio commented 3 years ago

Any news on this… I still have the issue…

pes10k commented 3 years ago

@efolio the live version of this issue is https://github.com/brave/brave-browser/issues/5440, but as good news, its coming up next in our queue, so we hope to have an answer soon