Avoid caching when API call fails

agners · agners · commit 2557939fd6b8 · 2025-04-17T17:22:16.000+02:00
Check the return code of the bash function call and only store the
result in the cache if the call was successful. This avoids caching
empty values.
diff --git a/lib/info.sh b/lib/info.sh
@@ -31,6 +31,10 @@ function bashio::info() {
         info=$(bashio::cache.get 'info')
     else
         info=$(bashio::api.supervisor GET /info false)
+        if [ "$?" -ne "${__BASHIO_EXIT_OK}" ]; then
+            bashio::log.error "Failed to get info from Supervisor API"
+            return "${__BASHIO_EXIT_NOK}"
+        fi
         bashio::cache.set 'info' "${info}"
     fi
 
