[PATCH 6/6] fff-dhcp: Add configuration scripts for dns

mail at adrianschmutzler.de mail at adrianschmutzler.de
Sa Mai 25 21:23:22 CEST 2019


Hallo Fabian,

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

mit ein paar Vorschlägen unten.

Ich habe die Datei bei mir 40-dns genannt, wenn du Lust hast das anzugleichen wäre das praktisch für mich.


> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Samstag, 25. Mai 2019 20:49
> To: franken-dev at freifunk.net
> Subject: [PATCH 6/6] fff-dhcp: Add configuration scripts for dns
> 
> Because DNS Forwarding is done by dnsmasq which we configure inside the
> fff-dhcp package, the configuration scripts for dns are placed in this package.
> ---
>  .../fff/fff-dhcp/files/etc/gateway.d/35-dns   | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)
>  create mode 100644 src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
> 
> diff --git a/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
> b/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
> new file mode 100644
> index 0000000..8ffd440
> --- /dev/null
> +++ b/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
> @@ -0,0 +1,21 @@
> +configure() {
> +	## dns
> +	uci -q del dhcp. at dnsmasq[0].server

Das habe ich ins if gezogen, dann bleibt alles beim alten, wenn der Block in der config fehlt. Ist aber reine Geschmackssache.

> +	if uci -q get gateway. at dns[0].server > /dev/null; then
> +		for f in $(uci get gateway. at dns[0].server); do

Das kann man schön zusammenfassen:

	if dnsservers=$(uci -q get gateway. at dns[0].server); then
		for f in $dnsservers; do

Grüße

Adrian

> +			uci add_list dhcp. at dnsmasq[0].server=$f
> +			uci add_list dhcp. at dnsmasq[0].server="/in-
> addr.arpa/$f"
> +			uci add_list dhcp. at dnsmasq[0].server="/ip6.arpa/$f"
> +		done
> +	else
> +		echo "WARNING: No DNS servers set!"
> +	fi
> +}
> +
> +commit() {
> +	uci commit dhcp
> +}
> +
> +revert() {
> +	uci revert dhcp
> +}
> --
> 2.21.0
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190525/c8bb20a6/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev