[PATCH] Disable VPN if not in use
Tim Niemeyer
tim at tn-x.org
So Feb 11 18:19:06 CET 2018
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.
> 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 Binärdaten wurde abgetrennt...
Dateiname : signature.asc
Dateityp : application/pgp-signature
Dateigröße : 488 bytes
Beschreibung: This is a digitally signed message part
URL : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180211/75d3207d/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev