[PATCH 5/5] WebUI: show VPN status for both fastd and l2tp individually (adds L2TP status)
Tobias Klaus
tk+ff at meskal.net
Di Mai 30 00:37:54 CEST 2017
Hallo Adrian,
ich kenne mich mit dem WebUI nicht so aus, aber der Patch an sich sieht sehr
plausibel aus und würde von mir auch ein Reviewed-by: kriegen.
Allerdings fällt mir leider jetzt erst auf, dass bei deinen bisherigen Patches
das "Signed-off-by:" fehlt. Wir haben uns in Anlehnung an Projekte wie
beispielsweise dem Linux-Kernel[0] entschieden diese Zeile ähnlich wie die
Reviewed-by: Zeilen für die Dokumentation zu nutzen.
Könntest du diese Zeile bitte noch jeweils einfügen? Am einfachsten geht das
durch einen interaktiven Rebase und einem "git commit --amend --signoff" für
jeden Commit.
Vielen Dank und entschuldige die Umstände.
Tobias
[0]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/
Documentation/process/submitting-patches.rst
Am Montag, 29. Mai 2017, 22:09:14 CEST schrieb Adrian Schmutzler:
> ---
> src/packages/fff/fff-web/files/www/ssl/cgi-bin/home.html | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/home.html
> b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/home.html index
> 9ca0107..fd0d8b4 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/home.html
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/home.html
> @@ -4,9 +4,14 @@
> <%in /www/ssl/cgi-bin/helpers %>
> <%
> if pidof fastd >/dev/null ; then
> - vpn_active="Ja"
> + vpn_active="aktiv"
> else
> - vpn_active="Nein"
> + vpn_active="inaktiv"
> +fi
> +if grep -q '1' /sys/class/net/l2tp*/carrier ; then
> + l2tp_active="aktiv"
> +else
> + l2tp_active="inaktiv"
> fi
> up="$(uptime)"
> load="${up##*:}"
> @@ -42,7 +47,8 @@ fi
> <tr><th>MAC-Adresse:</th><td><% uci -q get network.mesh.macaddr
> %></td></tr> <tr><th>Bekannte Knoten:</th><td><%= $((`cat
> /sys/kernel/debug/batman_adv/bat0/transtable_global | grep '^ [^ ]' | cut
> -b 39-55 | sort | uniq | wc -l 2>/dev/null`+1)) %></td></tr>
> <tr><th>Nachbarknoten:</th><td><%= $(cat
> /sys/kernel/debug/batman_adv/bat0/originators | grep '^[0-9a-f]' | cut -b
> 37-53 | sort | uniq | wc -l 2>/dev/null) %></td></tr> - <tr><th>VPN
> Aktiv:</th><td><%= ${vpn_active} %></td></tr>
> + <tr><th>VPN via fastd:</th><td><%= ${vpn_active} %></td></tr>
> + <tr><th>VPN via L2TP:</th><td><%= ${l2tp_active} %></td></tr>
> <tr><th>Laufzeit:</th><td><%= ${uptime} %></td></tr>
> <tr><th>Auslastung:</th><td><%= ${load} %></td></tr>
> <tr><th>Uhrzeit:</th><td><% date %></td></tr>
Mehr Informationen über die Mailingliste franken-dev