AdguardTeam / AdguardTeam/dnsproxy

Consider merging IP and port for clear explanation

Aperta
#516 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Go
Stelle
3.3k
Fork
343
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

https://github.com/AdguardTeam/dnsproxy/blob/f35ca3e650ab099795cb457b360d080af8125a24/config.yaml.dist#L12-L15

Current YAML have this:
```
listen-addrs:
- "0.0.0.0"
listen-ports:
- 53
```

and this is unclear - what will happen if I replace to:
```
listen-addrs:
- "127.0.0.1"
- "192.168.34.56"
listen-ports:
- 53
- 5353
```

will it open:
- 127.0.0.1:53
- 127.0.0.1:5353
- 192.168.34.56:53
- 192.168.34.56:5353

To avoid such confusion, why not merge IP and port like this:

```
server-listen:
- "127.0.0.1:53"
- "192.168.34.56:5353"
```

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.