AdguardTeam / AdguardTeam/CoreLibs

Add support for CIDR in "$network"

Abierto
#1,646 2 comentarios 0 reacciones 0 asignados Ver en GitHub
Feature request needs tech clarification P4: Low Priority: P4
Lenguaje dominante
Sin datos de lenguaje
Estrellas
52
Forks
10
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.