Message ID | 20181203195455.7261-2-fabian@blaese.de |
---|---|
State | Accepted |
Headers | show |
diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood index 4ae1364..2d448fc 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -77,6 +77,9 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then fi if [ -s "$hoodfilelocal" ]; then + # remove webroot hoodfile to prevent offering local hoodfiles to neighbours + rm -f "$hoodfilewww" + cp "$hoodfilelocal" "$hoodfiletmp" echo "Use local hood file" else
Hallo Fabian, ich frage mich zwar, ob das irgendwie an einer anderen Stelle noch "allgemeingültiger" geht, aber mir fällt nichts ein. Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Montag, 3. Dezember 2018 20:55 > To: franken-dev@freifunk.net > Subject: [PATCH 2/2] configurehood: Don't offer hoodfile when using local > hoodfile > > When using local hoodfile neither this nor central hoodfile > downloaded earlier should be offered to neighbours, so remove it > from the webroot. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 4ae1364..2d448fc 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -77,6 +77,9 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > fi > > if [ -s "$hoodfilelocal" ]; then > + # remove webroot hoodfile to prevent offering local hoodfiles to > neighbours > + rm -f "$hoodfilewww" > + > cp "$hoodfilelocal" "$hoodfiletmp" > echo "Use local hood file" > else > -- > 2.19.1
Dafür müsste man das ganze Ding wohl mal ordentlich restrukturieren. Es ist einfach zu sehr über die Zeit gewachsen. Aber das restrukturieren ist echt kompliziert und da habe ich regelrecht Angst davor.. ;-) Gruß Fabian On 06.12.18 13:02, Adrian Schmutzler wrote: > Hallo Fabian, > > ich frage mich zwar, ob das irgendwie an einer anderen Stelle noch "allgemeingültiger" geht, aber mir fällt nichts ein.
Hi, Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 03.12.18 um 20:54 schrieb Fabian Bläse: > When using local hoodfile neither this nor central hoodfile > downloaded earlier should be offered to neighbours, so remove it > from the webroot. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 4ae1364..2d448fc 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -77,6 +77,9 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > fi > > if [ -s "$hoodfilelocal" ]; then > + # remove webroot hoodfile to prevent offering local hoodfiles to neighbours > + rm -f "$hoodfilewww" > + > cp "$hoodfilelocal" "$hoodfiletmp" > echo "Use local hood file" > else
When using local hoodfile neither this nor central hoodfile downloaded earlier should be offered to neighbours, so remove it from the webroot. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 3 +++ 1 file changed, 3 insertions(+)