Skip to content

Conversation

@sgiehl
Copy link
Member

@sgiehl sgiehl commented Dec 11, 2025

Description

  • Map client-side faults to consistent 4xx responses: missing/invalid API params return 400, invalid actions return
    404, missing chunks return 404, missing plugins return 404, deactivated plugins return 403.
  • Silence noisy error logging for 4xx-only failures by tagging these paths as HttpCodeException and downgrading log
  • System/integration tests added for HTTP status expectations (400 missing param, 404 missing plugin, 404 invalid
    action, 404 missing asset chunk) and for plugin activation checks (404 missing plugin, 403 deactivated) plus missing
    chunk error in the asset fetcher.

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

Review

@sgiehl sgiehl added this to the 5.7.0 milestone Dec 11, 2025
@sgiehl sgiehl requested a review from a team December 11, 2025 11:17
@sgiehl sgiehl force-pushed the dev-19666 branch 3 times, most recently from e9a83d0 to 85a28c1 Compare December 11, 2025 16:54
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2026

If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'.

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Jan 3, 2026
@github-actions github-actions bot removed the Stale The label used by the Close Stale Issues action label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants