[PATCH] Add DNS over TLS option inside the Freifunk backbone

Robert Langhammer rlanghammer at web.de
So Apr 5 11:51:40 CEST 2020


Hallo Christian,

Am 04.04.20 um 23:44 schrieb Christian Dresel:
> dann ist es hier auch unnötig und man kann
> uci set stubby.$type.address=$(echo $f | cut -d @ -f 2)
> machen?

es ist wohl nicht nötig bei einer Variablenzuweisung eine $(..) zu quoten.

ein name=$(echo robert langhammer) funktioniert tatsächlich. Sieht aber
schon übel aus. Denn ein name=robert langhammer alleine funktioniert ja
nicht. Es sieht so aus, als würde daraus ein name="robert langhammer"

Das ist auch nicht konsistent. Wenn man das hier testet.

mkdir $(echo robert langhammer)  != mkdir "$(echo robert langhammer)"

Also ja, bei einer Variablenzuweisung darf man es wohl weg lassen,
obwohl shellcheck mault:

uci set stubby.$type.address=$(echo $f | cut -d @ -f 2)
               ^-- SC2086: Double quote to prevent globbing and word
splitting.
                             ^-- SC2046: Quote this to prevent word
splitting.
                                    ^-- SC2086: Double quote to prevent
globbing and word splitting.

Das war mir neu, darum hab ich die " bisher gerne drum rum gemacht und
werde auch dabei bleiben.

-> mach wie du meinst.

Viele Grüße
Robert






Mehr Informationen über die Mailingliste franken-dev