various improvement and securisation

profile.d/info.sh

@@ -39,6 +39,9 @@
 # ------------------------------------------------------------------------------
 ver()
 {
+    [[ -z $PROFVERSION ]] && \
+        disp W "No version defined. Profile is probably badly installed." && \
+        return 1
     disp "Profile version $PROFVERSION."
 }
 export -f ver
@@ -48,11 +51,13 @@ export -f ver
 # ------------------------------------------------------------------------------
 meteo()
 {
-    cities=$@
-    [[ $# -eq 0 ]] && local cities=$DEFAULT_CITY
+    local encoded cities=("$@")
+    [[ $# -eq 0 ]] && cities=("$DEFAULT_CITY")
 
-    for city in $cities; do
-        curl https://wttr.in/$city || disp E "Failed fetching datas for $city."
+    for city in "${cities[@]}"; do
+        encoded=$(urlencode "$city")
+        curl -s "https://wttr.in/$encoded" || \
+            disp E "Failed fetching datas for $city."
     done
 }
 export -f meteo
@@ -67,9 +72,13 @@ showinfo()
         if [[ -s /usr/share/figlet/ansi_shadow.flf ]]; then
             local figopt="-f ansi_shadow"
         fi
-        figlet -k $(hostname) $figopt
+        if [[ -n $figopt ]]; then
+            figlet -k "$figopt" "$(hostname)"
+        else
+            figlet "$(hostname)"
+        fi
     else
-        echo "$(hostname -f)"
+        hostname -f
     fi
     echo ""
     if command -v neofetch >/dev/null 2>&1; then
@@ -77,12 +86,13 @@ showinfo()
     else
         (
             if [[ -s /etc/os-release ]]; then
+                # shellcheck disable=SC1091
                 . /etc/os-release
                 echo "$NAME $VERSION"
             else
                 cat /proc/version
             fi
-            echo "Uptime: $(uptime)"
+            echo "Uptime: $(uptime -p)"
         )
     fi
 }