Highlights
Command Injection: 3 prior fixes. Scrutinize any change in this area.
python/opendataloader-pdf/src/opendataloader_pdf/hybrid_server.py: most-fixed (3 issues). Treat as high-risk during review.
3 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: Untrusted workflow triggers (such as issue titles, bodies, and comment contents) were directly interpolated inside run blocks, allowing collaborators or external actors to execute arbitrary code on the runner. Future workflow designs must strictly pass dynamic variables via environment variables and enforce strict authorization checks.
Denial of Service: The hybrid server was vulnerable to memory and resource exhaustion from large, unrestricted file streams. Implementing incremental stream size validations and strict maximum file size limits (e.g., 100MB) is vital to keep the server resilient against denial-of-service attempts.
Denial of Service: Parsing deeply nested or recursive PDF table structures could exhaust the stack space and trigger application crashes. Enforcing strict structural limit checks, such as maximum nesting depths, prevents stack overflow events during complex document ingestion.