node: Make vpn-select modular and add vxlan-vpn #78

Closed
rohammer wants to merge 2 commits from rohammer:vxlan into master
Member

Ich hab das nochmal neu zusammengebaut und vorallem das vpn-select erheblich vereinfacht. Es wird kein protocol bevorzugt. Die Auswahl erfolgt über das hoodfile/keyXchange.
Mit einem neueren batman-adv kann dann am Gateway eine Bevorzugung mit hop-penalty pro hardif eingestellt werden.
Somit liegt es an den Gatewaybetreibern dies zu steuern.

Ich hab das nochmal neu zusammengebaut und vorallem das vpn-select erheblich vereinfacht. Es wird kein protocol bevorzugt. Die Auswahl erfolgt über das hoodfile/keyXchange. Mit einem neueren batman-adv kann dann am Gateway eine Bevorzugung mit hop-penalty pro hardif eingestellt werden. Somit liegt es an den Gatewaybetreibern dies zu steuern.
rohammer changed title from node: Make vpn-select modulad and add vxlan-vpn to node: Make vpn-select modular and add vxlan-vpn 2021-01-14 22:21:00 +01:00
jkimmel reviewed 2021-01-14 23:06:47 +01:00
@ -0,0 +18,4 @@
echo "remote \"${address}\" port ${port};" >> "$filename"
echo "" >> "$filename"
echo "float yes;" >> "$filename"
}
Owner

Also ich finde ja erst mal alle Daten zusammensuchen und dann die Datei schreiben etwas übersichtlicher

json_get_var servername name
json_get_var key key
json_get_var address address
json_get_var port port

filename="/etc/fastd/fff/peers/$servername"

echo "#name \"${servername}\";" > "$filename"
echo "key \"${key}\";" >> "$filename"
echo "remote \"${address}\" port ${port};" >> "$filename"
echo "" >> "$filename"
echo "float yes;" >> "$filename"

Und anstatt der Wand aus echos, bietet sich nen here-document an:

cat << EOF > "/etc/fastd/fff/peers/$servername"
#name  "${servername}";
key    "${key}";
remote "${address}" port ${port};

float yes;
EOF
Also ich finde ja erst mal alle Daten zusammensuchen und dann die Datei schreiben etwas übersichtlicher ```sh json_get_var servername name json_get_var key key json_get_var address address json_get_var port port filename="/etc/fastd/fff/peers/$servername" echo "#name \"${servername}\";" > "$filename" echo "key \"${key}\";" >> "$filename" echo "remote \"${address}\" port ${port};" >> "$filename" echo "" >> "$filename" echo "float yes;" >> "$filename" ``` Und anstatt der Wand aus `echo`s, bietet sich nen `here-document` an: ```sh cat << EOF > "/etc/fastd/fff/peers/$servername" #name "${servername}"; key "${key}"; remote "${address}" port ${port}; float yes; EOF
Author
Member

Das ist erst mal nur 1:1 das alte fastd Geraffel. Da kam auch schon ein PR #13 von Adrian um das schoener zu machen. Das ist dann etwas fuer eigene Patches. Entweder hier oder in der alten vpn-select.

Das ist erst mal nur 1:1 das alte fastd Geraffel. Da kam auch schon ein PR #13 von Adrian um das schoener zu machen. Das ist dann etwas fuer eigene Patches. Entweder hier oder in der alten vpn-select.
Owner

Ja, habs dann auch gemerkt, dass das aus dem alten kopiert war und mich erinnert, dass da die Vorschläge kamen.

Also das gehoert verbessert, kann aber freilich später passieren.

Alternativ könnte man auch schnell den alten Code verbessern und dann den kopieren :)

Ja, habs dann auch gemerkt, dass das aus dem alten kopiert war und mich erinnert, dass da die Vorschläge kamen. Also das gehoert verbessert, kann aber freilich später passieren. Alternativ könnte man auch schnell den alten Code verbessern und dann den kopieren :)
jkimmel marked this conversation as resolved
rohammer force-pushed vxlan from e1dc6c1874 to b8898c9c2e 2021-01-16 15:26:20 +01:00 Compare
rohammer force-pushed vxlan from b8898c9c2e to 1a7640f32a 2021-01-16 15:30:55 +01:00 Compare
adschm reviewed 2021-01-19 16:37:12 +01:00
@ -0,0 +3,4 @@
PKG_NAME:=fff-vxlan-node-vpn
PKG_RELEASE:=1
PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
Owner

This line can be dropped.

This line can be dropped.
rohammer marked this conversation as resolved
adschm added the
node
packages/fff
labels 2021-01-19 16:37:34 +01:00
rohammer force-pushed vxlan from 1a7640f32a to f6f1df30b4 2021-01-19 16:58:12 +01:00 Compare
rohammer force-pushed vxlan from f6f1df30b4 to d68115a3ae 2021-01-27 22:12:01 +01:00 Compare
rohammer force-pushed vxlan from d68115a3ae to 516e81e5be 2021-01-29 22:31:21 +01:00 Compare
Author
Member

push: hatte set network.vxlan0.vid vergessen

push: hatte set network.vxlan0.vid vergessen
rohammer force-pushed vxlan from b1529289e9 to 2f6023f1a0 2021-01-30 01:01:02 +01:00 Compare
Author
Member

typo

typo
rohammer force-pushed vxlan from 2f6023f1a0 to 6156389626 2021-11-27 23:17:55 +01:00 Compare
rohammer force-pushed vxlan from 6156389626 to fbb44f320d 2021-12-04 14:57:41 +01:00 Compare
fbl added this to the 20220405-beta milestone 2021-12-21 14:48:20 +01:00
rohammer force-pushed vxlan from 20a789d5f7 to 3925d587d6 2021-12-22 11:32:07 +01:00 Compare
Owner

Sieht gut aus. Schön wäre noch, wenn wir das protocol gegen die supported_protocols prüfen würden. Kann irgendwann mal noch nachgereicht werden.

Für das modularisieren:
Reviewed-by: Fabian Bläse <fabian@blaese.de>

Für vxlan:
Acked-by: Fabian Bläse <fabian@blaese.de>

Sieht gut aus. Schön wäre noch, wenn wir das protocol gegen die supported_protocols prüfen würden. Kann irgendwann mal noch nachgereicht werden. Für das modularisieren: `Reviewed-by: Fabian Bläse <fabian@blaese.de>` Für vxlan: `Acked-by: Fabian Bläse <fabian@blaese.de>`
rohammer force-pushed vxlan from 3925d587d6 to 8a8837cf77 2021-12-30 16:50:50 +01:00 Compare
Owner

Letzten Stand ohne den supported-protocols Commit auf meinen staging tree applied.

Letzten Stand ohne den supported-protocols Commit auf meinen staging tree applied.
fbl closed this pull request 2021-12-30 16:57:39 +01:00

Pull request closed

Sign in to join this conversation.
No description provided.