[PATCH 4/4] Setzt einen Wert wenn der Router weder Position noch Mail Adresse hat, cronjob alle 5 Minuten

[mayosemmel]

Hallo Christoph,

die Intention hinter deinem Patch ist gut. Wir haben da auch auch schon
eine Weile im IRC diskutiert.
Da fehlt aber noch eine ganze Menge außen herum. Unten findest du ein
paar Anmerkungen Inline.

Weiterhin meine bisherigen Gedanken zu dem Thema. Ich hoffe ich vergesse
jetzt nichts:
1. Wenn der Router unkonfiguriert ist, sollten wir auf der Statuspage
des Routers eine große Warnung anzeigen, mit der Bitte den Router zu
konfigurieren.
Außerdem sollte dort stehen, das ggf. die Funktion eingeschränkt wird,
wenn es nicht geschieht.
2. Man sollte prüfen ob dieser Router eine Hood Mesht. Wie man das am
besten anstellt bin ich mir noch nicht sicher.
3. Wenn Hoods gemesht werden, die VPN Verbindung kappen und eine Meldung
auf der Statuspage ausgeben, weshalb die VPN Verbindung weg ist

Zusätzlich eventuell noch ein etwas extremeres Vorgehen:
4. Wenn der Router unkonfiguriert ist, automatisch ALLE Pakte (mit
Ausnahme OpenStreetMaps) auf den Router umleiten.
5. Auf dem Router sollte dann noch ein mini-DNS laufen, der alles mit
HIER beantwortet.
6. User sieht Statuspage und MUSS nun seinen Router konfigurieren.


Über Feedback zu meinen Gedanken würde ich mich freuen.

Am Donnerstag, den 24.03.2016, 23:21 +0100 schrieb Christoph:
> Signed-off-by: Christoph <suepke.christoph at gmail.com>
> ---
>  bsp/default/root_file_system/etc/config/system.tpl  |  2 +-
>  bsp/default/root_file_system/etc/default.sh         | 21 +++++++++++++++++++++
>  .../root_file_system/usr/lib/micron.d/default       |  1 +
>  3 files changed, 23 insertions(+), 1 deletion(-)
>  create mode 100755 bsp/default/root_file_system/etc/default.sh
> 
> diff --git a/bsp/default/root_file_system/etc/config/system.tpl b/bsp/default/root_file_system/etc/config/system.tpl
> index 24cf700..9794e11 100644
> --- a/bsp/default/root_file_system/etc/config/system.tpl
> +++ b/bsp/default/root_file_system/etc/config/system.tpl
> @@ -1,7 +1,7 @@
>  config 'system'
>  	option 'hostname' 'OpenWrt'
>  	option 'timezone' 'CET-1CEST,M3.5.0,M10.5.0/3'
> -        option 'default' '1'
> +        option 'NoPosAndMail' '1'
Ich denke nicht das dieser Wert Sinn macht. Siehe unten.
>  
>  config 'led' 'status_led_green'
>  	option 'name' 'status'
> diff --git a/bsp/default/root_file_system/etc/default.sh b/bsp/default/root_file_system/etc/default.sh
> new file mode 100755
> index 0000000..64ab896
> --- /dev/null
> +++ b/bsp/default/root_file_system/etc/default.sh
> @@ -0,0 +1,21 @@
> +#!/bin/sh
> +
> +uci get system. at system[0].latitude
> +lat=$?
> +uci get system. at system[0].longitude
> +lon=$?
> +uci get system. at system[0].contact
> +mail=$?
> +if [ $lat -eq 1 ]  && [ $lon -eq 1 ]  && [ $mail -eq 1 ] ; then
Ich denke das ließe sich auch schöner lösen z.B.
if [ uci get system. at system[0].latitude ] && \
[ uci get system. at system[0].longitude ] && \
[ uci get system. at system[0].contact ] ; then
> +echo "Bitte eintragen"
> +uci set system. at system[0].NoPosAndMail="1"
> +uci commit
> +
> +else
> +uci set system. at system[0].NoPosAndMail="0"
> +uci commit
Jetzt haben wir einem Wert gespeichert, ob der Router konfiguriert ist
oder nicht. Ich denke wir sollten stattdessen etwas triggern. Meine
Gedanken dazu habe ich ja bereits oben beschrieben.

Grüße Jan
> +
> +fi
> +
> +def=`uci get system. at system[0].NoPosAndMail`
> +echo $def
> diff --git a/bsp/default/root_file_system/usr/lib/micron.d/default b/bsp/default/root_file_system/usr/lib/micron.d/default
> index 3dbf3d7..defbfcf 100644
> --- a/bsp/default/root_file_system/usr/lib/micron.d/default
> +++ b/bsp/default/root_file_system/usr/lib/micron.d/default
> @@ -1,2 +1,3 @@
>  */5 * * * * sleep $(/usr/bin/random 0 29); sh /etc/fastdstart.sh
>  */1 * * * * /usr/sbin/batctl gw off; sleep 1; /usr/sbin/batctl gw client
> +*/5 * * * *  sh /etc/default.sh
> -- 
> 1.9.1
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160406/09178cb4/attachment-0001.sig>