[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