Security context

What an agent needs to avoid regressing past fixes and find the next vuln in this repo.

dappnode/DNP_DAPPMANAGER
develop @ 16b3722
15
Fixes
0
CVEs
CRITICAL
Peak severity
40.0%
Coverage
Highlights
Auth Bypass: 4 prior fixes. Scrutinize any change in this area.
packages/dappmanager/src/modules/compose/unsafeCompose.ts: most-fixed (2 issues). Treat as high-risk during review.
13 high-severity fixes in this history; regressions here are high-impact.
Recurring patterns

The bug types that recur here, drawn from past fixes, not open vulnerabilities.

Command Injection: Shell command built from user-controlled containerName, filename, and toPath without strict allowlisting; fixed with regex validation but sibling paths may still interpolate unsanitized values.
Command Injection: Docker-compose commands interpret ${} substitutions; recursive rejection added, but any compose path that skips this verification (e.g., unsafeCompose or direct dockerComposeUp) remains a risk.
Auth Bypass: Session validation logic failed to reject non-admin sessions; fixed by explicit else branch. Similar logic in other auth modules may have the same flaw.