uazo / cromite

Cromite a Bromite fork with ad blocking and privacy enhancements; take back your browser!
https://www.cromite.org/
GNU General Public License v3.0
3.42k stars 74 forks source link

Dns CNAME support in Adblock Plus #247

Closed Embassy4016 closed 4 months ago

Embassy4016 commented 1 year ago

Preliminary checklist

Is your feature request related to privacy?

No

Is there a patch available for this feature somewhere?

Not that I am aware of. But brave browser is pretty effective at doing this.

Describe the solution you would like

Cromite fail at blocking cosmetic elements, both static and dynamic ads. I am not talking about blank space in the webpage but rather page redirects in certain websites. Brave manages to block these redirects. Don't have the technical knowledge to actually understand or give solution to the problem but you can check at d3ward

Describe alternatives you have considered

Integrate cosmetic filtering from brave browser, adblock plus or ublock origin

github59173 commented 1 year ago

This is a very important issue to fix. Ublock origin is the gold standard and should be followed as closely as possible to resolve this issue.

uazo commented 1 year ago

Cromite fail at blocking cosmetic elements

cosmetic filter rules (i.e. css injection) is already a supported feature.

Integrate cosmetic filtering from brave browser, adblock plus or ublock origin

this browser already integrates adblock plus

but rather page redirects in certain websites.

with reference to adblock, I don't know what you are referring to.

Embassy4016 commented 1 year ago

This is screenshot from cromite browser

Details

![signal-2023-08-31-15-04-35-729-1](https://github.com/uazo/cromite/assets/140226326/dcd15963-5770-4e29-a882-6cdd123f7c7b)

And this one is from brave browser

Details

![signal-2023-08-31-15-04-35-729](https://github.com/uazo/cromite/assets/140226326/7f85a49a-72a2-4665-967e-96e27bfad4f4)

Unlike brave browser, Cromite browser is unable to block embedded ads Link to the site- https://solidtorrents.to/torrents/the-shawshank-redemption-1994-720p-avc-rus-leonard-6c661/5bcb41427da3981cefcc26b6/

uazo commented 1 year ago

the problem here is the lack of support for CNAME in AdBlock Plus. It needs some work to add it. Thank you for the report and especially for the example.

Issue 1317943: HTTP/3 protocol upgrade for HTTPS DNS records perhaps useful to understand how to extract new information from the dns.

jakariyaa commented 1 year ago

Not very sure if it's related to this issue as well, but the anti adblock killer function doesn't work as intended. (I do have adblock and anti circumvention and snippets feature enabled)

With Cromite:

Details

![IMG_Cromite](https://github.com/uazo/cromite/assets/44645135/26deae37-81cf-435c-8299-32a3454c6272)

With Brave:

Details

![IMG_Brave](https://github.com/uazo/cromite/assets/44645135/51dbe3b5-f2c3-4a8d-8058-1f8b366bf29b)

My ad blocking settings in Cromite:

Details

![IMG_Settings](https://github.com/uazo/cromite/assets/44645135/a49c8b65-972b-4baa-a0f5-4e3c4181c51a)

Example website: https://ottprime.store/

Embassy4016 commented 1 year ago

Glad I could clarify the issue. The website I mentioned respects user choice when they decide to close the ads. Some websites just redirect the user to other spam/phishing sites irrespective of where the user clicks in the webpage. Hope you find a way to solve this issue. Thanks for all the work you are doing.

larena1 commented 11 months ago

I just switched from Brave to Cromite and am also seeing ads now similar to those that @Embassy4016 posted but on other sites. These don't show in Brave or Firefox with uBlock Origin.

Can you rip out AdBlock Plus for uBlock Origin? It's way more popular and better maintained also.

larena1 commented 11 months ago

Seems it's not the actual AdBlock Plus addon but an implementation in C++ embedded in the Chrome code. Are you sure you can fix that in a way that really works? Because right now it's missing way too many ads compared to uBlock and also the regular AdBlock Plus addon.

uazo commented 11 months ago

but an implementation in C++ embedded in the Chrome code

correct, the implementation is the official eyeo implementation.

Are you sure you can fix that in a way that really works?

I can give it a try.

uazo commented 11 months ago

Because right now it's missing way too many ads compared to uBlock and also the regular AdBlock Plus addon.

I would like someone to open a task with examples and how to reproduce the differences.

larena1 commented 11 months ago

Because right now it's missing way too many ads compared to uBlock and also the regular AdBlock Plus addon.

I would like someone to open a task with examples and how to reproduce the differences.

There are actually so many examples of what the current implementation won't catch

https://www.independent.co.uk/news/uk/gaza-daily-mirror-israel-natwest-hamas-b2437449.html https://www.huffpost.com/entry/geert-wilders-netherlands-election-win_n_655e6a37e4b0c91d8279840e

And lots more that can be added to the task

uazo commented 11 months ago

@larena1 thanks, but I do not see any ads in the links you have indicated.

larena1 commented 11 months ago

@larena1 thanks, but I do not see any ads in the links you have indicated.

Which lists do you have enabled and do you have any DNS based AdBlocker like PiHole or AdGuard in your home network?

uazo commented 11 months ago

Which lists do you have enabled

everything by default:

Details

![image](https://github.com/uazo/cromite/assets/29201891/532c0fa5-9f10-4b70-aeb7-1c16253124c3)

do you have any DNS based AdBlocker like PiHole or AdGuard in your home network?

No

uazo commented 11 months ago

If I deactivate adblock I see a banner at the bottom:

Details

![image](https://github.com/uazo/cromite/assets/29201891/a05e16c6-0e75-41e2-bbd3-2029c8554b04)

larena1 commented 11 months ago

Can you try refreshing the page while you're on it and see if that changes anything? And there's actually a whole lot more of ads on that page than just the thing at the bottom.

uazo commented 11 months ago

https://github.com/uazo/cromite/assets/29201891/5edcd757-f949-4cd5-9cb5-658aca831b61

larena1 commented 11 months ago

Did you try refreshing the page either from the menu or pull to refresh?

larena1 commented 11 months ago

https://github.com/uazo/cromite/assets/60823161/2f044cd7-5aa9-4265-9f6d-f13d0aa898f3

The bottom area will be empty first but when I refresh the page, it'll show all ads.

uazo commented 11 months ago

please check that the legacy adblock is active. unfortunately, that flag is unique for now and deactivation also includes deactivation of adblock plus.

larena1 commented 11 months ago

I disabled legacy AdBlock as it'd trigger the AdBlock detection on one of the sites I frequently visit. But when you say that also disabled the new AdBlock, then it might've also been that what triggered the detection?

Do you think it'd be easier to just use uBlock as an addon instead of the current implementation?

uazo commented 11 months ago

I disabled legacy AdBlock as it'd trigger the AdBlock detection on one of the sites I frequently visit But when you say that also disabled the new AdBlock, then it might've also been that what triggered the detection?

tracked by https://github.com/uazo/cromite/issues/251

Do you think it'd be easier to just use uBlock as an addon instead of the current implementation?

easy, no, it's not. it's simpler that eyeo develops its adblock for chromium and i use it. then whether it can be improved is another matter.

uazo commented 10 months ago

Issue 1151047: Expose DNS record CNAME aliases to the SubresourceFilter

larena1 commented 10 months ago

easy, no, it's not. it's simpler that eyeo develops its adblock for chromium and i use it. then whether it can be improved is another matter.

I just thought it might be possible because Kiwi uses the addon interface and Brave also (maybe even for AdBlock but I didn't check). And Vivaldi's built-in AdBlock also blocks ads just fine. Maybe you can look into the latter two's implementation?