[PATCH v2 1/2] Start fastd only if there are fastd peers left

[Tobias Klaus]

Hey Robert,

alles klar, dann sehen wir das erstmal gleich. Aktuell kommen wir ja nicht 
sooo wirklich schnell voran mit dem dezKeyExchange. Vielleicht schaffen wir es 
ja auch vorher aufzuräumen ;-) 

Reviewed-by: Tobias Klaus <tk+ff at meskal.net>

Viele Grüße
Tobias

Am Dienstag, 30. Mai 2017, 10:51:39 CEST schrieb robert:
> Hi,
> 
> die fehlenden Anfuerungszeichen sollte man mit einem extra Patch rein
> machen (ich finde, man sollte da auch konsequent sein). Das betrifft
> auch noch ein paar andere Dateien. Dann waere das auf einen Schlag
> bereinigt. Das stammt noch so aus den alten fastd-skript-zeiten.
> 
> Wenn ich mich richtig erinnere, ging doch auch die Meinung um, da nicht
> mehr all zu viel Energie rein zu stecken, da sich das mit dem dez.keyX
> aendert.? Andererseits ist so ein Patch ja mal schnell gemacht.
> 
> Viele Gruesse
> 
> Robert
> 
> Am 29.05.2017 um 23:15 schrieb Tobias Klaus:
> > 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