mirror of
https://github.com/ChuckBuilds/LEDMatrix.git
synced 2026-08-10 21:28:05 +00:00
Compare commits
2
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
f8d700e5d9 | ||
|
|
e0d62a9e54 |
@@ -149,7 +149,7 @@
|
|||||||
"min_plugin_width": 8,
|
"min_plugin_width": 8,
|
||||||
"lead_in_width": 0,
|
"lead_in_width": 0,
|
||||||
"plugins_per_cycle": 6,
|
"plugins_per_cycle": 6,
|
||||||
"max_plugin_width_ratio": 3.0,
|
"max_plugin_width_ratio": 0.0,
|
||||||
"overflow_mode": "rotate",
|
"overflow_mode": "rotate",
|
||||||
"dynamic_duration_enabled": true,
|
"dynamic_duration_enabled": true,
|
||||||
"min_cycle_duration": 60,
|
"min_cycle_duration": 60,
|
||||||
|
|||||||
@@ -127,7 +127,7 @@ Read by `src/vegas_mode/config.py` (`VegasScrollConfig.from_config`). See
|
|||||||
| `min_plugin_width` | int, `8` |
|
| `min_plugin_width` | int, `8` |
|
||||||
| `lead_in_width` | int, `0` |
|
| `lead_in_width` | int, `0` |
|
||||||
| `plugins_per_cycle` | int, `6` |
|
| `plugins_per_cycle` | int, `6` |
|
||||||
| `max_plugin_width_ratio` | float, `3.0` |
|
| `max_plugin_width_ratio` | float, `0.0` |
|
||||||
| `overflow_mode` | string, `"rotate"` |
|
| `overflow_mode` | string, `"rotate"` |
|
||||||
| `dynamic_duration_enabled` | bool, `true` |
|
| `dynamic_duration_enabled` | bool, `true` |
|
||||||
| `min_cycle_duration` | int, `60` |
|
| `min_cycle_duration` | int, `60` |
|
||||||
|
|||||||
@@ -104,10 +104,22 @@ class VegasModeConfig:
|
|||||||
overflow_mode: str = "rotate"
|
overflow_mode: str = "rotate"
|
||||||
|
|
||||||
# Cap on one plugin's share of a cycle, as a multiple of display width.
|
# Cap on one plugin's share of a cycle, as a multiple of display width.
|
||||||
# A single ticker returning 7,000px would otherwise hold the panel for over
|
# 0 (the default) disables the cap, so every plugin contributes all of its
|
||||||
# two minutes. Overflow is deferred to later cycles rather than discarded.
|
# content and is always entered at its beginning.
|
||||||
# 0 disables the cap.
|
#
|
||||||
max_plugin_width_ratio: float = 3.0
|
# Capping was the default until it proved to cost more than it bought.
|
||||||
|
# Measured over a 17-plugin fleet on a 512px panel, only four plugins were
|
||||||
|
# ever wide enough to hit a 3.0 cap; for those four it produced two visible
|
||||||
|
# faults. Content resumed mid-item on each appearance (a news ticker entered
|
||||||
|
# at column 6027 of its own strip), and the final window of a rotation was
|
||||||
|
# whatever happened to be left — 348px of a 1840px stocks ticker, seven
|
||||||
|
# seconds of panel time. Both read as the display being broken rather than
|
||||||
|
# as deferral working.
|
||||||
|
#
|
||||||
|
# A wide plugin does hold the panel for a long time uncapped: set the cap
|
||||||
|
# per plugin with vegas_max_width_screens where that matters, rather than
|
||||||
|
# globally where it mostly hurts plugins that were never the problem.
|
||||||
|
max_plugin_width_ratio: float = 0.0
|
||||||
|
|
||||||
# Plugin management
|
# Plugin management
|
||||||
plugin_order: List[str] = field(default_factory=list)
|
plugin_order: List[str] = field(default_factory=list)
|
||||||
@@ -159,7 +171,7 @@ class VegasModeConfig:
|
|||||||
lead_in_width=int(vegas_config.get('lead_in_width', 0)),
|
lead_in_width=int(vegas_config.get('lead_in_width', 0)),
|
||||||
plugins_per_cycle=int(vegas_config.get('plugins_per_cycle', 6)),
|
plugins_per_cycle=int(vegas_config.get('plugins_per_cycle', 6)),
|
||||||
max_plugin_width_ratio=float(
|
max_plugin_width_ratio=float(
|
||||||
vegas_config.get('max_plugin_width_ratio', 3.0)),
|
vegas_config.get('max_plugin_width_ratio', 0.0)),
|
||||||
overflow_mode=str(vegas_config.get('overflow_mode', 'rotate')),
|
overflow_mode=str(vegas_config.get('overflow_mode', 'rotate')),
|
||||||
plugin_order=list(vegas_config.get('plugin_order', [])),
|
plugin_order=list(vegas_config.get('plugin_order', [])),
|
||||||
excluded_plugins=set(vegas_config.get('excluded_plugins', [])),
|
excluded_plugins=set(vegas_config.get('excluded_plugins', [])),
|
||||||
|
|||||||
@@ -4,7 +4,6 @@ Centralized error handling for web interface.
|
|||||||
Provides helpers for consistent error responses across API endpoints.
|
Provides helpers for consistent error responses across API endpoints.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
import re
|
|
||||||
from typing import Any, Optional
|
from typing import Any, Optional
|
||||||
from flask import jsonify
|
from flask import jsonify
|
||||||
|
|
||||||
@@ -17,74 +16,6 @@ from src.logging_config import get_logger
|
|||||||
logger = get_logger(__name__)
|
logger = get_logger(__name__)
|
||||||
|
|
||||||
|
|
||||||
# Credentials that turn up inside exception text. A requests error quotes the
|
|
||||||
# URL it failed on, and plugins that authenticate by query string put their key
|
|
||||||
# there, so echoing an exception verbatim can hand out an API key. Redact the
|
|
||||||
# value, keep the parameter name -- knowing *which* credential was involved is
|
|
||||||
# part of the diagnosis.
|
|
||||||
_REDACT_CREDENTIAL = re.compile(
|
|
||||||
r'((?:api[_-]?key|access[_-]?token|auth|apikey|key|passwd|password|pwd|'
|
|
||||||
r'secret|sig|signature|token)["\']?\s*[=:]\s*["\']?)([^\s&"\'<>,}]+)',
|
|
||||||
re.IGNORECASE,
|
|
||||||
)
|
|
||||||
|
|
||||||
# `Authorization: Bearer <token>`. The scheme is kept because it says which
|
|
||||||
# kind of credential failed; only the token goes. Not covered by the pattern
|
|
||||||
# above, whose value part stops at whitespace and so would keep the token when
|
|
||||||
# a space follows the scheme.
|
|
||||||
_REDACT_AUTH_HEADER = re.compile(
|
|
||||||
r'((?:proxy-)?authorization["\']?\s*[=:]\s*["\']?\s*'
|
|
||||||
r'(?:bearer|basic|digest|token)\s+)([^\s,"\'<>}]+)',
|
|
||||||
re.IGNORECASE,
|
|
||||||
)
|
|
||||||
|
|
||||||
# Credentials embedded in a URL: https://user:password@host. requests quotes
|
|
||||||
# the full URL in its exceptions, so this is a realistic leak. The username is
|
|
||||||
# kept -- it identifies which account failed without being the secret.
|
|
||||||
_REDACT_URL_USERINFO = re.compile(r'([a-z][a-z0-9+.-]*://[^/\s:@]+:)([^/\s@]+)(@)',
|
|
||||||
re.IGNORECASE)
|
|
||||||
|
|
||||||
# Long enough for an errno string with a path, short enough not to dump a
|
|
||||||
# parser's worth of context into a JSON field.
|
|
||||||
_MAX_DETAIL_LENGTH = 400
|
|
||||||
|
|
||||||
|
|
||||||
def describe_exception(exc: BaseException,
|
|
||||||
max_length: int = _MAX_DETAIL_LENGTH) -> str:
|
|
||||||
"""
|
|
||||||
One-line, safe-to-return description of an exception.
|
|
||||||
|
|
||||||
The generic "an error occurred; see logs for details" tells a user nothing
|
|
||||||
and, when the failure is bad enough, the logs are unreachable too: a device
|
|
||||||
whose storage was failing returned that message from every endpoint
|
|
||||||
*including* the log viewer, because journalctl could not be executed. The
|
|
||||||
underlying `[Errno 5] Input/output error` named the fault immediately.
|
|
||||||
|
|
||||||
Returns "TypeName: message", credentials redacted and length capped. The
|
|
||||||
type alone is worth carrying -- a bare PermissionError says more than any
|
|
||||||
generic sentence.
|
|
||||||
|
|
||||||
Args:
|
|
||||||
exc: The exception to describe
|
|
||||||
max_length: Truncate beyond this many characters
|
|
||||||
|
|
||||||
Returns:
|
|
||||||
A single-line description, never empty
|
|
||||||
"""
|
|
||||||
message = str(exc).strip()
|
|
||||||
text = f"{type(exc).__name__}: {message}" if message else type(exc).__name__
|
|
||||||
# Order matters: the URL and header forms are more specific than the
|
|
||||||
# generic key=value pattern, which would otherwise chew the scheme.
|
|
||||||
text = _REDACT_URL_USERINFO.sub(r'\1<redacted>\3', text)
|
|
||||||
text = _REDACT_AUTH_HEADER.sub(r'\1<redacted>', text)
|
|
||||||
text = _REDACT_CREDENTIAL.sub(r'\1<redacted>', text)
|
|
||||||
# Collapse newlines/tabs so the detail stays one line in a JSON field.
|
|
||||||
text = ' '.join(text.split())
|
|
||||||
if len(text) > max_length:
|
|
||||||
text = text[:max_length - 1].rstrip() + '…'
|
|
||||||
return text
|
|
||||||
|
|
||||||
|
|
||||||
def create_error_response(
|
def create_error_response(
|
||||||
error_code: ErrorCode,
|
error_code: ErrorCode,
|
||||||
message: str,
|
message: str,
|
||||||
|
|||||||
@@ -781,6 +781,20 @@ class TestNewConfigKeys:
|
|||||||
assert cfg.render_width_pct == 100
|
assert cfg.render_width_pct == 100
|
||||||
assert cfg.min_content_separation == 24
|
assert cfg.min_content_separation == 24
|
||||||
|
|
||||||
|
def test_width_cap_is_off_by_default(self):
|
||||||
|
# Capping made wide plugins resume mid-content on every appearance and
|
||||||
|
# emit runt final windows; it is now opt-in per plugin instead.
|
||||||
|
assert VegasModeConfig().max_plugin_width_ratio == 0.0
|
||||||
|
assert VegasModeConfig.from_config({}).max_plugin_width_ratio == 0.0
|
||||||
|
|
||||||
|
def test_width_cap_is_still_available_when_asked_for(self):
|
||||||
|
# Defaulting the cap off must not remove it: a user who sets a ratio
|
||||||
|
# still gets one, and 0 still means uncapped.
|
||||||
|
cfg = VegasModeConfig.from_config(
|
||||||
|
{'display': {'vegas_scroll': {'max_plugin_width_ratio': 3.0}}})
|
||||||
|
assert cfg.max_plugin_width_ratio == 3.0
|
||||||
|
assert cfg.validate() == []
|
||||||
|
|
||||||
@pytest.mark.parametrize('overrides,bad_key', [
|
@pytest.mark.parametrize('overrides,bad_key', [
|
||||||
({'render_width_pct': 5}, 'render_width_pct'),
|
({'render_width_pct': 5}, 'render_width_pct'),
|
||||||
({'render_width_pct': 101}, 'render_width_pct'),
|
({'render_width_pct': 101}, 'render_width_pct'),
|
||||||
|
|||||||
@@ -1,222 +0,0 @@
|
|||||||
"""Tests for surfacing the underlying error in web responses.
|
|
||||||
|
|
||||||
Regression under test: every failing endpoint returned "An error occurred; see
|
|
||||||
logs for details" and nothing else. On a device whose storage was failing that
|
|
||||||
sentence came back from the restart action, from /system/status, and from
|
|
||||||
/logs -- the log viewer itself -- because journalctl could not be executed. The
|
|
||||||
exception underneath said `[Errno 5] Input/output error: 'systemctl'`, which
|
|
||||||
names the fault outright, and nine handlers were discarding it entirely rather
|
|
||||||
than even logging it.
|
|
||||||
"""
|
|
||||||
|
|
||||||
import pytest
|
|
||||||
|
|
||||||
from src.web_interface.error_handler import describe_exception
|
|
||||||
|
|
||||||
|
|
||||||
class TestDescribeException:
|
|
||||||
def test_names_the_type_and_message(self):
|
|
||||||
detail = describe_exception(OSError(5, "Input/output error", "systemctl"))
|
|
||||||
assert detail == "OSError: [Errno 5] Input/output error: 'systemctl'"
|
|
||||||
|
|
||||||
def test_the_reported_failure_is_legible(self):
|
|
||||||
# The whole point: this string is the diagnosis.
|
|
||||||
assert "Input/output error" in describe_exception(
|
|
||||||
OSError(5, "Input/output error", "systemctl"))
|
|
||||||
|
|
||||||
def test_a_bare_exception_still_names_its_type(self):
|
|
||||||
# A PermissionError with no message still says more than "unknown".
|
|
||||||
assert describe_exception(PermissionError()) == "PermissionError"
|
|
||||||
assert describe_exception(Exception()) == "Exception"
|
|
||||||
|
|
||||||
def test_message_is_kept_when_present(self):
|
|
||||||
assert describe_exception(ValueError("bad port")) == "ValueError: bad port"
|
|
||||||
|
|
||||||
|
|
||||||
class TestCredentialRedaction:
|
|
||||||
"""Exception text quotes URLs, and plugins authenticate by query string."""
|
|
||||||
|
|
||||||
@pytest.mark.parametrize("secret_text,leaked", [
|
|
||||||
("failed: https://api.x.com/v1?api_key=SEC123&city=Tampa", "SEC123"),
|
|
||||||
("token=abcdef123456 was rejected", "abcdef123456"),
|
|
||||||
("connect failed password=hunter2", "hunter2"),
|
|
||||||
("GET /?access_token=zzz999", "zzz999"),
|
|
||||||
('{"secret": "topsecret"}', "topsecret"),
|
|
||||||
# requests quotes the URL it failed on, and both of these forms turn
|
|
||||||
# up in real client exceptions.
|
|
||||||
("401 for https://user:hunter2@example.com/api", "hunter2"),
|
|
||||||
("headers: {'Authorization': 'Bearer eyJ.SECRET.sig'}", "eyJ.SECRET.sig"),
|
|
||||||
("Authorization: Basic dXNlcjpwYXNzd29yZA==", "dXNlcjpwYXNzd29yZA=="),
|
|
||||||
("Proxy-Authorization: Bearer ptok999", "ptok999"),
|
|
||||||
])
|
|
||||||
def test_credentials_never_reach_the_response(self, secret_text, leaked):
|
|
||||||
detail = describe_exception(RuntimeError(secret_text))
|
|
||||||
assert leaked not in detail
|
|
||||||
assert "<redacted>" in detail
|
|
||||||
|
|
||||||
def test_the_parameter_name_survives_redaction(self):
|
|
||||||
# Knowing *which* credential was involved is part of the diagnosis.
|
|
||||||
detail = describe_exception(RuntimeError("https://x/y?api_key=SEC123"))
|
|
||||||
assert "api_key" in detail
|
|
||||||
|
|
||||||
def test_auth_scheme_and_username_survive(self):
|
|
||||||
# Which kind of credential, and whose, without the credential itself.
|
|
||||||
assert "Bearer" in describe_exception(
|
|
||||||
RuntimeError("Authorization: Bearer eyJ.SECRET.sig"))
|
|
||||||
assert "user" in describe_exception(
|
|
||||||
RuntimeError("https://user:hunter2@example.com"))
|
|
||||||
|
|
||||||
def test_non_secret_context_is_preserved(self):
|
|
||||||
detail = describe_exception(RuntimeError("https://api.x.com/v1?city=Tampa"))
|
|
||||||
assert "city=Tampa" in detail
|
|
||||||
assert "<redacted>" not in detail
|
|
||||||
|
|
||||||
|
|
||||||
class TestBounds:
|
|
||||||
def test_long_messages_are_truncated(self):
|
|
||||||
detail = describe_exception(ValueError("x" * 5000))
|
|
||||||
assert len(detail) <= 400
|
|
||||||
|
|
||||||
def test_newlines_are_collapsed_to_one_line(self):
|
|
||||||
detail = describe_exception(ValueError("line one\nline two\tthree"))
|
|
||||||
assert "\n" not in detail and "\t" not in detail
|
|
||||||
assert detail == "ValueError: line one line two three"
|
|
||||||
|
|
||||||
def test_custom_length_is_honoured(self):
|
|
||||||
assert len(describe_exception(ValueError("y" * 500), max_length=50)) <= 50
|
|
||||||
|
|
||||||
|
|
||||||
class TestHandlersCarryDetail:
|
|
||||||
"""The response shape callers actually see."""
|
|
||||||
|
|
||||||
def test_no_api_v3_handler_discards_its_exception(self):
|
|
||||||
"""Every generic-message handler must log a traceback and return detail.
|
|
||||||
|
|
||||||
Nine of them bound `e` and never used it, so the promised log entry was
|
|
||||||
never written either. Checking merely that *something* was logged is
|
|
||||||
too weak -- a `logger.info("failed")` would satisfy it while throwing
|
|
||||||
the exception away just as completely, so this asserts the two things
|
|
||||||
that actually make the failure diagnosable: an error-level record with
|
|
||||||
the traceback, and the sanitized detail in the response.
|
|
||||||
"""
|
|
||||||
import ast
|
|
||||||
|
|
||||||
src = open("web_interface/blueprints/api_v3.py").read()
|
|
||||||
tree = ast.parse(src)
|
|
||||||
generic = "An error occurred; see logs for details"
|
|
||||||
|
|
||||||
def logs_a_traceback(handler):
|
|
||||||
"""An error/exception-level log call carrying exc_info."""
|
|
||||||
for call in [n for n in ast.walk(handler) if isinstance(n, ast.Call)]:
|
|
||||||
func = call.func
|
|
||||||
if not isinstance(func, ast.Attribute):
|
|
||||||
continue
|
|
||||||
if func.attr == "exception": # implies exc_info
|
|
||||||
return True
|
|
||||||
if func.attr not in ("error", "critical"):
|
|
||||||
continue
|
|
||||||
if any(kw.arg == "exc_info" and getattr(kw.value, "value", False) is True
|
|
||||||
for kw in call.keywords):
|
|
||||||
return True
|
|
||||||
return False
|
|
||||||
|
|
||||||
def returns_the_detail(handler):
|
|
||||||
"""describe_exception() called on this handler's bound exception."""
|
|
||||||
for call in [n for n in ast.walk(handler) if isinstance(n, ast.Call)]:
|
|
||||||
name = call.func.id if isinstance(call.func, ast.Name) else None
|
|
||||||
if name != "describe_exception":
|
|
||||||
continue
|
|
||||||
if handler.name is None:
|
|
||||||
return True # bare `except:` cannot name it; accept
|
|
||||||
if any(isinstance(a, ast.Name) and a.id == handler.name
|
|
||||||
for a in call.args):
|
|
||||||
return True
|
|
||||||
return False
|
|
||||||
|
|
||||||
offenders = []
|
|
||||||
for h in [n for n in ast.walk(tree) if isinstance(n, ast.ExceptHandler)]:
|
|
||||||
seg = ast.get_source_segment(src, h) or ""
|
|
||||||
if generic not in seg:
|
|
||||||
continue
|
|
||||||
missing = []
|
|
||||||
if not logs_a_traceback(h):
|
|
||||||
missing.append("error-level log with exc_info")
|
|
||||||
if not returns_the_detail(h):
|
|
||||||
missing.append("describe_exception(e) in the response")
|
|
||||||
if missing:
|
|
||||||
offenders.append((h.lineno, missing))
|
|
||||||
|
|
||||||
assert not offenders, (
|
|
||||||
"handlers returning the generic message without %s: %r"
|
|
||||||
% ("both a traceback log and the detail", offenders))
|
|
||||||
|
|
||||||
def test_client_errors_keep_their_own_status(self):
|
|
||||||
"""A 405 must not be reported as a server-side UNKNOWN_ERROR.
|
|
||||||
|
|
||||||
Werkzeug's HTTPExceptions subclass Exception, so the catch-all saw them
|
|
||||||
too: a GET on a POST-only route came back 500 "an error occurred",
|
|
||||||
which tells the caller nothing and blames the wrong side. Found while
|
|
||||||
probing a device whose POST-only config endpoints answered every GET
|
|
||||||
with UNKNOWN_ERROR.
|
|
||||||
"""
|
|
||||||
from flask import Flask, jsonify
|
|
||||||
from werkzeug.exceptions import HTTPException
|
|
||||||
|
|
||||||
app = Flask(__name__)
|
|
||||||
|
|
||||||
@app.errorhandler(Exception)
|
|
||||||
def handle(error):
|
|
||||||
if isinstance(error, HTTPException):
|
|
||||||
return jsonify({
|
|
||||||
"status": "error",
|
|
||||||
"error_code": (error.name or "HTTP_ERROR").upper().replace(" ", "_"),
|
|
||||||
"message": error.description,
|
|
||||||
}), error.code or 500
|
|
||||||
return jsonify({
|
|
||||||
"status": "error",
|
|
||||||
"error_code": "UNKNOWN_ERROR",
|
|
||||||
"message": "An error occurred; see logs for details",
|
|
||||||
"details": describe_exception(error),
|
|
||||||
}), 500
|
|
||||||
|
|
||||||
@app.route("/only-post", methods=["POST"])
|
|
||||||
def only_post():
|
|
||||||
return jsonify({"ok": True})
|
|
||||||
|
|
||||||
@app.route("/boom")
|
|
||||||
def boom():
|
|
||||||
raise OSError(5, "Input/output error", "systemctl")
|
|
||||||
|
|
||||||
client = app.test_client()
|
|
||||||
|
|
||||||
resp = client.get("/only-post")
|
|
||||||
assert resp.status_code == 405, "a wrong method must stay a 405"
|
|
||||||
assert resp.get_json()["error_code"] == "METHOD_NOT_ALLOWED"
|
|
||||||
|
|
||||||
# A genuine server fault still reports as one, with its detail.
|
|
||||||
resp = client.get("/boom")
|
|
||||||
assert resp.status_code == 500
|
|
||||||
assert "Input/output error" in resp.get_json()["details"]
|
|
||||||
|
|
||||||
def test_global_handler_reports_the_underlying_error(self):
|
|
||||||
from flask import Flask, jsonify
|
|
||||||
|
|
||||||
app = Flask(__name__)
|
|
||||||
|
|
||||||
@app.errorhandler(Exception)
|
|
||||||
def handle(error):
|
|
||||||
return jsonify({
|
|
||||||
"status": "error",
|
|
||||||
"error_code": "UNKNOWN_ERROR",
|
|
||||||
"message": "An error occurred; see logs for details",
|
|
||||||
"details": describe_exception(error),
|
|
||||||
}), 500
|
|
||||||
|
|
||||||
@app.route("/boom")
|
|
||||||
def boom():
|
|
||||||
raise OSError(5, "Input/output error", "systemctl")
|
|
||||||
|
|
||||||
client = app.test_client()
|
|
||||||
body = client.get("/boom").get_json()
|
|
||||||
assert body["error_code"] == "UNKNOWN_ERROR"
|
|
||||||
assert "Input/output error" in body["details"]
|
|
||||||
+3
-33
@@ -16,8 +16,6 @@ from datetime import datetime, timedelta
|
|||||||
sys.path.insert(0, str(Path(__file__).parent.parent))
|
sys.path.insert(0, str(Path(__file__).parent.parent))
|
||||||
|
|
||||||
from src.config_manager import ConfigManager
|
from src.config_manager import ConfigManager
|
||||||
from src.web_interface.error_handler import describe_exception
|
|
||||||
from werkzeug.exceptions import HTTPException
|
|
||||||
from src.exceptions import ConfigError
|
from src.exceptions import ConfigError
|
||||||
from src.plugin_system.plugin_manager import PluginManager
|
from src.plugin_system.plugin_manager import PluginManager
|
||||||
from src.plugin_system.store_manager import PluginStoreManager
|
from src.plugin_system.store_manager import PluginStoreManager
|
||||||
@@ -393,42 +391,15 @@ def internal_error(error):
|
|||||||
import logging
|
import logging
|
||||||
logger = logging.getLogger('web_interface')
|
logger = logging.getLogger('web_interface')
|
||||||
logger.error("Internal server error", exc_info=True)
|
logger.error("Internal server error", exc_info=True)
|
||||||
payload = {
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'error_code': 'INTERNAL_ERROR',
|
'error_code': 'INTERNAL_ERROR',
|
||||||
'message': 'An internal error occurred; see logs for details',
|
'message': 'An internal error occurred; see logs for details',
|
||||||
}
|
}), 500
|
||||||
# Flask hands the original exception over as `error.original_exception`
|
|
||||||
# when propagation is off; without it there is nothing to describe.
|
|
||||||
original = getattr(error, 'original_exception', None) or (
|
|
||||||
error if isinstance(error, BaseException) else None)
|
|
||||||
if original is not None:
|
|
||||||
payload['details'] = describe_exception(original)
|
|
||||||
return jsonify(payload), 500
|
|
||||||
|
|
||||||
@app.errorhandler(Exception)
|
@app.errorhandler(Exception)
|
||||||
def handle_exception(error):
|
def handle_exception(error):
|
||||||
"""Handle all unhandled exceptions.
|
"""Handle all unhandled exceptions."""
|
||||||
|
|
||||||
Returning only "see logs for details" is fine until the logs are exactly
|
|
||||||
what you cannot reach. A device with failing storage answered every
|
|
||||||
endpoint with that sentence -- including the log viewer, because journalctl
|
|
||||||
could not be executed -- while the exception underneath said
|
|
||||||
`[Errno 5] Input/output error`. Naming the error costs nothing here and is
|
|
||||||
frequently the whole diagnosis, so include it alongside the log pointer.
|
|
||||||
"""
|
|
||||||
# Werkzeug's HTTPExceptions subclass Exception, so this catch-all sees
|
|
||||||
# them too and was reporting every 405, 400, 413 and 415 as a server-side
|
|
||||||
# UNKNOWN_ERROR 500. A GET on a POST-only route came back as "an error
|
|
||||||
# occurred" rather than "method not allowed", which tells the caller
|
|
||||||
# nothing and blames the wrong side. Hand those back as themselves.
|
|
||||||
if isinstance(error, HTTPException):
|
|
||||||
return jsonify({
|
|
||||||
'status': 'error',
|
|
||||||
'error_code': (error.name or 'HTTP_ERROR').upper().replace(' ', '_'),
|
|
||||||
'message': error.description,
|
|
||||||
}), error.code or 500
|
|
||||||
|
|
||||||
import logging
|
import logging
|
||||||
logger = logging.getLogger('web_interface')
|
logger = logging.getLogger('web_interface')
|
||||||
logger.error("Unhandled exception", exc_info=True)
|
logger.error("Unhandled exception", exc_info=True)
|
||||||
@@ -436,7 +407,6 @@ def handle_exception(error):
|
|||||||
'status': 'error',
|
'status': 'error',
|
||||||
'error_code': 'UNKNOWN_ERROR',
|
'error_code': 'UNKNOWN_ERROR',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details',
|
||||||
'details': describe_exception(error),
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
# Captive portal redirect middleware
|
# Captive portal redirect middleware
|
||||||
|
|||||||
@@ -21,7 +21,6 @@ logger = logging.getLogger(__name__)
|
|||||||
# Import new infrastructure
|
# Import new infrastructure
|
||||||
from src.web_interface.api_helpers import success_response, error_response, validate_request_json
|
from src.web_interface.api_helpers import success_response, error_response, validate_request_json
|
||||||
from src.web_interface.errors import ErrorCode
|
from src.web_interface.errors import ErrorCode
|
||||||
from src.web_interface.error_handler import describe_exception
|
|
||||||
from src.plugin_system.operation_types import OperationType
|
from src.plugin_system.operation_types import OperationType
|
||||||
from src.web_interface.validators import (
|
from src.web_interface.validators import (
|
||||||
validate_file_upload
|
validate_file_upload
|
||||||
@@ -272,7 +271,7 @@ def get_main_config():
|
|||||||
return jsonify({'status': 'success', 'data': config})
|
return jsonify({'status': 'success', 'data': config})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/config/schedule', methods=['GET'])
|
@api_v3.route('/config/schedule', methods=['GET'])
|
||||||
def get_schedule_config():
|
def get_schedule_config():
|
||||||
@@ -290,11 +289,9 @@ def get_schedule_config():
|
|||||||
|
|
||||||
return success_response(data=schedule_config)
|
return success_response(data=schedule_config)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.CONFIG_LOAD_FAILED,
|
ErrorCode.CONFIG_LOAD_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
details=describe_exception(e),
|
|
||||||
status_code=500
|
status_code=500
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -470,7 +467,7 @@ def save_schedule_config():
|
|||||||
ErrorCode.CONFIG_SAVE_FAILED,
|
ErrorCode.CONFIG_SAVE_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
|
|
||||||
status_code=500, details=describe_exception(e)
|
status_code=500
|
||||||
)
|
)
|
||||||
|
|
||||||
@api_v3.route('/config/dim-schedule', methods=['GET'])
|
@api_v3.route('/config/dim-schedule', methods=['GET'])
|
||||||
@@ -518,14 +515,14 @@ def get_dim_schedule_config():
|
|||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.CONFIG_LOAD_FAILED,
|
ErrorCode.CONFIG_LOAD_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
status_code=500, details=describe_exception(e)
|
status_code=500
|
||||||
)
|
)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logging.error(f"[DIM SCHEDULE] Unexpected error loading config: {e}", exc_info=True)
|
logging.error(f"[DIM SCHEDULE] Unexpected error loading config: {e}", exc_info=True)
|
||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.CONFIG_LOAD_FAILED,
|
ErrorCode.CONFIG_LOAD_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
status_code=500, details=describe_exception(e)
|
status_code=500
|
||||||
)
|
)
|
||||||
|
|
||||||
@api_v3.route('/config/dim-schedule', methods=['POST'])
|
@api_v3.route('/config/dim-schedule', methods=['POST'])
|
||||||
@@ -689,7 +686,7 @@ def save_dim_schedule_config():
|
|||||||
ErrorCode.CONFIG_SAVE_FAILED,
|
ErrorCode.CONFIG_SAVE_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
|
|
||||||
status_code=500, details=describe_exception(e)
|
status_code=500
|
||||||
)
|
)
|
||||||
|
|
||||||
@api_v3.route('/config/main', methods=['POST'])
|
@api_v3.route('/config/main', methods=['POST'])
|
||||||
@@ -1346,7 +1343,7 @@ def save_main_config():
|
|||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.CONFIG_SAVE_FAILED,
|
ErrorCode.CONFIG_SAVE_FAILED,
|
||||||
"An error occurred; see logs for details",
|
"An error occurred; see logs for details",
|
||||||
status_code=500, details=describe_exception(e)
|
status_code=500
|
||||||
)
|
)
|
||||||
|
|
||||||
@api_v3.route('/config/secrets', methods=['GET'])
|
@api_v3.route('/config/secrets', methods=['GET'])
|
||||||
@@ -1360,7 +1357,7 @@ def get_secrets_config():
|
|||||||
return jsonify({'status': 'success', 'data': config})
|
return jsonify({'status': 'success', 'data': config})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/config/raw/main', methods=['POST'])
|
@api_v3.route('/config/raw/main', methods=['POST'])
|
||||||
def save_raw_main_config():
|
def save_raw_main_config():
|
||||||
@@ -1393,7 +1390,6 @@ def save_raw_main_config():
|
|||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.CONFIG_SAVE_FAILED,
|
ErrorCode.CONFIG_SAVE_FAILED,
|
||||||
error_message,
|
error_message,
|
||||||
details=describe_exception(e),
|
|
||||||
|
|
||||||
context={'config_path': e.config_path} if hasattr(e, 'config_path') and e.config_path else None,
|
context={'config_path': e.config_path} if hasattr(e, 'config_path') and e.config_path else None,
|
||||||
status_code=500
|
status_code=500
|
||||||
@@ -1403,7 +1399,6 @@ def save_raw_main_config():
|
|||||||
return error_response(
|
return error_response(
|
||||||
ErrorCode.UNKNOWN_ERROR,
|
ErrorCode.UNKNOWN_ERROR,
|
||||||
error_message,
|
error_message,
|
||||||
details=describe_exception(e),
|
|
||||||
|
|
||||||
status_code=500
|
status_code=500
|
||||||
)
|
)
|
||||||
@@ -1443,8 +1438,7 @@ def save_raw_secrets_config():
|
|||||||
else:
|
else:
|
||||||
error_message = 'An error occurred; see logs for details'
|
error_message = 'An error occurred; see logs for details'
|
||||||
|
|
||||||
return jsonify({'status': 'error', 'message': error_message,
|
return jsonify({'status': 'error', 'message': error_message}), 500
|
||||||
'details': describe_exception(e)}), 500
|
|
||||||
|
|
||||||
@api_v3.route('/system/status', methods=['GET'])
|
@api_v3.route('/system/status', methods=['GET'])
|
||||||
def get_system_status():
|
def get_system_status():
|
||||||
@@ -1532,7 +1526,7 @@ def get_system_status():
|
|||||||
return jsonify({'status': 'success', 'data': status})
|
return jsonify({'status': 'success', 'data': status})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/health', methods=['GET'])
|
@api_v3.route('/health', methods=['GET'])
|
||||||
def get_health():
|
def get_health():
|
||||||
@@ -1631,11 +1625,9 @@ def get_health():
|
|||||||
|
|
||||||
return jsonify({'status': 'success', 'data': health_status})
|
return jsonify({'status': 'success', 'data': health_status})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details',
|
||||||
'details': describe_exception(e),
|
|
||||||
'data': {'status': 'unhealthy'}
|
'data': {'status': 'unhealthy'}
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@@ -2405,7 +2397,7 @@ def get_display_current():
|
|||||||
return jsonify({'status': 'success', 'data': display_data})
|
return jsonify({'status': 'success', 'data': display_data})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/display/on-demand/status', methods=['GET'])
|
@api_v3.route('/display/on-demand/status', methods=['GET'])
|
||||||
def get_on_demand_status():
|
def get_on_demand_status():
|
||||||
@@ -2429,7 +2421,7 @@ def get_on_demand_status():
|
|||||||
})
|
})
|
||||||
except Exception as exc:
|
except Exception as exc:
|
||||||
logger.error('Error in get_on_demand_status', exc_info=True)
|
logger.error('Error in get_on_demand_status', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(exc)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/display/on-demand/start', methods=['POST'])
|
@api_v3.route('/display/on-demand/start', methods=['POST'])
|
||||||
def start_on_demand_display():
|
def start_on_demand_display():
|
||||||
@@ -2532,7 +2524,7 @@ def start_on_demand_display():
|
|||||||
return jsonify({'status': 'success', 'data': response_data})
|
return jsonify({'status': 'success', 'data': response_data})
|
||||||
except Exception as exc:
|
except Exception as exc:
|
||||||
logger.error('Error in start_on_demand_display', exc_info=True)
|
logger.error('Error in start_on_demand_display', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(exc)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/display/on-demand/stop', methods=['POST'])
|
@api_v3.route('/display/on-demand/stop', methods=['POST'])
|
||||||
def stop_on_demand_display():
|
def stop_on_demand_display():
|
||||||
@@ -2568,7 +2560,7 @@ def stop_on_demand_display():
|
|||||||
})
|
})
|
||||||
except Exception as exc:
|
except Exception as exc:
|
||||||
logger.error('Error in stop_on_demand_display', exc_info=True)
|
logger.error('Error in stop_on_demand_display', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(exc)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/installed', methods=['GET'])
|
@api_v3.route('/plugins/installed', methods=['GET'])
|
||||||
def get_installed_plugins():
|
def get_installed_plugins():
|
||||||
@@ -2716,7 +2708,7 @@ def get_installed_plugins():
|
|||||||
return jsonify({'status': 'success', 'data': {'plugins': plugins}})
|
return jsonify({'status': 'success', 'data': {'plugins': plugins}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_installed_plugins', exc_info=True)
|
logger.error('Error in get_installed_plugins', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
def _installed_plugin_ids():
|
def _installed_plugin_ids():
|
||||||
"""Best-effort list of installed plugin IDs for the web process.
|
"""Best-effort list of installed plugin IDs for the web process.
|
||||||
@@ -2782,7 +2774,7 @@ def get_plugin_health():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_plugin_health', exc_info=True)
|
logger.error('Error in get_plugin_health', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/health/<plugin_id>', methods=['GET'])
|
@api_v3.route('/plugins/health/<plugin_id>', methods=['GET'])
|
||||||
def get_plugin_health_single(plugin_id):
|
def get_plugin_health_single(plugin_id):
|
||||||
@@ -2807,7 +2799,7 @@ def get_plugin_health_single(plugin_id):
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_plugin_health_single', exc_info=True)
|
logger.error('Error in get_plugin_health_single', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/health/<plugin_id>/reset', methods=['POST'])
|
@api_v3.route('/plugins/health/<plugin_id>/reset', methods=['POST'])
|
||||||
def reset_plugin_health(plugin_id):
|
def reset_plugin_health(plugin_id):
|
||||||
@@ -2832,7 +2824,7 @@ def reset_plugin_health(plugin_id):
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in reset_plugin_health', exc_info=True)
|
logger.error('Error in reset_plugin_health', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/metrics', methods=['GET'])
|
@api_v3.route('/plugins/metrics', methods=['GET'])
|
||||||
def get_plugin_metrics():
|
def get_plugin_metrics():
|
||||||
@@ -2872,7 +2864,7 @@ def get_plugin_metrics():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_plugin_metrics', exc_info=True)
|
logger.error('Error in get_plugin_metrics', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/metrics/<plugin_id>', methods=['GET'])
|
@api_v3.route('/plugins/metrics/<plugin_id>', methods=['GET'])
|
||||||
def get_plugin_metrics_single(plugin_id):
|
def get_plugin_metrics_single(plugin_id):
|
||||||
@@ -2897,7 +2889,7 @@ def get_plugin_metrics_single(plugin_id):
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_plugin_metrics_single', exc_info=True)
|
logger.error('Error in get_plugin_metrics_single', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/metrics/<plugin_id>/reset', methods=['POST'])
|
@api_v3.route('/plugins/metrics/<plugin_id>/reset', methods=['POST'])
|
||||||
def reset_plugin_metrics(plugin_id):
|
def reset_plugin_metrics(plugin_id):
|
||||||
@@ -2922,7 +2914,7 @@ def reset_plugin_metrics(plugin_id):
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in reset_plugin_metrics', exc_info=True)
|
logger.error('Error in reset_plugin_metrics', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/limits/<plugin_id>', methods=['GET', 'POST'])
|
@api_v3.route('/plugins/limits/<plugin_id>', methods=['GET', 'POST'])
|
||||||
def manage_plugin_limits(plugin_id):
|
def manage_plugin_limits(plugin_id):
|
||||||
@@ -2977,7 +2969,7 @@ def manage_plugin_limits(plugin_id):
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in manage_plugin_limits', exc_info=True)
|
logger.error('Error in manage_plugin_limits', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/toggle', methods=['POST'])
|
@api_v3.route('/plugins/toggle', methods=['POST'])
|
||||||
def toggle_plugin():
|
def toggle_plugin():
|
||||||
@@ -3986,7 +3978,7 @@ def install_plugin():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in install_plugin', exc_info=True)
|
logger.error('Error in install_plugin', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/install-from-url', methods=['POST'])
|
@api_v3.route('/plugins/install-from-url', methods=['POST'])
|
||||||
def install_plugin_from_url():
|
def install_plugin_from_url():
|
||||||
@@ -4041,7 +4033,7 @@ def install_plugin_from_url():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in install_plugin_from_url', exc_info=True)
|
logger.error('Error in install_plugin_from_url', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/registry-from-url', methods=['POST'])
|
@api_v3.route('/plugins/registry-from-url', methods=['POST'])
|
||||||
def get_registry_from_url():
|
def get_registry_from_url():
|
||||||
@@ -4073,7 +4065,7 @@ def get_registry_from_url():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_registry_from_url', exc_info=True)
|
logger.error('Error in get_registry_from_url', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/saved-repositories', methods=['GET'])
|
@api_v3.route('/plugins/saved-repositories', methods=['GET'])
|
||||||
def get_saved_repositories():
|
def get_saved_repositories():
|
||||||
@@ -4086,7 +4078,7 @@ def get_saved_repositories():
|
|||||||
return jsonify({'status': 'success', 'data': {'repositories': repositories}})
|
return jsonify({'status': 'success', 'data': {'repositories': repositories}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_saved_repositories', exc_info=True)
|
logger.error('Error in get_saved_repositories', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/saved-repositories', methods=['POST'])
|
@api_v3.route('/plugins/saved-repositories', methods=['POST'])
|
||||||
def add_saved_repository():
|
def add_saved_repository():
|
||||||
@@ -4117,7 +4109,7 @@ def add_saved_repository():
|
|||||||
}), 400
|
}), 400
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in add_saved_repository', exc_info=True)
|
logger.error('Error in add_saved_repository', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/saved-repositories', methods=['DELETE'])
|
@api_v3.route('/plugins/saved-repositories', methods=['DELETE'])
|
||||||
def remove_saved_repository():
|
def remove_saved_repository():
|
||||||
@@ -4147,7 +4139,7 @@ def remove_saved_repository():
|
|||||||
}), 404
|
}), 404
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in remove_saved_repository', exc_info=True)
|
logger.error('Error in remove_saved_repository', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/store/list', methods=['GET'])
|
@api_v3.route('/plugins/store/list', methods=['GET'])
|
||||||
def list_plugin_store():
|
def list_plugin_store():
|
||||||
@@ -4200,7 +4192,7 @@ def list_plugin_store():
|
|||||||
return jsonify({'status': 'success', 'data': {'plugins': formatted_plugins}})
|
return jsonify({'status': 'success', 'data': {'plugins': formatted_plugins}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in list_plugin_store', exc_info=True)
|
logger.error('Error in list_plugin_store', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/store/github-status', methods=['GET'])
|
@api_v3.route('/plugins/store/github-status', methods=['GET'])
|
||||||
def get_github_auth_status():
|
def get_github_auth_status():
|
||||||
@@ -4251,7 +4243,7 @@ def get_github_auth_status():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_github_auth_status', exc_info=True)
|
logger.error('Error in get_github_auth_status', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/store/refresh', methods=['POST'])
|
@api_v3.route('/plugins/store/refresh', methods=['POST'])
|
||||||
def refresh_plugin_store():
|
def refresh_plugin_store():
|
||||||
@@ -4278,7 +4270,7 @@ def refresh_plugin_store():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in refresh_plugin_store', exc_info=True)
|
logger.error('Error in refresh_plugin_store', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
def deep_merge(base_dict, update_dict):
|
def deep_merge(base_dict, update_dict):
|
||||||
"""
|
"""
|
||||||
@@ -5830,7 +5822,7 @@ def get_plugin_schema():
|
|||||||
return jsonify({'status': 'success', 'data': {'schema': default_schema}})
|
return jsonify({'status': 'success', 'data': {'schema': default_schema}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in get_plugin_schema', exc_info=True)
|
logger.error('Error in get_plugin_schema', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/skins', methods=['GET'])
|
@api_v3.route('/skins', methods=['GET'])
|
||||||
def list_skins():
|
def list_skins():
|
||||||
@@ -5865,9 +5857,9 @@ def list_skins():
|
|||||||
'has_preview': bool(preview and (skin_dir / preview).is_file()),
|
'has_preview': bool(preview and (skin_dir / preview).is_file()),
|
||||||
})
|
})
|
||||||
return jsonify({'status': 'success', 'data': {'skins': payload}})
|
return jsonify({'status': 'success', 'data': {'skins': payload}})
|
||||||
except Exception as e:
|
except Exception:
|
||||||
logger.error('Error in list_skins', exc_info=True)
|
logger.error('Error in list_skins', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/config/reset', methods=['POST'])
|
@api_v3.route('/plugins/config/reset', methods=['POST'])
|
||||||
def reset_plugin_config():
|
def reset_plugin_config():
|
||||||
@@ -5978,7 +5970,7 @@ def reset_plugin_config():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in reset_plugin_config', exc_info=True)
|
logger.error('Error in reset_plugin_config', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/action', methods=['POST'])
|
@api_v3.route('/plugins/action', methods=['POST'])
|
||||||
def execute_plugin_action():
|
def execute_plugin_action():
|
||||||
@@ -6238,7 +6230,7 @@ sys.exit(proc.returncode)
|
|||||||
logger.error("Error executing action step 1", exc_info=True)
|
logger.error("Error executing action step 1", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
else:
|
else:
|
||||||
# Simple script execution
|
# Simple script execution
|
||||||
@@ -6288,7 +6280,7 @@ sys.exit(proc.returncode)
|
|||||||
return jsonify({'status': 'error', 'message': 'Action timed out'}), 408
|
return jsonify({'status': 'error', 'message': 'Action timed out'}), 408
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in execute_plugin_action', exc_info=True)
|
logger.error('Error in execute_plugin_action', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/authenticate/spotify', methods=['POST'])
|
@api_v3.route('/plugins/authenticate/spotify', methods=['POST'])
|
||||||
def authenticate_spotify():
|
def authenticate_spotify():
|
||||||
@@ -6421,12 +6413,12 @@ sys.exit(proc.returncode)
|
|||||||
logger.error("Error getting Spotify auth URL", exc_info=True)
|
logger.error("Error getting Spotify auth URL", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in authenticate_spotify', exc_info=True)
|
logger.error('Error in authenticate_spotify', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/authenticate/ytm', methods=['POST'])
|
@api_v3.route('/plugins/authenticate/ytm', methods=['POST'])
|
||||||
def authenticate_ytm():
|
def authenticate_ytm():
|
||||||
@@ -6476,7 +6468,7 @@ def authenticate_ytm():
|
|||||||
return jsonify({'status': 'error', 'message': 'Authentication timed out'}), 408
|
return jsonify({'status': 'error', 'message': 'Authentication timed out'}), 408
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in authenticate_ytm', exc_info=True)
|
logger.error('Error in authenticate_ytm', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/fonts/catalog', methods=['GET'])
|
@api_v3.route('/fonts/catalog', methods=['GET'])
|
||||||
def get_fonts_catalog():
|
def get_fonts_catalog():
|
||||||
@@ -6571,10 +6563,7 @@ def get_fonts_catalog():
|
|||||||
|
|
||||||
return jsonify({'status': 'success', 'data': {'catalog': catalog}})
|
return jsonify({'status': 'success', 'data': {'catalog': catalog}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
return jsonify({'status': 'error',
|
|
||||||
'message': 'An error occurred; see logs for details',
|
|
||||||
'details': describe_exception(e)}), 500
|
|
||||||
|
|
||||||
@api_v3.route('/fonts/tokens', methods=['GET'])
|
@api_v3.route('/fonts/tokens', methods=['GET'])
|
||||||
def get_font_tokens():
|
def get_font_tokens():
|
||||||
@@ -6593,7 +6582,7 @@ def get_font_tokens():
|
|||||||
return jsonify({'status': 'success', 'data': {'tokens': tokens}})
|
return jsonify({'status': 'success', 'data': {'tokens': tokens}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/fonts/overrides', methods=['GET'])
|
@api_v3.route('/fonts/overrides', methods=['GET'])
|
||||||
def get_fonts_overrides():
|
def get_fonts_overrides():
|
||||||
@@ -6605,7 +6594,7 @@ def get_fonts_overrides():
|
|||||||
return jsonify({'status': 'success', 'data': {'overrides': overrides}})
|
return jsonify({'status': 'success', 'data': {'overrides': overrides}})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/fonts/overrides', methods=['POST'])
|
@api_v3.route('/fonts/overrides', methods=['POST'])
|
||||||
def save_fonts_overrides():
|
def save_fonts_overrides():
|
||||||
@@ -6619,7 +6608,7 @@ def save_fonts_overrides():
|
|||||||
return jsonify({'status': 'success', 'message': 'Font overrides saved'})
|
return jsonify({'status': 'success', 'message': 'Font overrides saved'})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/fonts/overrides/<element_key>', methods=['DELETE'])
|
@api_v3.route('/fonts/overrides/<element_key>', methods=['DELETE'])
|
||||||
def delete_font_override(element_key):
|
def delete_font_override(element_key):
|
||||||
@@ -6629,7 +6618,7 @@ def delete_font_override(element_key):
|
|||||||
return jsonify({'status': 'success', 'message': f'Font override for {element_key} deleted'})
|
return jsonify({'status': 'success', 'message': f'Font override for {element_key} deleted'})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/fonts/upload', methods=['POST'])
|
@api_v3.route('/fonts/upload', methods=['POST'])
|
||||||
def upload_font():
|
def upload_font():
|
||||||
@@ -6694,7 +6683,7 @@ def upload_font():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
|
|
||||||
@api_v3.route('/fonts/preview', methods=['GET'])
|
@api_v3.route('/fonts/preview', methods=['GET'])
|
||||||
@@ -6839,7 +6828,7 @@ def get_font_preview() -> tuple[Response, int] | Response:
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
|
|
||||||
@api_v3.route('/fonts/<font_family>', methods=['DELETE'])
|
@api_v3.route('/fonts/<font_family>', methods=['DELETE'])
|
||||||
@@ -6927,7 +6916,7 @@ def delete_font(font_family: str) -> tuple[Response, int] | Response:
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
|
|
||||||
@api_v3.route('/plugins/assets/upload', methods=['POST'])
|
@api_v3.route('/plugins/assets/upload', methods=['POST'])
|
||||||
@@ -7075,7 +7064,7 @@ def upload_plugin_asset():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/of-the-day/json/upload', methods=['POST'])
|
@api_v3.route('/plugins/of-the-day/json/upload', methods=['POST'])
|
||||||
def upload_of_the_day_json():
|
def upload_of_the_day_json():
|
||||||
@@ -7225,7 +7214,7 @@ def upload_of_the_day_json():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/of-the-day/json/delete', methods=['POST'])
|
@api_v3.route('/plugins/of-the-day/json/delete', methods=['POST'])
|
||||||
def delete_of_the_day_json():
|
def delete_of_the_day_json():
|
||||||
@@ -7272,7 +7261,7 @@ def delete_of_the_day_json():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/<plugin_id>/static/<path:file_path>', methods=['GET'])
|
@api_v3.route('/plugins/<plugin_id>/static/<path:file_path>', methods=['GET'])
|
||||||
def serve_plugin_static(plugin_id, file_path):
|
def serve_plugin_static(plugin_id, file_path):
|
||||||
@@ -7318,7 +7307,7 @@ def serve_plugin_static(plugin_id, file_path):
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
|
|
||||||
@api_v3.route('/plugins/calendar/upload-credentials', methods=['POST'])
|
@api_v3.route('/plugins/calendar/upload-credentials', methods=['POST'])
|
||||||
@@ -7400,7 +7389,7 @@ def upload_calendar_credentials():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in upload_calendar_credentials', exc_info=True)
|
logger.error('Error in upload_calendar_credentials', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/assets/delete', methods=['POST'])
|
@api_v3.route('/plugins/assets/delete', methods=['POST'])
|
||||||
def delete_plugin_asset():
|
def delete_plugin_asset():
|
||||||
@@ -7443,7 +7432,7 @@ def delete_plugin_asset():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/plugins/assets/list', methods=['GET'])
|
@api_v3.route('/plugins/assets/list', methods=['GET'])
|
||||||
def list_plugin_assets():
|
def list_plugin_assets():
|
||||||
@@ -7471,7 +7460,7 @@ def list_plugin_assets():
|
|||||||
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Unhandled exception', exc_info=True)
|
logger.error('Unhandled exception', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/display/current-status', methods=['GET'])
|
@api_v3.route('/display/current-status', methods=['GET'])
|
||||||
def get_current_display_status():
|
def get_current_display_status():
|
||||||
@@ -7492,9 +7481,9 @@ def get_current_display_status():
|
|||||||
'last_updated': None,
|
'last_updated': None,
|
||||||
}
|
}
|
||||||
return jsonify({'status': 'success', 'data': state})
|
return jsonify({'status': 'success', 'data': state})
|
||||||
except Exception as e:
|
except Exception:
|
||||||
logger.error('Error in get_current_display_status', exc_info=True)
|
logger.error('Error in get_current_display_status', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/logs', methods=['GET'])
|
@api_v3.route('/logs', methods=['GET'])
|
||||||
def get_logs():
|
def get_logs():
|
||||||
@@ -7533,11 +7522,9 @@ def get_logs():
|
|||||||
'message': 'Timeout while fetching logs'
|
'message': 'Timeout while fetching logs'
|
||||||
}), 500
|
}), 500
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
# Multi-Display Sync Endpoints
|
# Multi-Display Sync Endpoints
|
||||||
@@ -7602,11 +7589,9 @@ def get_wifi_status():
|
|||||||
}
|
}
|
||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/scan', methods=['GET'])
|
@api_v3.route('/wifi/scan', methods=['GET'])
|
||||||
@@ -7728,7 +7713,7 @@ def connect_wifi():
|
|||||||
logger.error("Error connecting to WiFi", exc_info=True)
|
logger.error("Error connecting to WiFi", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/disconnect', methods=['POST'])
|
@api_v3.route('/wifi/disconnect', methods=['POST'])
|
||||||
@@ -7754,7 +7739,7 @@ def disconnect_wifi():
|
|||||||
logger.error("Error disconnecting from WiFi", exc_info=True)
|
logger.error("Error disconnecting from WiFi", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/ap/enable', methods=['POST'])
|
@api_v3.route('/wifi/ap/enable', methods=['POST'])
|
||||||
@@ -7779,11 +7764,9 @@ def enable_ap_mode():
|
|||||||
'message': message
|
'message': message
|
||||||
}), 400
|
}), 400
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/ap/disable', methods=['POST'])
|
@api_v3.route('/wifi/ap/disable', methods=['POST'])
|
||||||
@@ -7806,11 +7789,9 @@ def disable_ap_mode():
|
|||||||
'message': message
|
'message': message
|
||||||
}), 400
|
}), 400
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/ap/auto-enable', methods=['GET'])
|
@api_v3.route('/wifi/ap/auto-enable', methods=['GET'])
|
||||||
@@ -7829,11 +7810,9 @@ def get_auto_enable_ap_mode():
|
|||||||
}
|
}
|
||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/ap/auto-enable', methods=['POST'])
|
@api_v3.route('/wifi/ap/auto-enable', methods=['POST'])
|
||||||
@@ -7863,11 +7842,9 @@ def set_auto_enable_ap_mode():
|
|||||||
}
|
}
|
||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error("%s failed", request.path, exc_info=True)
|
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details',
|
'message': 'An error occurred; see logs for details'
|
||||||
'details': describe_exception(e)
|
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/radio', methods=['GET'])
|
@api_v3.route('/wifi/radio', methods=['GET'])
|
||||||
@@ -7887,7 +7864,7 @@ def get_wifi_radio():
|
|||||||
logger.error("Error getting WiFi radio state", exc_info=True)
|
logger.error("Error getting WiFi radio state", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/wifi/radio', methods=['POST'])
|
@api_v3.route('/wifi/radio', methods=['POST'])
|
||||||
@@ -7935,7 +7912,7 @@ def set_wifi_radio():
|
|||||||
logger.error("Error setting WiFi radio state", exc_info=True)
|
logger.error("Error setting WiFi radio state", exc_info=True)
|
||||||
return jsonify({
|
return jsonify({
|
||||||
'status': 'error',
|
'status': 'error',
|
||||||
'message': 'An error occurred; see logs for details', 'details': describe_exception(e)
|
'message': 'An error occurred; see logs for details'
|
||||||
}), 500
|
}), 500
|
||||||
|
|
||||||
@api_v3.route('/cache/list', methods=['GET'])
|
@api_v3.route('/cache/list', methods=['GET'])
|
||||||
@@ -7960,7 +7937,7 @@ def list_cache_files():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in list_cache_files', exc_info=True)
|
logger.error('Error in list_cache_files', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
@api_v3.route('/cache/delete', methods=['POST'])
|
@api_v3.route('/cache/delete', methods=['POST'])
|
||||||
def delete_cache_file():
|
def delete_cache_file():
|
||||||
@@ -7986,7 +7963,7 @@ def delete_cache_file():
|
|||||||
})
|
})
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
logger.error('Error in delete_cache_file', exc_info=True)
|
logger.error('Error in delete_cache_file', exc_info=True)
|
||||||
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details', 'details': describe_exception(e)}), 500
|
return jsonify({'status': 'error', 'message': 'An error occurred; see logs for details'}), 500
|
||||||
|
|
||||||
|
|
||||||
# =============================================================================
|
# =============================================================================
|
||||||
|
|||||||
@@ -556,11 +556,11 @@
|
|||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="form-group" id="setting-display-vegas_max_plugin_width_ratio" data-setting-key="display.vegas_scroll.max_plugin_width_ratio">
|
<div class="form-group" id="setting-display-vegas_max_plugin_width_ratio" data-setting-key="display.vegas_scroll.max_plugin_width_ratio">
|
||||||
<label for="vegas_max_plugin_width_ratio" class="block text-sm font-medium text-gray-700">Max Plugin Width (screens){{ ui.help_tip('Caps how much of one cycle a single plugin may occupy, measured in screen widths (0–20).\nDefault: 3. A long ticker such as a news feed or leaderboard is trimmed to this and the remainder shown on later cycles, so one plugin cannot hold the display for minutes. Set 0 for no limit.', 'Max Plugin Width') }}</label>
|
<label for="vegas_max_plugin_width_ratio" class="block text-sm font-medium text-gray-700">Max Plugin Width (screens){{ ui.help_tip('Caps how much of one cycle a single plugin may occupy, measured in screen widths (0–20).\nDefault: 0 (no limit) — every plugin shows all of its content and always starts at the beginning.\nSet a limit to stop one long ticker holding the display for minutes: it is cut to this width and the remainder shown on later cycles. The trade-off is that such a plugin then resumes mid-content on each appearance instead of starting fresh.', 'Max Plugin Width') }}</label>
|
||||||
<input type="number"
|
<input type="number"
|
||||||
id="vegas_max_plugin_width_ratio"
|
id="vegas_max_plugin_width_ratio"
|
||||||
name="vegas_max_plugin_width_ratio"
|
name="vegas_max_plugin_width_ratio"
|
||||||
value="{{ main_config.display.get('vegas_scroll', {}).get('max_plugin_width_ratio', 3.0) }}"
|
value="{{ main_config.display.get('vegas_scroll', {}).get('max_plugin_width_ratio', 0.0) }}"
|
||||||
min="0"
|
min="0"
|
||||||
max="20"
|
max="20"
|
||||||
step="0.5"
|
step="0.5"
|
||||||
|
|||||||
Reference in New Issue
Block a user