AdguardTeam / AdguardTeam/dnsproxy

Consider merging IP and port for clear explanation

Abierto
#516 0 comentarios 2 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Go
Estrellas
3.3k
Forks
343
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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"
```

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.