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

Get blocklist names from "! Title:" definition for Adblock lists #6020

Closed
3 tasks done
cenk opened this issue Jul 16, 2023 · 5 comments
Closed
3 tasks done

Get blocklist names from "! Title:" definition for Adblock lists #6020

cenk opened this issue Jul 16, 2023 · 5 comments

Comments

@cenk
Copy link

cenk commented Jul 16, 2023

Prerequisites

  • I have checked the Wiki and Discussions and found no answer

  • I have searched other issues and found no duplicates

  • I want to request a feature or enhancement and not ask a question

The problem

AGH should respect adblock blocklist's name (title) definition in DNS blocklists page.

Proposed solution

Geting blocklist names from "! Title: " definition for Adblock lists.

Alternatives considered and additional information

No response

@Freebase394
Copy link

Freebase394 commented Jul 16, 2023

I could not understand perfectly what you mean.
Could you give an example or be more specific please?

Snario 1###
AdBlock as an extension has the particularity to eliminate efficiently the scripts | functions | injectors inserted in the pages and this can be analyzed which HERE. <<---Universal example
These extensions are very efficient as in sites with many ads, as in Youtube.com itself for example!

Snario 2###
Or do you want to block all domains that contain that specific keyword?
For example, you want to block the word "statistics":

dl1.statistics.universalwebsite.com
statistics.youtube.com
statistics.blogspot.com
statistics.facebook.com
Do you want to block all domains that contain the specified keyword (target words) in AdGuardHome ?

@ainar-g
Copy link
Contributor

ainar-g commented Jul 17, 2023

I guess we could let users leave the Name field blank, which would use the ! Title directive content instead. But we'll need a third option in case there is neither a user-provided name nor a directive.

@cenk
Copy link
Author

cenk commented Jul 17, 2023

I think there is a title parser function for Adblock blocklists. But I'm really not sure why it is not working. Or it is completly different thing. I'm not familiar with Go. 🙂

func (p *Parser) parseLineTitle(line []byte) (badIdx int, isRule bool) {

@ainar-g ainar-g added this to the v0.107.37 milestone Aug 8, 2023
adguard pushed a commit that referenced this issue Aug 11, 2023
Updates #6020.

Squashed commit of the following:

commit fedb941
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Fri Aug 11 12:45:06 2023 +0300

    filtering: imp tests

commit d85d193
Merge: f1c1edd 94cf50a
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Fri Aug 11 11:07:39 2023 +0300

    Merge remote-tracking branch 'origin/master' into 6020-rulelist-name

commit f1c1edd
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Fri Aug 11 10:59:07 2023 +0300

    filtering: imp tests

commit 39e9d54
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Fri Aug 11 10:02:48 2023 +0300

    filtering: imp code

commit 230f15d
Merge: 1940fb3 111005b
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Thu Aug 10 15:03:08 2023 +0300

    Merge remote-tracking branch 'origin/master' into 6020-rulelist-name

commit 1940fb3
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Thu Aug 10 15:01:57 2023 +0300

    all: docs

commit 810f6d1
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Thu Aug 10 15:00:59 2023 +0300

    filtering: imp code

commit f310dd2
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Thu Aug 10 12:19:55 2023 +0300

    client: flt name

commit 9494771
Author: Dimitry Kolyshev <dkolyshev@adguard.com>
Date:   Thu Aug 10 12:18:57 2023 +0300

    filtering: flt name
@Mizzick
Copy link
Contributor

Mizzick commented Aug 11, 2023

Please have a look, we have implemented the requested changes.
The new build version v0.108.0-a.650+c54635e8 has been just published to the edge channel.

@cenk
Copy link
Author

cenk commented Aug 11, 2023

Yes works on v0.108.0-a.650+c54635e8.
When I leave the name blank, getting from adblock file. Thanks.

@ainar-g ainar-g closed this as completed Aug 14, 2023
@ainar-g ainar-g modified the milestones: v0.107.38, v0.107.37 Sep 7, 2023
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