fff-web-ui: Make public page usable for layer 3 variante #250
No reviewers
Labels
No Label
RFC
RFT
WIP
blocked
bsp
bug
build/scripts/tools
duplicate
feature
fixed
layer3
mantis
more details required
needs changes
node
packages/fff
rejected
security
trivial
upstream
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: freifunk-franken/firmware#250
Loading…
Reference in New Issue
No description provided.
Delete Branch "ChristianD/firmware:webuineu"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The public WebUi was made for the node variante.
With this patch, only show Batman-adv infos on the public web-ui if batctl exists. If babeld or bird exist, show infos about babel and count all clients from the router on the bridge and not from batctl.
@ -57,6 +57,7 @@ contact="$(uci -q get "fff.system.contact")"
</table>
</fieldset>
</td><td>
<% if [ -f /usr/sbin/batctl ] ; then %>
Hi Christian,
grossartig, dass du dich dem web-ui annimmst!
So absolute Pfade koennen ganz schnell zu Stolperfallen werden. Dieses Skript verwendet batctl ohne Pfad. Dann sollte man auch mit which entlang von $PATH suchen.
Habs mal mit dem command -v von lemmi gemacht. Sorry für den doppelten Push, ich habs mal wieder vermasselt auf Anhieb (ich werde mich nie mit git anfreunden...)
An der Stelle noch mal der Hinweis auf Debian's which hunt. POSIX kennt kein
which
. Mitcommand -v
hat man aber eine Alternative.91fe4f1500
tob3384b1d0e
b3384b1d0e
toa6f7d34e6c
@ -69,0 +87,4 @@
fi
if
pgrep babeld >/dev/null; then
neighbours="$(echo dump | nc ::1 33123 | grep '^add neighbour' |
War da nicht was mit dem babeld dump? Bekommt man da nicht von aussen die Kisten ganz leicht kaputt.
509846fa0b
da war was... Ich hab mal bisschen was aus simple babelweb geklaut und noch etwas verbessert ;)
a6f7d34e6c
toec17108880
ec17108880
tod1adf5cbdd
d1adf5cbdd
to230c258ca6
@ -69,0 +73,4 @@
<legend>Status</legend>
<table>
<% #Count Clients
client_count=$(bridge fdb show br br_client brport $(ls "/sys/class/net/br-client/brif" | grep -v '^bat') | grep -v self | grep -v permanent -c)
Das mesh_interface sollte aus dem UCI kommen:
uci get nodewatcher.@network[0].mesh_interface
(siehe50-clients.sh
im Paketfff-network
).Kann
bridge fdb
mit mehreren Argumenten hinterbrport
umgehen? Ich fürchte das geht so nicht wenn mehr als ein Interface gelistet ist.@ -69,0 +88,4 @@
if
pgrep babeld >/dev/null; then
lockfile="/tmp/fff-web-ui.babel.lock"
if [ -e $lockfile ] && [ 30 -gt $(( $(date +%s) - $(date -r $lockfile +%s) )) ] ; then
Wäre das nicht sinnvoll das für bird ebenfalls zu machen?
An sich benötigen wir hier kein extra Lockfile. Es genügt das Änderungsdatum der Inhaltsdatei (
fff-web-ui.babel
) zu prüfen.Vom Codefluss her wird es möglicherweise schöner, wenn man einfach immer den Inhalt der Datei anzeigt, und kurz darüber per lockfile-check den Dateiinhalt aktualisiert, wenn er schon älter als eine gewisse Zeit ist.