AdguardTeam / AdguardTeam/AdGuardHome

Filtering check doesn’t work for custom rules with multiple client names

Ouverte
#7,705 13 commentaires 3 réactions 1 personne assignée Réclamée par @ainar-g Voir sur GitHub
needs investigation
Langage dominant
TypeScript
Étoiles
36.9k
Forks
2.5k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### Prerequisites

- [x] I have checked the [Wiki](https://github.com/AdguardTeam/AdGuardHome/wiki) and [Discussions](https://github.com/AdguardTeam/AdGuardHome/discussions/categories/q-a) and found no answer

- [x] I have searched other issues and found no duplicates

- [x] I want to report a bug and not [ask a question or ask for help](https://github.com/AdguardTeam/AdGuardHome/discussions/categories/q-a)

- [x] I have set up AdGuard Home correctly and [configured clients to use it](https://github.com/AdguardTeam/AdGuardHome/wiki/Clients). (Use the [Discussions](https://github.com/AdguardTeam/AdGuardHome/discussions/categories/q-a) for help with installing and configuring clients.)

### Platform (OS and CPU architecture)

Linux, AMD64 (aka x86_64)

### Installation

GitHub releases or script from README

### Setup

On one machine

### AdGuard Home version

v0.107.58

### Action

Check DNS Filtering for a domain and ClientID to confirm it’s not blocked (per custom filtering rule), but the custom filtering rule has multiple ClIentID’s to allow the domain.

E.g: @@||events.data.microsoft.com^$important,client='device1'|'device2'|'device3'

### Expected result

Filtering check should return that the domain is allowed for any of the entered ClientID’s

### Actual result

Domain is returned as blocked by the relevant bloxklist.

Checking domains seems only to apply to custom filtering rules with one device specified.

### Additional information and/or screenshots

_No response_

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.