mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-08-04 22:32:12 +03:00
2175f0c041
Replaces \`return NextResponse.json({ error: error.message }, ...)\` and the
ad-hoc \`error instanceof Error ? error.message : String(error)\` helpers with
\`sanitizeErrorMessage()\` from \`@omniroute/open-sse/utils/error\` across the
remaining management/api routes flagged by semgrep:
analytics/diversity, cache, cache/reasoning, db-backups (root, export,
import), evals (root + suiteId), mcp (audit, audit/stats, sse, status,
stream, tools), memory/health, middleware/hooks (root + name), models/test,
providers/[id]/models, providers/[id]/sync-models, resilience (root +
model-cooldowns), sessions, settings/proxy/test, storage/health,
sync/cloud, telemetry/summary, translator/history.
\`sanitizeErrorMessage\` strips stack traces, absolute paths, and the
common Error.toString prefix before serializing — Hard Rule #12 / see
docs/security/ERROR_SANITIZATION.md. Behaviour for legitimate clients is
unchanged; only the leak surface contracts.
Also adds tests/unit/management-auth-hardening.test.ts to lock down the
new contract end-to-end so any future regression to raw \`err.message\`
in these routes fails CI.