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

Fix https://github.com/brave/adblock-lists/issues/43 (not being blocked) #295

Merged
merged 1 commit into from Nov 25, 2019

Conversation

@ryanbr
Copy link
Collaborator

ryanbr commented Nov 24, 2019

Visiting https://3dnews.ru/982615 is show the market.yandex widget. With any of the Russian lists enabled in brave://adblock doesn't seem to work when blocking this. I mainly tested Adguard Russian list, but enabling the other Russian lists didn't remove it.

In Adguard; https://filters.adtidy.org/extension/chromium/filters/1.txt (spectiically on https://github.com/AdguardTeam/AdguardFilters/blob/master/RussianFilter/sections/specific.txt#L5635)

||aflt.market.yandex.$domain=3dnews.ru

The script we're blocking is https://aflt.market.yandex.ru/widget/script/api

Doesn't seem to work well in Brave as its not treating everything after the "." as a wildcard. This is a temp fix, but its a bug on our end

@pes10k
pes10k approved these changes Nov 25, 2019
@pes10k pes10k merged commit 293dc52 into brave:master Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.