Message ID | 20180708091151.3288-1-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 1399e11..86d83fc 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -7,9 +7,6 @@ . /lib/functions/fff/timeserver . /usr/lib/functions/fff/hoodfile -sectorlocal=/etc/sectorfile -sectortmp=/tmp/sectorfile -sectorcopy=/www/hood/sectorfile hoodlocal=/etc/hoodfile rm -f "$hoodfile" @@ -78,9 +75,6 @@ else if hasInternet ; then getKeyserverHoodfile "$hoodfile" - rm -f "$sectortmp" - [ -s "$sectorlocal" ] || rm -f "$sectorcopy" # If internet present, no custom config should be distributed, except when local file is present - #UPLINK: No uplink download if internet present #if no Internet, we connect to the hidden AP and download the file from another Node in range else # connect to wireless hidden ap here and download the json File from the nearest router @@ -89,8 +83,6 @@ else #now we haven't a gateway in Range, we search for a hidden AP to get a keyxchangev2data file! #first we delete all wifi settings rm -f "$hoodfilecopy" # delete this, so interfaces are recreated if reconnect with unchanged hood file takes place - rm -f "$sectorcopy" # always delete: no broadcast for isolated device - rm -f "$sectortmp" uci -q del "system.@system[0].hood" uci -q commit system @@ -99,15 +91,9 @@ else sleep 30 # Wait for the config AP, which may be created at the same time as this script has started getEthernetHoodfile "$hoodfile" || getWirelessHoodfile "$hoodfile" - - #UPLINK: Set up uplink data on first contact: - if [ -s /tmp/keyxchangev2data ]; then - wget -T15 -t5 "http://[fe80::1%w2sta]:2342/sectorfile" -O "$sectortmp" || wget -T15 -t5 "http://[fe80::1%w5sta]:2342/sectorfile" -O "$sectortmp" - fi else echo "We have a Gateway in Range, we load the keyxchangev2data from fe80::1" getGatewayHoodfile "$hoodfile" - #UPLINK: Do nothing fi fi fi @@ -117,20 +103,6 @@ if [ -s "$hoodfile" ]; then # we get a json file in this format: # https://pw.freifunk-franken.de/patch/205/ # but without signature, every hood file we get is valid! - - [ -s "$sectorlocal" ] && sectortmp="$sectorlocal" # Use local file if present (configuration set by user) - if [ -s "$sectortmp" ] ; then - json_load "$(cat "$sectortmp")" - json_select hood - json_get_var mesh_id mesh_id - json_get_var mesh_bssid mesh_bssid - json_get_var mesh_essid mesh_essid - json_get_var essid essid - json_get_var chan2ghz channel2 - json_get_var mesh_type2 mesh_type2 - json_get_var chan5ghz channel5 - json_get_var mesh_type5 mesh_type5 - fi catnew="$(cat "$hoodfile" | sed 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" @@ -213,7 +185,6 @@ if [ -s "$hoodfile" ]; then # copy the file to webroot so that other mesh routers can download it; # copy only after all other steps so IF can be reentered if something goes wrong cp "$hoodfile" "$hoodfilecopy" - [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" # This is a workaround to enable alfred on devices which do not see a configap during initial setup /etc/init.d/alfred restart diff --git a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade index d9ebe15..7ff83f5 100644 --- a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade +++ b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade @@ -10,5 +10,4 @@ cat > /etc/sysupgrade.conf <<-__EOF__ /etc/network.config /etc/config/fff /etc/hoodfile -/etc/sectorfile __EOF__
Hi Schau soweit schlüssig aus. Wir sollten es nicht _sofort_ nach dem Release wieder einbauen sondern erst ein bisschen warten. Ich rechne damit, dass wir nach dem Release "zeitnah" ein Bugfix Release machen müssen und dann wäre das hinderlich. Ich plädiere also dafür, dass wir das Sector-File erst wieder einspielen, wenn wir der Meinung sind, dass der keyXchangeV2 allgemein "stabil" läuft. Am Sonntag, den 08.07.2018, 11:11 +0200 schrieb Fabian Bläse: > To reduce the number of possible corner cases > in the first keyxchangev2 release, this patch > removes the sector file with the intention to > add it back after release. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> Reviewed-by: Tim Niemeyer <tim@tn-x.org> Tim > --- > .../fff-hoods/files/usr/sbin/configurehood | 29 --------------- > ---- > .../files/etc/uci-defaults/99-fff-sysupgrade | 1 - > 2 files changed, 30 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 1399e11..86d83fc 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -7,9 +7,6 @@ > . /lib/functions/fff/timeserver > . /usr/lib/functions/fff/hoodfile > > -sectorlocal=/etc/sectorfile > -sectortmp=/tmp/sectorfile > -sectorcopy=/www/hood/sectorfile > hoodlocal=/etc/hoodfile > > rm -f "$hoodfile" > @@ -78,9 +75,6 @@ else > if hasInternet ; then > getKeyserverHoodfile "$hoodfile" > > - rm -f "$sectortmp" > - [ -s "$sectorlocal" ] || rm -f "$sectorcopy" # If > internet present, no custom config should be distributed, except when > local file is present > - #UPLINK: No uplink download if internet present > #if no Internet, we connect to the hidden AP and > download the file from another Node in range > else > # connect to wireless hidden ap here and download > the json File from the nearest router > @@ -89,8 +83,6 @@ else > #now we haven't a gateway in Range, we > search for a hidden AP to get a keyxchangev2data file! > #first we delete all wifi settings > rm -f "$hoodfilecopy" # delete this, so > interfaces are recreated if reconnect with unchanged hood file takes > place > - rm -f "$sectorcopy" # always delete: no > broadcast for isolated device > - rm -f "$sectortmp" > > uci -q del "system.@system[0].hood" > uci -q commit system > @@ -99,15 +91,9 @@ else > sleep 30 # Wait for the config AP, which may > be created at the same time as this script has started > > getEthernetHoodfile "$hoodfile" || > getWirelessHoodfile "$hoodfile" > - > - #UPLINK: Set up uplink data on first > contact: > - if [ -s /tmp/keyxchangev2data ]; then > - wget -T15 -t5 "http://[fe80::1%w2sta > ]:2342/sectorfile" -O "$sectortmp" || wget -T15 -t5 > "http://[fe80::1%w5sta]:2342/sectorfile" -O "$sectortmp" > - fi > else > echo "We have a Gateway in Range, we load > the keyxchangev2data from fe80::1" > getGatewayHoodfile "$hoodfile" > - #UPLINK: Do nothing > fi > fi > fi > @@ -117,20 +103,6 @@ if [ -s "$hoodfile" ]; then > # we get a json file in this format: > # https://pw.freifunk-franken.de/patch/205/ > # but without signature, every hood file we get is valid! > - > - [ -s "$sectorlocal" ] && sectortmp="$sectorlocal" # Use > local file if present (configuration set by user) > - if [ -s "$sectortmp" ] ; then > - json_load "$(cat "$sectortmp")" > - json_select hood > - json_get_var mesh_id mesh_id > - json_get_var mesh_bssid mesh_bssid > - json_get_var mesh_essid mesh_essid > - json_get_var essid essid > - json_get_var chan2ghz channel2 > - json_get_var mesh_type2 mesh_type2 > - json_get_var chan5ghz channel5 > - json_get_var mesh_type5 mesh_type5 > - fi > > catnew="$(cat "$hoodfile" | sed 's/"timestamp”: *"[0- > 9]*"/"timestamp":0/')" > catold="$(cat "$hoodfilecopy" 2>/dev/null | sed > 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" > @@ -213,7 +185,6 @@ if [ -s "$hoodfile" ]; then > # copy the file to webroot so that other mesh > routers can download it; > # copy only after all other steps so IF can be > reentered if something goes wrong > cp "$hoodfile" "$hoodfilecopy" > - [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > # This is a workaround to enable alfred on devices > which do not see a configap during initial setup > /etc/init.d/alfred restart > diff --git a/src/packages/fff/fff-sysupgrade/files/etc/uci- > defaults/99-fff-sysupgrade b/src/packages/fff/fff- > sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade > index d9ebe15..7ff83f5 100644 > --- a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff- > sysupgrade > +++ b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff- > sysupgrade > @@ -10,5 +10,4 @@ cat > /etc/sysupgrade.conf <<-__EOF__ > /etc/network.config > /etc/config/fff > /etc/hoodfile > -/etc/sectorfile > __EOF__
> On 8. Jul 2018, at 11:37, Tim Niemeyer <tim@tn-x.org> wrote: > > Schau soweit schlüssig aus. Wir sollten es nicht _sofort_ nach dem > Release wieder einbauen sondern erst ein bisschen warten. Ich rechne > damit, dass wir nach dem Release "zeitnah" ein Bugfix Release machen > müssen und dann wäre das hinderlich. Ich plädiere also dafür, dass wir > das Sector-File erst wieder einspielen, wenn wir der Meinung sind, dass > der keyXchangeV2 allgemein "stabil" läuft. Sehe ich genauso, das war auch das, was ich mit “zügig” meinte. Auf jeden Fall genug Platz lassen für Bugfixes aber das wieder einbauen nicht unnötig lange herausziehen. Fabian
Hallo, ich bin inzwischen auch für raus. Mein erster Impuls war zwar drin lassen, ist doch klasse. Allerdings sehe ich es inzwischen als problematisch, da das Sectorfile suggeriert, man könne damit komplexere "Kirchturmsituationen" mit der Firmware realisieren. Was meines Erachtens der falsche Weg ist. Es ist zwar reizvoll die Firmware intelligent zu machen, ich glaube aber, dass wir damit nicht froh werden. Wenn die Netze komplexer werden, wird sich jedes mal eine andere Situation ergeben, die wir unmöglich alle im Vorfeld in der Firmware berücksichtigen können. Und dann heißt es, "des Freifunk get aber net gut" Was ist eigentlich mit dem /etc/hoodfile ? Sollte das nicht auch noch weg? Der Patch sieht soweit schlüssig aus. Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 08.07.2018 um 11:41 schrieb Fabian Bläse: >> On 8. Jul 2018, at 11:37, Tim Niemeyer <tim@tn-x.org> wrote: >> >> Schau soweit schlüssig aus. Wir sollten es nicht _sofort_ nach dem >> Release wieder einbauen sondern erst ein bisschen warten. Ich rechne >> damit, dass wir nach dem Release "zeitnah" ein Bugfix Release machen >> müssen und dann wäre das hinderlich. Ich plädiere also dafür, dass wir >> das Sector-File erst wieder einspielen, wenn wir der Meinung sind, dass >> der keyXchangeV2 allgemein "stabil" läuft. > Sehe ich genauso, das war auch das, was ich mit “zügig” meinte. > Auf jeden Fall genug Platz lassen für Bugfixes aber das wieder einbauen nicht unnötig lange herausziehen. > > Fabian
Tested-by: Fabian Bläse <fabian@blaese.de> Fabian > On 8. Jul 2018, at 11:11, Fabian Bläse <fabian@blaese.de> wrote: > > To reduce the number of possible corner cases > in the first keyxchangev2 release, this patch > removes the sector file with the intention to > add it back after release. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > .../fff-hoods/files/usr/sbin/configurehood | 29 ------------------- > .../files/etc/uci-defaults/99-fff-sysupgrade | 1 - > 2 files changed, 30 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 1399e11..86d83fc 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -7,9 +7,6 @@ > . /lib/functions/fff/timeserver > . /usr/lib/functions/fff/hoodfile > > -sectorlocal=/etc/sectorfile > -sectortmp=/tmp/sectorfile > -sectorcopy=/www/hood/sectorfile > hoodlocal=/etc/hoodfile > > rm -f "$hoodfile" > @@ -78,9 +75,6 @@ else > if hasInternet ; then > getKeyserverHoodfile "$hoodfile" > > - rm -f "$sectortmp" > - [ -s "$sectorlocal" ] || rm -f "$sectorcopy" # If internet present, no custom config should be distributed, except when local file is present > - #UPLINK: No uplink download if internet present > #if no Internet, we connect to the hidden AP and download the file from another Node in range > else > # connect to wireless hidden ap here and download the json File from the nearest router > @@ -89,8 +83,6 @@ else > #now we haven't a gateway in Range, we search for a hidden AP to get a keyxchangev2data file! > #first we delete all wifi settings > rm -f "$hoodfilecopy" # delete this, so interfaces are recreated if reconnect with unchanged hood file takes place > - rm -f "$sectorcopy" # always delete: no broadcast for isolated device > - rm -f "$sectortmp" > > uci -q del "system.@system[0].hood" > uci -q commit system > @@ -99,15 +91,9 @@ else > sleep 30 # Wait for the config AP, which may be created at the same time as this script has started > > getEthernetHoodfile "$hoodfile" || getWirelessHoodfile "$hoodfile" > - > - #UPLINK: Set up uplink data on first contact: > - if [ -s /tmp/keyxchangev2data ]; then > - wget -T15 -t5 "http://[fe80::1%w2sta]:2342/sectorfile" -O "$sectortmp" || wget -T15 -t5 "http://[fe80::1%w5sta]:2342/sectorfile" -O "$sectortmp" > - fi > else > echo "We have a Gateway in Range, we load the keyxchangev2data from fe80::1" > getGatewayHoodfile "$hoodfile" > - #UPLINK: Do nothing > fi > fi > fi > @@ -117,20 +103,6 @@ if [ -s "$hoodfile" ]; then > # we get a json file in this format: > # https://pw.freifunk-franken.de/patch/205/ > # but without signature, every hood file we get is valid! > - > - [ -s "$sectorlocal" ] && sectortmp="$sectorlocal" # Use local file if present (configuration set by user) > - if [ -s "$sectortmp" ] ; then > - json_load "$(cat "$sectortmp")" > - json_select hood > - json_get_var mesh_id mesh_id > - json_get_var mesh_bssid mesh_bssid > - json_get_var mesh_essid mesh_essid > - json_get_var essid essid > - json_get_var chan2ghz channel2 > - json_get_var mesh_type2 mesh_type2 > - json_get_var chan5ghz channel5 > - json_get_var mesh_type5 mesh_type5 > - fi > > catnew="$(cat "$hoodfile" | sed 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" > catold="$(cat "$hoodfilecopy" 2>/dev/null | sed 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" > @@ -213,7 +185,6 @@ if [ -s "$hoodfile" ]; then > # copy the file to webroot so that other mesh routers can download it; > # copy only after all other steps so IF can be reentered if something goes wrong > cp "$hoodfile" "$hoodfilecopy" > - [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > # This is a workaround to enable alfred on devices which do not see a configap during initial setup > /etc/init.d/alfred restart > diff --git a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade > index d9ebe15..7ff83f5 100644 > --- a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade > +++ b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade > @@ -10,5 +10,4 @@ cat > /etc/sysupgrade.conf <<-__EOF__ > /etc/network.config > /etc/config/fff > /etc/hoodfile > -/etc/sectorfile > __EOF__ > -- > 2.18.0 >
Hi Und applied. Tim Am Sonntag, den 08.07.2018, 11:11 +0200 schrieb Fabian Bläse: > To reduce the number of possible corner cases > in the first keyxchangev2 release, this patch > removes the sector file with the intention to > add it back after release. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > .../fff-hoods/files/usr/sbin/configurehood | 29 --------------- > ---- > .../files/etc/uci-defaults/99-fff-sysupgrade | 1 - > 2 files changed, 30 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 1399e11..86d83fc 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -7,9 +7,6 @@ > . /lib/functions/fff/timeserver > . /usr/lib/functions/fff/hoodfile > > -sectorlocal=/etc/sectorfile > -sectortmp=/tmp/sectorfile > -sectorcopy=/www/hood/sectorfile > hoodlocal=/etc/hoodfile > > rm -f "$hoodfile" > @@ -78,9 +75,6 @@ else > if hasInternet ; then > getKeyserverHoodfile "$hoodfile" > > - rm -f "$sectortmp" > - [ -s "$sectorlocal" ] || rm -f "$sectorcopy" # If > internet present, no custom config should be distributed, except when > local file is present > - #UPLINK: No uplink download if internet present > #if no Internet, we connect to the hidden AP and > download the file from another Node in range > else > # connect to wireless hidden ap here and download > the json File from the nearest router > @@ -89,8 +83,6 @@ else > #now we haven't a gateway in Range, we > search for a hidden AP to get a keyxchangev2data file! > #first we delete all wifi settings > rm -f "$hoodfilecopy" # delete this, so > interfaces are recreated if reconnect with unchanged hood file takes > place > - rm -f "$sectorcopy" # always delete: no > broadcast for isolated device > - rm -f "$sectortmp" > > uci -q del "system.@system[0].hood" > uci -q commit system > @@ -99,15 +91,9 @@ else > sleep 30 # Wait for the config AP, which may > be created at the same time as this script has started > > getEthernetHoodfile "$hoodfile" || > getWirelessHoodfile "$hoodfile" > - > - #UPLINK: Set up uplink data on first > contact: > - if [ -s /tmp/keyxchangev2data ]; then > - wget -T15 -t5 "http://[fe80::1%w2sta > ]:2342/sectorfile" -O "$sectortmp" || wget -T15 -t5 > "http://[fe80::1%w5sta]:2342/sectorfile" -O "$sectortmp" > - fi > else > echo "We have a Gateway in Range, we load > the keyxchangev2data from fe80::1" > getGatewayHoodfile "$hoodfile" > - #UPLINK: Do nothing > fi > fi > fi > @@ -117,20 +103,6 @@ if [ -s "$hoodfile" ]; then > # we get a json file in this format: > # https://pw.freifunk-franken.de/patch/205/ > # but without signature, every hood file we get is valid! > - > - [ -s "$sectorlocal" ] && sectortmp="$sectorlocal" # Use > local file if present (configuration set by user) > - if [ -s "$sectortmp" ] ; then > - json_load "$(cat "$sectortmp")" > - json_select hood > - json_get_var mesh_id mesh_id > - json_get_var mesh_bssid mesh_bssid > - json_get_var mesh_essid mesh_essid > - json_get_var essid essid > - json_get_var chan2ghz channel2 > - json_get_var mesh_type2 mesh_type2 > - json_get_var chan5ghz channel5 > - json_get_var mesh_type5 mesh_type5 > - fi > > catnew="$(cat "$hoodfile" | sed 's/"timestamp”: *"[0- > 9]*"/"timestamp":0/')" > catold="$(cat "$hoodfilecopy" 2>/dev/null | sed > 's/"timestamp”: *"[0-9]*"/"timestamp":0/')" > @@ -213,7 +185,6 @@ if [ -s "$hoodfile" ]; then > # copy the file to webroot so that other mesh > routers can download it; > # copy only after all other steps so IF can be > reentered if something goes wrong > cp "$hoodfile" "$hoodfilecopy" > - [ -s "$sectortmp" ] && cp "$sectortmp" "$sectorcopy" > > # This is a workaround to enable alfred on devices > which do not see a configap during initial setup > /etc/init.d/alfred restart > diff --git a/src/packages/fff/fff-sysupgrade/files/etc/uci- > defaults/99-fff-sysupgrade b/src/packages/fff/fff- > sysupgrade/files/etc/uci-defaults/99-fff-sysupgrade > index d9ebe15..7ff83f5 100644 > --- a/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff- > sysupgrade > +++ b/src/packages/fff/fff-sysupgrade/files/etc/uci-defaults/99-fff- > sysupgrade > @@ -10,5 +10,4 @@ cat > /etc/sysupgrade.conf <<-__EOF__ > /etc/network.config > /etc/config/fff > /etc/hoodfile > -/etc/sectorfile > __EOF__
To reduce the number of possible corner cases in the first keyxchangev2 release, this patch removes the sector file with the intention to add it back after release. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- .../fff-hoods/files/usr/sbin/configurehood | 29 ------------------- .../files/etc/uci-defaults/99-fff-sysupgrade | 1 - 2 files changed, 30 deletions(-)