[PATCH] fastd: make secret key updatesafe

[Fabian Bläse]

Hey Christian,

On 04.01.20 18:28, Christian Dresel wrote:
> To use a whitelist easy, it is neccessary to make the fastd key updatesafe
> This patch safe the key to uci fff and recover it, if a key is after the update available
> 
> Signed-off-by: Christian Dresel <fff at chrisi01.de>
> ---
>  .../fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd         | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd b/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> index d53eb43..4b03229 100644
> --- a/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> +++ b/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
> @@ -15,9 +15,18 @@ uci batch <<EOF
>    set fastd.fff.mtu='1426'
>    set fastd.fff.on_up="/etc/fastd/fff/up.sh"
>    set fastd.fff.secure_handshakes='0'
> -  set fastd.fff.secret="generate"
>  EOF
>  
> +if uci -q get fff.fastd.secret; then
> +	uci set fastd.fff.secret=$(uci get fff.fastd.secret)
Man könnte das hier genau wie in den ganzen gateway.d files direkt beim Lesen fürs if in eine Variable schreiben.
Das macht einerseits den ansonsten möglicherweise irritierenden Output weg und spart eine Prozessausführung.

> +else
> +	secret=$(/usr/bin/fastd --generate-key --machine-readable)
> +	uci -q set fastd.fff.secret="$secret"
Warum -q?
Wenn dieses set nicht klappt, wäre eine Fehlermeldung eine gute Sache.. Weiter unten beim set und commit genauso.

> +	uci set fff.fastd='fff'
> +	uci -q set fff.fastd.secret="$secret" && uci -q commit fff	
trailing whitespace.

Ich persönlich würde die commits im uci-defaults weglassen, aber das hat Adrian ja schon in Erfahrung gebracht, dass das eher Geschmackssache ist.

Gruß
Fabian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200104/8891f380/attachment.sig>