| Message ID | 1471029649-3782-1-git-send-email-rlanghammer@web.de |
|---|---|
| State | Accepted, archived |
| Commit | 23203334a4932eaaf317471a9bc15bb93faee4f9 |
| Headers | show |
diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select index b071ff6..1394cc3 100755 --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select @@ -21,7 +21,7 @@ for file in $filecounts; do # Gateway offers l2tp FDPORT=$(awk '/remote/{gsub(";", ""); print $5}' /etc/fastd/${project}/peers/$file) L2PORT=$((FDPORT + 10000)) - UUID=_$hostname + UUID=$hostname uci set tunneldigger.$count=broker uci set tunneldigger.$count.address="$IP:$L2PORT"
On 08/12/2016 09:20 PM, Robert Langhammer wrote: Hi Robert > The current broker is no longer cutting off the first character from UUID Ist doch nicht schlimmer, die UUID soll doch nur eindeutig den Tunnel identifizieren, oder? Ob der Unterstrich da ist oder nicht macht fuer mich technisch keinen Unterschied. Gab es frueher wirklich eine Anforderung, die auf diesen Unterstrich bestand? Gruss -Steffen > Signed-off-by: Robert Langhammer <rlanghammer@web.de> > --- > src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select > index b071ff6..1394cc3 100755 > --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select > +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select > @@ -21,7 +21,7 @@ for file in $filecounts; do > # Gateway offers l2tp > FDPORT=$(awk '/remote/{gsub(";", ""); print $5}' /etc/fastd/${project}/peers/$file) > L2PORT=$((FDPORT + 10000)) > - UUID=_$hostname > + UUID=$hostname > > uci set tunneldigger.$count=broker > uci set tunneldigger.$count.address="$IP:$L2PORT" >
Hallo, Am 15.08.2016 um 19:28 schrieb Steffen Pankratz: > On 08/12/2016 09:20 PM, Robert Langhammer wrote: > > Hi Robert > >> The current broker is no longer cutting off the first character from UUID > Ist doch nicht schlimmer, die UUID soll doch nur eindeutig den Tunnel > identifizieren, oder? > Ob der Unterstrich da ist oder nicht macht fuer mich technisch keinen > Unterschied. > Gab es frueher wirklich eine Anforderung, die auf diesen Unterstrich > bestand? Technisch gesehen nein. Mir war nur aufgefallen, dass in den Logs der erste Buchstabe gefehlt hat, was unschön ist. Ich habe ja den $hostname als UUID genommen. Beim aktuellen Broker ist das nicht mehr der Fall, drum kann das Zeichen weg. Scheinbar haben die Slovenen was gepatcht, ich hab aber nicht danach gesucht. Ist auch nich wirklich wichtig. Robert > > > Gruss > -Steffen > >> Signed-off-by: Robert Langhammer <rlanghammer@web.de> >> --- >> src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select >> index b071ff6..1394cc3 100755 >> --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select >> +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select >> @@ -21,7 +21,7 @@ for file in $filecounts; do >> # Gateway offers l2tp >> FDPORT=$(awk '/remote/{gsub(";", ""); print $5}' /etc/fastd/${project}/peers/$file) >> L2PORT=$((FDPORT + 10000)) >> - UUID=_$hostname >> + UUID=$hostname >> >> uci set tunneldigger.$count=broker >> uci set tunneldigger.$count.address="$IP:$L2PORT" >> > > >
On 08/15/2016 08:29 PM, Robert Langhammer wrote: Hi Robert >> Ist doch nicht schlimmer, die UUID soll doch nur eindeutig den Tunnel >> identifizieren, oder? >> Ob der Unterstrich da ist oder nicht macht fuer mich technisch keinen >> Unterschied. >> Gab es frueher wirklich eine Anforderung, die auf diesen Unterstrich >> bestand? > Technisch gesehen nein. Mir war nur aufgefallen, dass in den Logs der > erste Buchstabe gefehlt hat, was unschön ist. Ich habe ja den $hostname > als UUID genommen. Beim aktuellen Broker ist das nicht mehr der Fall, > drum kann das Zeichen weg. Scheinbar haben die Slovenen was gepatcht, > ich hab aber nicht danach gesucht. Ist auch nich wirklich wichtig. Verstanden, danke fuer die Erklaerung. Es koennte dieser Commit gewesen sein: https://github.com/wlanslovenija/tunneldigger/commit/87e75d99d2e23df6dae63bb735af6e031967bf8f Danke fuer den Patch, dieser ist jetzt im Master: https://github.com/FreifunkFranken/firmware/commit/23203334a4932eaaf317471a9bc15bb93faee4f9 Gruss -Steffen
Hi Steffen, ja das scheint der commit zu sein. Du hast dir echt die Mühe gemacht danach zu suchen! wow! jetzt fehlt nur noch der Patch vom 24.7. [PATCH 2/2] fff-vpn-select: add fastd start/stop for various situations Der wäre schon noch wichtig, da sonst fastd an bleibt ohne peers, bzw. nicht an geht, wenn wenn vorher nur l2tp. Den anderen vom 24.7 verwerfen! Grüße Robert Am 16.08.2016 um 18:48 schrieb Steffen Pankratz: > On 08/15/2016 08:29 PM, Robert Langhammer wrote: > > Hi Robert > >>> Ist doch nicht schlimmer, die UUID soll doch nur eindeutig den Tunnel >>> identifizieren, oder? >>> Ob der Unterstrich da ist oder nicht macht fuer mich technisch keinen >>> Unterschied. >>> Gab es frueher wirklich eine Anforderung, die auf diesen Unterstrich >>> bestand? >> Technisch gesehen nein. Mir war nur aufgefallen, dass in den Logs der >> erste Buchstabe gefehlt hat, was unschön ist. Ich habe ja den $hostname >> als UUID genommen. Beim aktuellen Broker ist das nicht mehr der Fall, >> drum kann das Zeichen weg. Scheinbar haben die Slovenen was gepatcht, >> ich hab aber nicht danach gesucht. Ist auch nich wirklich wichtig. > Verstanden, danke fuer die Erklaerung. > Es koennte dieser Commit gewesen sein: > https://github.com/wlanslovenija/tunneldigger/commit/87e75d99d2e23df6dae63bb735af6e031967bf8f > > > Danke fuer den Patch, dieser ist jetzt im Master: > https://github.com/FreifunkFranken/firmware/commit/23203334a4932eaaf317471a9bc15bb93faee4f9 > > Gruss > -Steffen >
On 08/16/2016 08:13 PM, Robert Langhammer wrote: Hi Robert [...] > jetzt fehlt nur noch der Patch vom 24.7. > [PATCH 2/2] fff-vpn-select: add fastd start/stop for various situations > > Der wäre schon noch wichtig, da sonst fastd an bleibt ohne peers, bzw. > nicht an geht, wenn wenn vorher nur l2tp. Okay, den gucke ich mir die Tage mal an :) > Den anderen vom 24.7 verwerfen! Du meinst "[1/2] nodewatcher: ignore direct neighbours via l2tp (gateways)"? Wieso soll der weg? Gruss -Steffen
Am 16.08.2016 um 21:31 schrieb Steffen Pankratz: > On 08/16/2016 08:13 PM, Robert Langhammer wrote: > > Hi Robert > > [...] > >> jetzt fehlt nur noch der Patch vom 24.7. >> [PATCH 2/2] fff-vpn-select: add fastd start/stop for various situations >> >> Der wäre schon noch wichtig, da sonst fastd an bleibt ohne peers, bzw. >> nicht an geht, wenn wenn vorher nur l2tp. > Okay, den gucke ich mir die Tage mal an :) > > >> Den anderen vom 24.7 verwerfen! > Du meinst "[1/2] nodewatcher: ignore direct neighbours via l2tp (gateways)"? > Wieso soll der weg? Ich bin mir noch nicht ganz sicher, ob es der richtige Weg ist, das einfach im nodewatcher aus zu blenden. Ich will mir das nochmal anschauen, warum batman manchmal die mac der Bridge als Originator sieht, aber eben nicht immer. Darum tauchen auch nicht alle l2tp Tunnelpartner als neighbours auf. Hast du eine Idee warum das so ist? Würfelt der Kernel da? Grüße Robert > > > Gruss > -Steffen >
On 08/17/2016 02:10 AM, Robert Langhammer wrote: Hi Robert [...] >>> Den anderen vom 24.7 verwerfen! >> Du meinst "[1/2] nodewatcher: ignore direct neighbours via l2tp (gateways)"? >> Wieso soll der weg? > Ich bin mir noch nicht ganz sicher, ob es der richtige Weg ist, das > einfach im nodewatcher aus zu blenden. > Ich will mir das nochmal anschauen, warum batman manchmal die mac der > Bridge als Originator sieht, aber eben nicht immer. Darum tauchen auch > nicht alle l2tp Tunnelpartner als neighbours auf. Verstehe. > Hast du eine Idee > warum das so ist? Würfelt der Kernel da? Nope, mit der Thematik habe ich mich bisher noch nicht beschaeftigt. Gruss -Steffen
The current broker is no longer cutting off the first character from UUID Signed-off-by: Robert Langhammer <rlanghammer@web.de> --- src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)