[PATCH] fff-wireguard: rename public_key to public_remote_key
Adrian Schmutzler
mail at adrianschmutzler.de
Mo Aug 19 15:20:34 CEST 2019
Man könnte auch noch über peer_public_key nachdenken.
Und dann müsste man eigentlich auch local_private_key nehmen.
Finde den Patch aber gut, das hat beim ersten Gateway-Setup die Leute immer verwirrt.
Grüße
Adrian
From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of Alexander Gutzeit
Sent: Sonntag, 18. August 2019 09:20
To: Fabian Bläse <fabian at blaese.de>
Cc: Ich <alexander.gutzeit at web.de>; franken-dev <franken-dev at freifunk.net>
Subject: Re: [PATCH] fff-wireguard: rename public_key to public_remote_key
Hi Fabian, :)
Fabian Bläse <fabian at blaese.de <mailto:fabian at blaese.de> > schrieb am Sa., 17. Aug. 2019, 22:34:
Hey Alex,
gute Idee. Mehr solche Patches, solange wir noch nicht an bestimmten Variablennamen fest hängen!
Ich würde die Variable remote_public_key nennen.
Finde ich auch sehr gut, wenn nicht sogar besser. :):)
Irgendwas ist aber bei deiner E-Mail aber schief gegangen, da ist bisschen "3D"-Zeichensalat drin.
Tjooooo, ich bin ja froh, dass meine Mail überhaupt irgendwie raus gegangen ist. ;)
Verbesserungen oder Neuversuche gibt's wenn ich genug Zeit hab meine History nachzuvollziehen oder ich wieder mit nemm "Eingeweihten" zusammen sitze. ;)
LG Alex. ;)
Gruß
Fabian
On 17.08.19 00:36, Alexander Gutzeit wrote:
> From: Alexander Gutzeit <alexgutfried at gmail.com <mailto:alexgutfried at gmail.com> >
>
> public_key could be confusing which public key should be used.
> public_remote_key should be more clearly.
>
> Signed-off-by: Alexander Gutzeit <alexander.gutzeit at web.de <mailto:alexander.gutzeit at web.de> >
> =2D--
> .../fff/fff-wireguard/files/etc/gateway.d/50-wireguard | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/packages/fff/fff-wireguard/files/etc/gateway.d/50-wiregua=
> rd b/src/packages/fff/fff-wireguard/files/etc/gateway.d/50-wireguard
> index b2e876d..4df3030 100644
> =2D-- a/src/packages/fff/fff-wireguard/files/etc/gateway.d/50-wireguard
> +++ b/src/packages/fff/fff-wireguard/files/etc/gateway.d/50-wireguard
> @@ -64,7 +64,7 @@ configure() {
> uci set gateway.$name.private_key=3D"$privkey"
> fi
>
> - if ! pubkey=3D$(uci get gateway.$name.public_key); then
> + if ! pubkey=3D$(uci get gateway.$name.public_remote_key); then
> echo "ERROR: publickey for ${name} missing!"
> exit 1
> fi
> @@ -103,7 +103,7 @@ configure() {
> cfg=3D$(uci add network wireguard_$prefixname)
> fi
>
> - uci set network.$cfg.public_key=3D"$pubkey"
> + uci set network.$cfg.public_remote_key=3D"$pubkey"
> uci set network.$cfg.endpoint_host=3D"$endpoint_host"
> uci set network.$cfg.endpoint_port=3D"$endpoint_port"
> uci set network.$cfg.persistent_keepalive=3D"$persistent_keepalive"
> =2D-
> 2.17.1
>
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190819/4dd81ad2/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : openpgp-digital-signature.asc
Dateityp : application/pgp-signature
Dateigröße : 834 bytes
Beschreibung: nicht verfügbar
URL : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190819/4dd81ad2/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev