serverless-dns / blocklists

An opinionated collection of blocklists for RethinkDNS.
https://rethinkdns.com/configure
Mozilla Public License 2.0
81 stars 24 forks source link

config: rmv Adguard combined disguised trackers #139

Closed SpencerIsGiddy closed 1 year ago

SpencerIsGiddy commented 1 year ago

I was doing some thinking and maybe adguard cname blocklist should not be with adguard sdns since there is a higher chance for false positives and you mentioned in the past that serverless dns already uncloaks cnames anyways. So it seems like a good idea to remove this list from sdns

ignoramous commented 1 year ago

Does leaving it in affect blocking adversely? If not, it is okay to leave it as-is, I feel.

SpencerIsGiddy commented 1 year ago

Does leaving it in affect blocking adversely? If not, it is okay to leave it as-is, I feel.

I have seen people say there are some false positives with the list. https://github.com/AdguardTeam/cname-trackers/issues/62

maybe we can switch it out for this https://github.com/AdguardTeam/cname-trackers/blob/master/combined_original_trackers.txt

ignoramous commented 1 year ago

maybe we can switch it out for this https://github.com/AdguardTeam/cname-trackers/blob/master/combined_original_trackers.txt

This one contains the cname + AdGuard orig list?

SpencerIsGiddy commented 1 year ago

maybe we can switch it out for this https://github.com/AdguardTeam/cname-trackers/blob/master/combined_original_trackers.txt

This one contains the cname + AdGuard orig list?

I think it just uses the cname list but this is probably the list meant for blocklists capable of scanning CNAME records unlike the current one which is very general

ignoramous commented 1 year ago

Hm, that list has like 10 to 20 entries only

ignoramous commented 1 year ago

Is the current commit good to be merged?

SpencerIsGiddy commented 1 year ago

Is the current commit good to be merged?

Sure!

ignoramous commented 1 year ago

Thanks (again).

SpencerIsGiddy commented 1 year ago

Thanks (again).

😁👍