[PATCH 1/3] fff-hoods: Remove hardcoded IPv4 address and foreign domain

Robert Langhammer rlanghammer at web.de
Mi Jun 17 08:32:52 CEST 2020


Hi Fabian,

ja, kann man machen.

Reviewed-by: Robert Langhammer <rlanghammer at web.de>

Mit Patch 2/3 bleibt nur noch ein Test für den vpn-Start übrig. Da wäre
zu überlegen, ob man das wirklich davon abhängig macht, ob man den keyX
oder heise.de pingen kann. Dann könnte der Test ganz weg. Aber da kann
man sich Gedanken machen, wenn irgendwann mal ein weiteres Protokoll
dazu kommt.


Am 06.05.20 um 16:48 schrieb Fabian Bläse:
> IPv4 is not a requirement for VPN nodes. However, a working DNS
> is a requirement for nodes connecting via VPN. Therefore the
> hardcoded IPv4 address is removed from the internet test.
>
> To improve readability, a loop is used instead of having a seperate
> ping command for each host. The ping utility built into our firmware
> supports both IPv4 and IPv6.
>
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
>  .../fff/fff-hoods/files/usr/sbin/configurehood       | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
>
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index 8c7bb3d..3b92cbc 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -31,14 +31,10 @@ isGatewayAvailable() {
>  
>  # Ping test
>  hasInternet() {
> -	test_ipv4_host1="keyserver.freifunk-franken.de" # Freifunk-Franken keyserver
> -	test_ipv4_host2="8.8.8.8"        # Google DNS
> -	test_ipv6_host1="heise.de"       # heise Zeitschriftenverlag
> -	if ping -w5 -c3 "$test_ipv4_host1" &>/dev/null ||
> -		ping -w5 -c3 "$test_ipv4_host2" &>/dev/null ||
> -		ping6 -w5 -c3 "$test_ipv6_host1" &>/dev/null ; then
> -		return 0
> -	fi
> +	internetAddresses="keyserver.freifunk-franken.de heise.de"
> +	for addr in $internetAddresses; do
> +		ping -w5 -c3 "$addr" &>/dev/null && return 0
> +	done
>  	return 1
>  }
>  



Mehr Informationen über die Mailingliste franken-dev