favicons

pytorrent/routes/api.py

@@ -508,15 +508,33 @@ def trackers_summary():
 
 
 @bp.get("/trackers/favicon/<path:domain>")
+@bp.get("/tracker-favicon/<path:domain>")
 def tracker_favicon(domain: str):
     prefs = preferences.get_preferences()
-    enabled = bool(prefs and prefs.get("tracker_favicons_enabled"))
     force = str(request.args.get("refresh") or "").lower() in {"1", "true", "yes", "force"}
+    # Note: Manual refresh must work from CLI even when tracker favicons are disabled in Preferences.
+    enabled = force or bool(prefs and prefs.get("tracker_favicons_enabled"))
     static_url = tracker_cache.favicon_public_url(domain, enabled=enabled, create=True, force=force)
     if static_url:
         # Note: The API only discovers/cache-warms the icon; the browser receives the file from /static/tracker_favicons/.
         return redirect(static_url, code=302)
-    abort(404)
+    cached = tracker_cache.favicon_cache_row(domain)
+    return jsonify({
+        "ok": False,
+        "error": "favicon not found",
+        "domain": tracker_cache.tracker_domain(domain),
+        "enabled": bool(enabled),
+        "cached_error": (cached or {}).get("error") if cached else None,
+    }), 404
+
+
+@bp.get("/trackers/favicon")
+def tracker_favicon_query():
+    # Note: Query-string alias makes cache warming easier from shell scripts where path routing/proxies may differ.
+    domain = str(request.args.get("domain") or "").strip()
+    if not domain:
+        return jsonify({"ok": False, "error": "domain is required"}), 400
+    return tracker_favicon(domain)
 
 @bp.get("/torrent-stats")
 def torrent_stats_get():