[PATCH] Disable VPN if not in use

robert rlanghammer at web.de
So Feb 11 21:59:27 CET 2018 

Hi Christian,

lass es erst mal so. Das vpn-select muss man so und so mal neu machen.

Reviewed-by: Robert Langhammer <rlanghammer at web.de>


Am 11.02.2018 um 19:26 schrieb robert:
>
> Hi,
>
> 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.
>
> 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.
>
> Robert
>
>
> Am 11.02.2018 um 18:21 schrieb Christian Dresel:
>> hi
>>
>> On 11.02.2018 18:19, Tim Niemeyer wrote:
>>> Hi Christian
>>>
>>> Am Sonntag, den 11.02.2018, 18:06 +0100 schrieb Christian Dresel:
>>> [..]
>>>>>>>>> -----Original Message-----
>>>>>>>>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net]
>>>>>>>>> On
>>>>>>>>> Behalf
>>>>>>>>> Of Christian Dresel
>>>>>>>>> Sent: Sonntag, 11. Februar 2018 09:34
>>>>>>>>> To: franken-dev at freifunk.net
>>>>>>>>> Subject: [PATCH] Disable VPN if not in use
>>>>>>>>>
>>>>>>>>> fixes 72
>>>>>>> Ich bin mir nicht sicher, aber ich dachte bisher immer, der Tag
>>>>>>> muss so
>>>>>>> aussehen:
>>>>>>> Fixes #72
>>>>>>>
>>>>>> 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 ;)
>>>>> http://lmgtfy.com/?q=mantis+fixes+commit+message
>>>> https://wiki.freifunk-
>>>> franken.de/mediawiki/index.php?title=FirmwareEntwicklung&curid=42&dif
>>>> f=10432&oldid=10161
>>> Danke.
>>>
>>>>>>>>> Signed-off-by: Christian Dresel <fff at chrisi01.de>
>>>>>>>>> ---
>>>>>>>>>  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
>>>>>>>> src/packages/fff/fff-vpn-select/files/usr/sbin/disable-
>>>>>>>>> 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
>>>>>>> Der Name suggeriert, man könnte damit das VPN dauerhaft
>>>>>>> ausschalten.
>>>>>> ich kann mich auch mit vpn-stop o.ä. anfreunden, kein Problem.
>>>>> Lass mal kurz überlegen, ob das eine ausführbare Datei oder eher
>>>>> eine
>>>>> Art library sein sollte.
>>>> ich hab auch mal überlegt, vermutlich ist ne library tatsächlich
>>>> besser,
>>> Ich glaube auch.
>>>
>>>> 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).
>>> Das könnte man mal als Ticket festhalten.
>>>
>>>> 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.
>>> Ich sehe da kein großen Aufwand drin dein neues Script als Lib zu
>>> machen. Du wolltest es ja eh noch renamen.
>> 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
>>
>>>> Weitere Meinungen gern gesehen
>>> Jo..
>>>
>>> Tim
>>>
>>>>>>> Dem ist nicht so, daher fand ich deine erste Variante von
>>>>>>> damals
>>>>>>> http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/20
>>>>>>> 17-N
>>>>>>> ovember/013303.html
>>>>>> 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.
>>>>> Jupp. Das stimmt wohl.
>>>>>
>>>>> Da fällt mir auf, dass die Abhängigkeit von fff-hoods -> auf fff-
>>>>> vpn-
>>>>> select fehlt.
>>>> uff...
>>>>
>>>>> 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.
>>>> 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...
>>>>
>>>>> Aber im Moment gehört die Funktionalität in der Tat am ehesten ins
>>>>> fff-
>>>>> vpn-select.
>>>> ... ja ;)
>>>>
>>>> mfg
>>>>
>>>> Christian
>>>>
>>>>> Tim
>>>>>
>>>>>> mfg
>>>>>>
>>>>>> Christian
>>>>>>
>>>>>>> besser.
>>>>>>>
>>>>>>>>> @@ -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
>>>>>>>>> franken-dev at freifunk.net
>>>>>>>>> http://lists.freifunk.net/mailman/listinfo/franken-dev-frei
>>>>>>>>> funk
>>>>>>>>> .net
>>>>>>
>>
>>
>
>
>

-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180211/d33f32b8/attachment.html>

Mehr Informationen über die Mailingliste franken-dev