[PATCH v2] fff-gateway: Add sanity checks

[Adrian Schmutzler]

Hallo,

um das nochmal etwas zu spezifizieren, ich würde folgendes machen:

config gateway 'meta'
	option config_version '1'

Ansonsten noch ein Kommentar/Frage:

> > +	if ! uci -q get gateway; then

Mir war gar nicht bewusst, dass das geht (aber ja, habs getestet). Ich hätte da jetzt ! [ -s "/etc/config/gateway" ] gemacht.

Da die von mir genannte Variante an diversen Stellen (für andere uci config Files) verwendet wird, würde mich interessieren, was ihr für besser haltet (ist ja auch ein bisschen Speicher vs. Dateisystem).
Zudem frage ich mich, ob die Syntax von Fabian absichtlich oder aus Versehen funktioniert, da sie nichts ins std-out schreibt.

Wenn Fabian eine v3 schickt, gibt’s ein Review.

Grüße

Adrian


> > +		echo "ERROR: Gateway config does not exists."
> > +
> > +		exit 1
> > +	fi
> > +
> > +	# check version of configuration
> > +	if [ "$config_version" != "$expected_version" ]; then
> > +		echo "ERROR: Invalid config version. Expected '$expected_version', got '$config_version'."
> > +		echo "Please check what has been changed and adjust your config appropriately."
> > +
> > +		exit 1
> > +	fi
> > +}
-------------- 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/e2098c6d/attachment.sig>