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

Custom filter subscriptions (add by URL) #13

Closed
ameshkov opened this issue Aug 24, 2015 · 21 comments
Closed

Custom filter subscriptions (add by URL) #13

ameshkov opened this issue Aug 24, 2015 · 21 comments

Comments

@ameshkov
Copy link
Member

ameshkov commented Aug 24, 2015

#13 (comment)

@ameshkov ameshkov added this to the 2.3 milestone Aug 24, 2015
@TPS
Copy link
Contributor

TPS commented Aug 24, 2015

#12 could be enhanced by this issue's implementation including a "Submit for review to add to public directory" or some such.… ☺

@ameshkov
Copy link
Member Author

There's a difference between #12 and this one.

We serve all popular filter subscriptions (like EasyList and such) from our server:
http://adguard.com/en/filter-rules.html?id=101

This version of EasyList is transformed a bit to be 100% compatible with Adguard. We also validate every change to EasyList and fix some errors automatically.

One more bonus from this is that we don't need to download whole filter to check if it has changed.

@TPS
Copy link
Contributor

TPS commented Aug 24, 2015

I understand they're different. I'm saying that URLs submitted via this mechanism could benefit #12 if there's a checkbox (default off, probably, & with comment field → maybe a beta APK feature?) to submit URL to AdGuard devs to review for inclusion in later releases.

@ameshkov
Copy link
Member Author

Now I got it, thank you! Looks too complicated to me.

Maybe simpler solution? We can add a comment like this:
"If you want this subscription to be included to default list report it to us here (link to github or our website)."

@TPS
Copy link
Contributor

TPS commented Aug 24, 2015

That's, of course, simpler, but much less likely to produce results. Given volume of URLs custom-added & then submitted should be very low, submissions should be high-enough quality that the complication be worth the effort.…

@ameshkov
Copy link
Member Author

Depends on what result do we want. Normally we proxify & transform only known subscriptions which are currently supported.

@uBlock-user
Copy link

Just for clarification, will this include HOSTS file too ?

@ameshkov
Copy link
Member Author

ameshkov commented Sep 5, 2017

Just for clarification, will this include HOSTS file too ?

Nope, hosts will be supported as part of the "DNS" module.

@TPS
Copy link
Contributor

TPS commented Sep 5, 2017

Nope, hosts will be supported as part of the "DNS" module.

Even online HOSTS-file subscriptions?

@ameshkov
Copy link
Member Author

ameshkov commented Sep 5, 2017

@TPS I don't mean we won't do it:) What I am saying is that hosts-files integration should be implemented as a part of the DNS module of AG as hosts might be used not just for blocking something, but for redirection or for enforcing safe search in some search engines.

@uBlock-user
Copy link

uBlock-user commented Sep 8, 2017

You mean you will add it in the DNS section of the app ?

@ameshkov
Copy link
Member Author

ameshkov commented Sep 8, 2017

You mean you will add it in the DNS section of the app ?

Yeah. And they'll be used as real hosts file, not just as a list of domains to block.

@TPS
Copy link
Contributor

TPS commented Sep 8, 2017

You mean you will add it in the DNS section of the app ?

Yeah. And they'll be used as real hosts file, not just as a list of domains to block.

Then will the "Simplified Domain Names Filter" subscription be moved here, as well?

@ameshkov
Copy link
Member Author

Then will the "Simplified Domain Names Filter" subscription be moved here, as well?

Nope, it is different. This is a "filter" with filtering rules syntax support, and not a list of domains.

@TPS
Copy link
Contributor

TPS commented Sep 10, 2017

Then its blurb (from AGDNS repo, no less) confuses me mightily:

Decsription: Filter composed from several other filters (English filter, Social media filter, Spyware filter, Mobile ads filter, EasyList and EasyPrivacy) and simplified specifically to be better compatible with DNS-level ad blocking.

P.S.: "Description," of course!

@ameshkov
Copy link
Member Author

The good thing about AG DNS is that it understands basic rules syntax.

Regarding the "simplified" part: some of the unsupported modifiers are stripped, it blocks only ad servers and does not contain rules which are not applicable (cosmetic, js, rules that block some/location/ instead of a domain).

@Anonymous142
Copy link

This seems to be the most important, basic issue...
It's already been over 2 years... I'd like to get this feature before Android 10.0.

@ghost
Copy link

ghost commented Mar 5, 2018

I would like to subscribe to NoCoin if AdGuard has this feature.

@uBlock-user
Copy link

Won't happen before v3.0.

@TPS
Copy link
Contributor

TPS commented Apr 5, 2018

Just a note for @AdguardTeam's sanity: Don't forget the disclaimer, "We don't support any sort of shenanigan that anything you add may do to your AG instance, device, or yourself. You have been warned! Take it up with the filter maintainer, & don't forgot about Settings > User Filter overrides [w/ a link to the KB on that]."

Or whatever that does the same job. The mega-metalist from #1818 (comment) gave me a distinct sinking feeling of impending doom! 😱

@ameshkov
Copy link
Member Author

Good idea, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants