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

[mail at adrianschmutzler.de]

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

Wird in 5 Minuten gemergt (wieder nur 1/3), ich werden das PKG_RELEASE noch erhöhen.

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Robert Langhammer
> Sent: Mittwoch, 17. Juni 2020 08:33
> To: franken-dev at freifunk.net
> Subject: Re: [PATCH 1/3] fff-hoods: Remove hardcoded IPv4 address and
> foreign domain
> 
> 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
> >  }
> >
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200618/358cdd24/attachment.sig>