[PATCH 2/3] fff-web: add web interface package

[Tim Niemeyer]

Hi Tobias

Am Mittwoch, den 30.12.2015, 10:29 +0100 schrieb Tobias Klaus:
> ich hab mich  mal durchgeklickt. Ich finde die Weboberfläche schon recht 
> gelungen. Schön schlicht und übersichtlich.
Das Lob geht natürlich an die Bielefelder. :-)

> Einzelne Anmerkungen sind dann unten.
> 
> 
> 
> Am Sonntag, 13. Dezember 2015, 22:16:10 schrieb Tim Niemeyer:
> > Signed-off-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> > ---
> > 
> >  src/packages/fff/fff-web/Makefile                  |  42 +
> >  src/packages/fff/fff-web/files/etc/config/uhttpd   |  35 +
> Soweit ich das verstanden habe sollten wir in Paketen keine /etc/config 
> Dateien ausliefern. Wir wissen ja nie, ob wir das letzte oder das erst Paket 
> sind. uci-Einstellungen sollten daher wohl durch ein uci-defaults Skript 
> getätitgt werden. Das wäre für mich kein Grund diesen Patch nicht 
> einzuspielen, aber ich denke langfristig sollten wir das ändern.
Ja, das ist ein sehr guter Einwand. Mir gefällt dein Vorschlag das über
einen zusätzlichen Commit zu fixen, weil es dann gleich im Nodewatcher
passieren könnte.


> >  src/packages/fff/fff-web/files/etc/httpd.conf      |   1 +
> >  .../fff/fff-web/files/www/public/cgi-bin/status    |  63 ++
> >  .../files/www/public/css/grids-responsive-min.css  | 491 ++++++++++++
> >  .../fff/fff-web/files/www/public/css/pure-min.css  | 892
> > +++++++++++++++++++++ .../fff/fff-web/files/www/public/css/side-menu.css |
> > 276 +++++++
> >  .../fff/fff-web/files/www/public/css/style.css     |  78 ++
> >  .../fff/fff-web/files/www/public/index.html        |  13 +
> >  .../fff/fff-web/files/www/ssl/cgi-bin/home         |  71 ++
> >  .../fff/fff-web/files/www/ssl/cgi-bin/misc         |  33 +
> >  .../fff/fff-web/files/www/ssl/cgi-bin/password     |  16 +
> >  .../fff/fff-web/files/www/ssl/cgi-bin/settings     |  16 +
> >  .../fff/fff-web/files/www/ssl/cgi-bin/upgrade      |  36 +
> >  src/packages/fff/fff-web/files/www/ssl/home.html   |  54 ++
> >  src/packages/fff/fff-web/files/www/ssl/home.js     |  46 ++
> >  src/packages/fff/fff-web/files/www/ssl/index.html  | 124 +++
> >  .../fff/fff-web/files/www/ssl/password.html        |  28 +
> src/packages/fff/fff-web $ grep -a charset files/www/ssl/password.html 
> <meta charset="utf-8"/>
> src/packages/fff/fff-web $ file files/www/ssl/password.html
> files/www/ssl/password.html: HTML document, ISO-8859 text
> 
> Das führt dann natürlich zu Darstellungsfehlern. Kann durchaus sein, dass 
> kmail da mal wieder was am encoding verkackt hat, aber zumindestens derjenige 
> der den Patch dann pusht, sollte darauf achten.
Richtig. Ich hab den Patch mal gerade hier applied. Dabei viel mir auf,
dass der auf den letzten fff-nodewatcher Patch noch gerebased werden
muss (Kleinigkeit in fff/Makefile).

file files/www/ssl/password.html 
files/www/ssl/password.html: HTML document, UTF-8 Unicode text


> Insgesamt, müssen wir meiner Meinung nach noch einiges - nicht in diesem Patch 
> - erledigen:
Richtig. Diese Punkte sind jeweils eigene Änderungen, finde ich.

> * "Massenkompatiblen" Zugang auf das webinterface
Hier hatte ich gestern die Idee:
Sobald wir die richtigen Hood Daten am Knoten haben, können die Knoten
für ihre Hood ein definiertes IPv6 Prefix Announcen, wobei das "Router
Lifetime" Flag auf 0 gesetzt wird. Damit wird für dieses Announcement
kein default Gateway eingetragen. Jeder Client in der Hood bekommt also
von den Knoten die RA's. Der Client kann sich dann anhand seines
Prefixes und der MAC vom Knoten die IP des Knoten errechnen.
Dadurch dass die Knoten die Hood Daten haben, wird das falsche Meshen
untereinander auch ausgeschlossen.

> * Graphische Auswahl der Geokoordinaten
> 
> Da ich den Patch hauptsächlich als Arbeitsgrundlage dafür sehe, würde ich ihn 
> auch so applien, damit dann viele damit spielen und Erweiterungen schicken 
> können.
> Deshalb: 
> Reviewed-by: Tobias Klaus <tk+ff at meskal.net
> 
> Grüße
> Tobias
> 
> PS: Um auf das Webinterface zu kommen, verwende ich bisher das Portforwarding 
> von ssh:
> ssh -L 10443:localhost:443 -L 1080:localhost:80 root at ipv6%device
> 
> Damit kann man dann unter http://localhost:80 https://localhost:10443 die
Du meinst bestimmt http://localhost:1080  ..
>  
> öffentliche und die administrationsseite ansehen.
> 
> 
Tim

-------------- 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/20151230/c779c635/attachment-0002.sig>