[PATCH v2 1/2] Start fastd only if there are fastd peers left
Tobias Klaus
tk+ff at meskal.net
Mo Mai 29 23:15:59 CEST 2017
Hallo,
ich finde [ -n "$(ls 2> /dev/null)" ] zwar marginal lesbarer, aber so
funktioniert es ja auch.
Etwas problematischer finde ich die fehlenden Anführungszeichen um ${project}.
Ich glaube zwar, dass wir nicht so schnell ein $VPN_PROJECT mit Leerzeichen
und Sonderzeichen kriegen, aber schöner ist es schon mit Anführungszeichen.
Leider fehlen die auch in der ganzen Datei.
Keine Ahnung wie wir damit Umgehen sollen. Da die Datei im Falle eines
"kaputten" $project vermutlich eh ziemlichen schaden anrichten würde, das aber
nicht bald ins Haus steht, würde ich das wahrscheinlich so nehmen und dann
sollte mal[tm] jemand[tm] mit dem shellchecker drüber gehen...
Wie seht du/ihr da?
Viele Grüße
Tobias
Am Samstag, 29. April 2017, 18:52:28 CEST schrieb Robert Langhammer:
> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
> ---
> src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
> b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select index
> a0878c5..016c940 100755
> --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
> +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
> @@ -67,7 +67,8 @@ if ping -w5 -c3 "$test_ipv4_host1" &>/dev/null ||
> uci commit fastd
> fi
> make_config
> - /etc/init.d/fastd start
> + # start fastd only if there are some peers left
> + [ "$(ls /etc/fastd/${project}/peers/* 2>/dev/null)" ] &&
> /etc/init.d/fastd start /etc/init.d/tunneldigger start
> else
> # check if new tunneldigger conf is different
Mehr Informationen über die Mailingliste franken-dev