AdguardTeam / AdguardTeam/AdGuardHome

Rate limiting allow list regression

Aberta
#8,349 2 comentários 1 reação 1 responsável Reivindicada por @EugeneOne1 Ver no GitHub
bug
Linguagem predominante
TypeScript
Estrelas
36.9k
Forks
2.5k
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

### 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)

FreeBSD, AMD64 (aka x86_64)

### Installation

Custom package (OpenWrt, HomeAssistant, etc; please mention in the description)

### Setup

On a router, DHCP is handled by the router

### AdGuard Home version

v0.107.74

### Action

```sh
for c in {0..31}; do dig example.com +short && echo; done
```

### Expected result

I expected to see 32 successfully resolved sets of IP addresses:

```
123.123.123.123
231.231.231.231
```
repeated 32 times.

### Actual result

I only see 20 sets of IP addresses (my rate limit is set to 20):
```
123.123.123.123
231.231.231.231
```
repeated 20 times.

### Additional information and/or screenshots

I have AdGuardHome installed on an OPNSense firewall and after the latest update, rate limiting allowlist seems to have stopped working.

With rate limit set to 20, AdGuardHome should only resolve 20 DNS requests per second, unless the client issuing the request is on the allow list. When the client is on the allow list, AdGuardHome shouldn't apply any rate limiting for that client.

Setting rate limit to `0` correctly disables rate limiting.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.