Fix Emby 4.9 API breakage in session report — update activity types and Items UserId requirement; fix local outside function in cert_monitor

Monitors/cert_monitor.sh

@@ -263,7 +263,7 @@ echo ""
 # Per-domain results — problems always shown with days remaining; healthy only with --log
 for domain in "${CERT_MONITOR_DOMAINS[@]}"; do
     [[ -z "$domain" ]] && continue
-    local _days="${DOMAIN_DAYS[$domain]:-?}" _exp="${DOMAIN_EXPIRY[$domain]:-unknown}"
+    _days="${DOMAIN_DAYS[$domain]:-?}" _exp="${DOMAIN_EXPIRY[$domain]:-unknown}"
     case "${DOMAIN_STATUS[$domain]:-UNKN}" in
         OK)   log  "  $ICON_SUCCESS $domain — healthy (${_days}d, expires ${_exp})" ;;
         WARN) warn "  $ICON_WARN   $domain — warning (${_days}d, expires ${_exp})" ;;

Monitors/emby_session_report.sh

@@ -257,31 +257,28 @@ REPORT_START=$(date -d "${EMBY_REPORT_DAYS} days ago" '+%Y-%m-%dT00:00:00.000Z')
 ACTIVITY=$(emby_api "System/ActivityLog/Entries?MinDate=${REPORT_START}&Limit=1000" \
     2>/dev/null) || { warn "Could not fetch activity log"; ACTIVITY="{}"; }
 
+# Emby 4.9 event types changed: VideoPlayback → playback.stop (completed session)
 TOTAL_PLAYS=$(echo "$ACTIVITY" | \
-    jq '[.Items // [] | .[] | select(.Type == "VideoPlayback" or .Type == "AudioPlayback")] | length' \
-    2>/dev/null || echo 0)
-
-TRANSCODE_PLAYS=$(echo "$ACTIVITY" | \
-    jq '[.Items // [] | .[] | select(.Type == "VideoPlaybackUnplugged" or
-        (.Type == "VideoPlayback" and (.Overview // "" | contains("Transcode"))))] | length' \
+    jq '[.Items // [] | .[] | select(.Type == "playback.stop")] | length' \
     2>/dev/null || echo 0)
 
 echo "  $ICON_EMBY Total play events:  $TOTAL_PLAYS"
-
-if [[ "$TOTAL_PLAYS" -gt 0 ]]; then
-    TRANSCODE_PCT=$(awk "BEGIN {printf \"%.0f\", ($TRANSCODE_PLAYS / $TOTAL_PLAYS) * 100}")
-    DIRECT_PCT=$(( 100 - TRANSCODE_PCT ))
-    echo "  $ICON_EMBY Direct play:         ~${DIRECT_PCT}%"
-    echo "  $ICON_EMBY Transcoded:          ~${TRANSCODE_PCT}%"
-    log "$ICON_EMBY Activity: $TOTAL_PLAYS plays — ~${DIRECT_PCT}% direct / ~${TRANSCODE_PCT}% transcode"
-fi
+[[ "$TOTAL_PLAYS" -gt 0 ]] && log "$ICON_EMBY Activity: $TOTAL_PLAYS plays"
 echo ""
 
 # ── Top Content ───────────────────────────────────────────────────────────────────────────────
 echo "━━━ $ICON_EMBY Top ${EMBY_REPORT_TOP_N} Content ━━━"
 
-TOP_ITEMS=$(emby_api "Items?SortBy=DatePlayed&SortOrder=Descending&Limit=${EMBY_REPORT_TOP_N}&Recursive=true&Fields=Overview&IncludeItemTypes=Movie,Episode" \
+# Emby 4.9 requires a UserId on the Items endpoint — fetch admin ID first
+EMBY_ADMIN_ID=$(emby_api "Users" 2>/dev/null | \
+    jq -r '[.[] | select(.Policy.IsAdministrator == true)] | first | .Id // empty' 2>/dev/null)
+if [[ -z "$EMBY_ADMIN_ID" ]]; then
+    warn "Could not resolve admin UserId — skipping top content"
+    TOP_ITEMS="{}"
+else
+    TOP_ITEMS=$(emby_api "Users/${EMBY_ADMIN_ID}/Items?SortBy=DatePlayed&SortOrder=Descending&Limit=${EMBY_REPORT_TOP_N}&Recursive=true&Fields=Overview&IncludeItemTypes=Movie,Episode" \
         2>/dev/null) || { warn "Could not fetch top content"; TOP_ITEMS="{}"; }
+fi
 
 TOP_COUNT=$(echo "$TOP_ITEMS" | jq '.Items // [] | length' 2>/dev/null || echo 0)
 if [[ "$TOP_COUNT" -gt 0 ]]; then

Watchdogs/docker_watchdog.sh

@@ -904,9 +904,13 @@ CYCLE_START=$(date +%s)
                 is_skipped "$container" && continue
                 RESTART_COUNT=$(timeout "$DOCKER_TIMEOUT" docker inspect -f \
                     '{{.RestartCount}}' "$container" 2>/dev/null || echo 0)
+                EXIT_CODE=$(timeout "$DOCKER_TIMEOUT" docker inspect -f \
+                    '{{.State.ExitCode}}' "$container" 2>/dev/null || echo 0)
                 PREV_COUNT=$(grep "^${container}_docker:" "$WATCHDOG_STATE_FILE" \
                     2>/dev/null | cut -d: -f2 || echo 0)
                 set_strikes "${container}_docker" "$RESTART_COUNT" "$WATCHDOG_STATE_FILE"
+                # Skip clean-exit containers (exit 0 = completed, not crashed)
+                [[ "$EXIT_CODE" == "0" ]] && continue
                 if [[ "$RESTART_COUNT" -gt "$PREV_COUNT" && "$RESTART_COUNT" -gt 0 ]]; then
                     ((T2_WARNINGS++))
                     if [[ "$RESTART_COUNT" -ge "$WATCHDOG_CRASH_LIMIT" ]]; then