AdguardTeam / AdguardTeam/CoreLibs

Add support for CIDR in "$network"

オープン
#1,646 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
Feature request needs tech clarification P4: Low Priority: P4
主要言語
言語のデータがありません
スター
52
フォーク
10
PR マージ指標
30日以内にマージされた PR はありません

説明

I received a good tip on Slack that `propellerads` primarily rotate their domains within the CIDR range `139.45.192.0/18`, as seen with `https://securitytrails.com/list/ip/139.45.192.0%2f18`. So I initially opened up my *AdGuard for Windows* installation and tried to enter `139.45.192.0/18$network` into the "User Filters"... and saw that AdGuard refused that entry, claiming it was incorrectly formatted.

Thus I propose adding CIDR support in `$network` blocking rules.

The alternative would be for AdGuard Base to add entries against all 16,384 individual IP addresses, which I assume would be very impractical.

### Steps to reproduce

1. Go to Settings → Adblocker → User filter
2. Try to add `139.45.192.0/18$network` as a new entry.
3. See that the textbox receives red error edges and that AdGuard refuses to save the entry.

### Expected behavior

CIDR can be used in `$network` entries.

### Actual behavior

CIDR cannot be used in `$network` entries.

Screenshot:

![image](https://user-images.githubusercontent.com/22780683/89107104-fc052080-d42e-11ea-8b81-2a7c12ca1d74.png)

### Customer ID

Can't find within the AdGuard program.

### Your environment

* Environment name and version: AdGuard for Windows 7.4.2
* Operating system and version: Windows 10 May 2020 Update, build 19041.388

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。