[PATCH 3/4] nodewatcher: Only include subscripts result if executed successfully

[mail at adrianschmutzler.de]

Hallo,

wenn das Monitoring unvollständige Daten kriegt, ist das sehr eklig. Außerdem führt das dazu, dass der User u.U. nicht merkt, dass etwas kaputt ist, außer er liest regelmäßig seine Logs.

Eine entsprechende Prüfung macht Sinn, aber dann bitte exit 1, sonst haben wir damit keinen Spaß.

Wenn dann gar nichts mehr ans Monitoring raus geht, ist auch die Wahrscheinlichkeit höher, dass die Person in ihr Log schaut (oder das gesagt bekommt).

Nicht mehr sicher bin ich mir, was das $? macht, wenn davor eine Zuweisung gemacht wird. Die Zuweisung ist u.U. ja immer true?

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Sonntag, 19. April 2020 15:40
> To: franken-dev at freifunk.net
> Subject: [PATCH 3/4] nodewatcher: Only include subscripts result if executed
> successfully
> 
> If a subscript exits with an exit status != 0, its output should not be used.
> Therefore, it is discarded.
> 
> Other scripts should not be affected by this, so the execution of
> nodewatcher is continued.
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
>  src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> index 1226f49..f7b6778 100755
> --- a/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> +++ b/src/packages/fff/fff-nodewatcher/files/usr/sbin/nodewatcher
> @@ -29,6 +29,10 @@ crawl() {
> 
>  	for f in /usr/lib/nodewatcher.d/*.sh; do
>  		tmp="$($f)"
> +		if [ $? -ne 0 ]; then
> +			debug "Error when executing subscript $1, discarding
> its result!"
> +			continue
> +		fi
>  		DATA="$DATA$tmp"
>  	done
> 
> --
> 2.26.0
-------------- 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/20200419/84a43dfe/attachment.sig>