[PATCH] fff-wireguard: rename public_key to public_remote_key

[Adrian Schmutzler]

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>