<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi Christian,</p>
<p>lass es erst mal so. Das vpn-select muss man so und so mal neu
machen.</p>
<pre wrap="">Reviewed-by: Robert Langhammer <a class="moz-txt-link-rfc2396E" href="mailto:rlanghammer@web.de"><rlanghammer@web.de></a></pre>
<br>
<div class="moz-cite-prefix">Am 11.02.2018 um 19:26 schrieb robert:<br>
</div>
<blockquote type="cite"
cite="mid:22716631-455f-76e8-0f05-6c0da072e8d5@web.de">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<p>Hi,</p>
<p>in Anbetracht von #72 sehe ich es auch als notwendig an, einen
VPN-stop ein zu bauen. Ausserdem ist es schoener ungenutzte
Prozesse weg zu haben. <br>
</p>
<p>Wie waere es denn, wenn man "kein VPN" auch als zu waehlenden
Zustand bertachtet? Dann gehoert die Funktionalitaet komplett
ins vpn-select. Aktuell und auch mit diesem Patch kann
vpn-select das nicht. Es kommt vom configurehood. <br>
</p>
<p>Robert<br>
</p>
<br>
<div class="moz-cite-prefix">Am 11.02.2018 um 18:21 schrieb
Christian Dresel:<br>
</div>
<blockquote type="cite"
cite="mid:eb110cd4-7c18-8cd6-171c-37576edc95a3@chrisi01.de">
<pre wrap="">hi
On 11.02.2018 18:19, Tim Niemeyer wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi Christian
Am Sonntag, den 11.02.2018, 18:06 +0100 schrieb Christian Dresel:
</pre>
<blockquote type="cite"> </blockquote>
<pre wrap="">[..]
</pre>
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">-----Original Message-----
From: franken-dev [<a class="moz-txt-link-freetext" href="mailto:franken-dev-bounces@freifunk.net" moz-do-not-send="true">mailto:franken-dev-bounces@freifunk.net</a>]
On
Behalf
Of Christian Dresel
Sent: Sonntag, 11. Februar 2018 09:34
To: <a class="moz-txt-link-abbreviated" href="mailto:franken-dev@freifunk.net" moz-do-not-send="true">franken-dev@freifunk.net</a>
Subject: [PATCH] Disable VPN if not in use
fixes 72
</pre>
</blockquote>
</blockquote>
<pre wrap="">Ich bin mir nicht sicher, aber ich dachte bisher immer, der Tag
muss so
aussehen:
Fixes #72
</pre>
</blockquote>
<pre wrap="">ich bin mir da nie sicher, bitte mal sicher sagen was richtig
ist,
damit
ich es ins Wiki schreiben kann, hab vorhin schon wieder ewig
gesucht
und
bin in irgendeiner Mail auf ohne # gestolpert klang aber für mich
auch
falsch :/ Wenn ich sicher weiß was richtig ist, kommts in Wiki
dann
kann
man dort nachschlagen wenn man es braucht ;)
</pre>
</blockquote>
<pre wrap=""><a class="moz-txt-link-freetext" href="http://lmgtfy.com/?q=mantis+fixes+commit+message" moz-do-not-send="true">http://lmgtfy.com/?q=mantis+fixes+commit+message</a>
</pre>
</blockquote>
<pre wrap=""><a class="moz-txt-link-freetext" href="https://wiki.freifunk" moz-do-not-send="true">https://wiki.freifunk</a>-
franken.de/mediawiki/index.php?title=FirmwareEntwicklung&curid=42&dif
f=10432&oldid=10161
</pre>
</blockquote>
<pre wrap="">Danke.
</pre>
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">Signed-off-by: Christian Dresel <a class="moz-txt-link-rfc2396E" href="mailto:fff@chrisi01.de" moz-do-not-send="true"><fff@chrisi01.de></a>
---
src/packages/fff/fff-
hoods/files/usr/sbin/configurehood | 2
++
src/packages/fff/fff-vpn-select/files/usr/sbin/disable-vpn
| 7
+++++++
2 files changed, 9 insertions(+)
create mode 100755
</pre>
</blockquote>
<pre wrap="">src/packages/fff/fff-vpn-select/files/usr/sbin/disable-
</pre>
<blockquote type="cite">
<pre wrap="">vpn
diff --git a/src/packages/fff/fff-
hoods/files/usr/sbin/configurehood
b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
index cf79b42..1831454 100755
--- a/src/packages/fff/fff-
hoods/files/usr/sbin/configurehood
+++ b/src/packages/fff/fff-
hoods/files/usr/sbin/configurehood
@@ -279,6 +279,8 @@ if [ -s "$hoodfile" ]; then
if hasInternet ; then
sh /usr/sbin/vpn-select
+ else
+ sh /usr/sbin/disable-vpn
fi
# now we load the prefix from the hoodfile and set
this to
br-mesh
diff --git a/src/packages/fff/fff-vpn-
select/files/usr/sbin/disable-vpn
b/src/packages/fff/fff-vpn-select/files/usr/sbin/disable-
vpn
new file mode 100755
index 0000000..9d7fd9a
--- /dev/null
+++ b/src/packages/fff/fff-vpn-
select/files/usr/sbin/disable-
vpn
</pre>
</blockquote>
</blockquote>
<pre wrap="">Der Name suggeriert, man könnte damit das VPN dauerhaft
ausschalten.
</pre>
</blockquote>
<pre wrap="">ich kann mich auch mit vpn-stop o.ä. anfreunden, kein Problem.
</pre>
</blockquote>
<pre wrap="">Lass mal kurz überlegen, ob das eine ausführbare Datei oder eher
eine
Art library sein sollte.
</pre>
</blockquote>
<pre wrap="">ich hab auch mal überlegt, vermutlich ist ne library tatsächlich
besser,
</pre>
</blockquote>
<pre wrap="">Ich glaube auch.
</pre>
<blockquote type="cite">
<pre wrap="">dann müsste man die vpn-select aber auch als library führen denk ich
(oder zumindest Teile auslagern? sollte man sich dann mal genauer
angucken).
</pre>
</blockquote>
<pre wrap="">Das könnte man mal als Ticket festhalten.
</pre>
<blockquote type="cite">
<pre wrap="">Ich wäre dafür das jetzt dennoch erstmal so aufzunehmen und
wegen mir ein Mantis Ticket anzulegen das mal irgendwann als Library
umzubauen, mir ist aktuell der Aufwand dazu zu viel.
</pre>
</blockquote>
<pre wrap="">Ich sehe da kein großen Aufwand drin dein neues Script als Lib zu
machen. Du wolltest es ja eh noch renamen.
</pre>
</blockquote>
<pre wrap="">stimmt das wäre kein Aufwand, dann wäre es aber nicht mehr "gleich" mit
den vpn-select und wir hätten bisschen durcheinander, deshalb meine Idee
das in einen rutsch mal irgendwann umzubauen. Wegen mir können wir aber
auch erstmal nur die vpn-stop machen und irgendwann schaut man sich dann
mal die vpn-select an, mir solls egal sein.
mfg
Christian
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">Weitere Meinungen gern gesehen
</pre>
</blockquote>
<pre wrap="">Jo..
Tim
</pre>
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">Dem ist nicht so, daher fand ich deine erste Variante von
damals
<a class="moz-txt-link-freetext" href="http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/20" moz-do-not-send="true">http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/20</a>
17-N
ovember/013303.html
</pre>
</blockquote>
<pre wrap="">gefällt mir mittlerweile weniger da wir sonst VPN Kram im fff-
hood
Packages machen und das dort mitpflegen müssen wenn ein neues VPN
Protokoll dazu kommen sollte. Das vpn-select ist das Paket wo
dies
übernehmen sollte und daher sollte dort auch das ausschalten
passieren
und nur vom fff-hoods aufgerufen werden.
</pre>
</blockquote>
<pre wrap="">Jupp. Das stimmt wohl.
Da fällt mir auf, dass die Abhängigkeit von fff-hoods -> auf fff-
vpn-
select fehlt.
</pre>
</blockquote>
<pre wrap="">uff...
</pre>
<blockquote type="cite">
<pre wrap="">Aber auch das ist alles von der Architektur her nicht sauber. fff-
vpn-
select muss ja aktuell auch mitgepflegt werden, wenn ein neues VPN
kommt.
</pre>
</blockquote>
<pre wrap="">stimmt, aber das ist einfach das "Meta" VPN Paket, da sollte meines
erachtens alles drinnen sein was irgendwie mit VPN zu tun hat, daher
muss dies natürlich angepasst werden, wenn neues VPN dazu kommt. Wenn
man das jetzt aber weiter aufdröselt und auch die fff-hood mitpflegen
muss wirds bald ganz unübersichtlicht, daher...
</pre>
<blockquote type="cite">
<pre wrap="">Aber im Moment gehört die Funktionalität in der Tat am ehesten ins
fff-
vpn-select.
</pre>
</blockquote>
<pre wrap="">... ja ;)
mfg
Christian
</pre>
<blockquote type="cite">
<pre wrap="">Tim
</pre>
<blockquote type="cite">
<pre wrap="">mfg
Christian
</pre>
<blockquote type="cite">
<pre wrap="">besser.
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">@@ -0,0 +1,7 @@
+#!/bin/sh
+
+>/etc/config/tunneldigger
+rm /tmp/fastd_fff_peers/*
+/etc/init.d/fastd stop
+/etc/init.d/tunneldigger stop
+
--
2.11.0
--
franken-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:franken-dev@freifunk.net" moz-do-not-send="true">franken-dev@freifunk.net</a>
<a class="moz-txt-link-freetext" href="http://lists.freifunk.net/mailman/listinfo/franken-dev-frei" moz-do-not-send="true">http://lists.freifunk.net/mailman/listinfo/franken-dev-frei</a>
funk
.net
</pre>
</blockquote>
</blockquote>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
</body>
</html>