AdguardTeam / AdguardTeam/dnsproxy

Consider merging IP and port for clear explanation

オープン
#516 コメント 0 件 リアクション 2 件 担当者 0 名 GitHub で見る
主要言語
Go
スター
3.3k
フォーク
343
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。