AdguardTeam / AdguardTeam/FiltersCompiler

Allow using `$permissions` rules without URL part

Open
#229 3 comments 0 reactions 1 assignee Claimed by @slavaleleka View on GitHub
enhancement Priority: P4
Dominant language
JavaScript
Stars
64
Forks
11
PR merge metrics
No merged PRs in 30d

Description

`$permissions=join-ad-interest-group=()` can work without URL, and `/*/` is redundant (like `$removeparam=s_cid`).

Related to https://github.com/AdguardTeam/FiltersRegistry/blob/e529458be21800609358ce99ff1bcc8bb5b02305/filters/filter_3_Spyware/diff.txt:
```
! Error: "The rule is too wide, add domain restriction or make the pattern more specific" in the rule: "*.*$permissions=join-ad-interest-group=()\, run-ad-auction=()\, browsing-topics=()"
*.*$permissions=join-ad-interest-group=()\, run-ad-auction=()\, browsing-topics=()
```

TODO: remove `/*/` in https://github.com/AdguardTeam/AdguardFilters/pull/188049/files after update.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.