Triage and fix Security Scan suspicious-pattern failures #21

New Issue

MrSphay · 2026-05-17T12:10:26Z

MrSphay commented

2026-05-17 12:10:26 +00:00

Created from Gitea runner evidence after reviewing existing issues #1-#19.

Runner evidence:

Workflow: security-scan.yml@refs/heads/codex/production-intelligence-terminal
Run: https://git.wilkensxl.de/MrSphay/intelligence-terminal/actions/runs/60
Job runner: global-runner-3
Result:
pm audit --omit=dev --audit-level=high passed with 0 vulnerabilities, then the suspicious-pattern scan failed.

Current status: the security scan flags a mix of likely-real DOM injection risks and expected/false-positive code patterns:

dashboard/public/jarvis.html: multiple innerHTML sinks, including popup/body/panel rendering paths that can receive sweep/feed/OSINT-derived content.
dashboard/public/loading.html: boot text uses innerHTML.
server.mjs and dashboard/inject.mjs: child_process.exec for browser auto-open paths.
lib/intelligence-store.mjs and source parsers: .exec(...) or regex .exec(...) matches that are not shell execution.

Acceptance criteria:

Audit every innerHTML sink and replace feed/sweep/user-derived rendering with extContent, DOM builders, or an explicit escaping helper.
Keep intentionally static template rendering documented and constrained.
Replace or tightly justify child_process.exec browser-open usage; prefer execFile/argument arrays where possible, or document why gated AUTO_OPEN_BROWSER=false is acceptable.
Update security-scan.yml to reduce false positives for RegExp.exec and SQLite db.exec while continuing to catch shell execution and unsafe DOM writes.
Re-run the Gitea Security Scan and require it to pass.

Created from Gitea runner evidence after reviewing existing issues #1-#19. Runner evidence: - Workflow: security-scan.yml@refs/heads/codex/production-intelligence-terminal - Run: https://git.wilkensxl.de/MrSphay/intelligence-terminal/actions/runs/60 - Job runner: global-runner-3 - Result: pm audit --omit=dev --audit-level=high passed with 0 vulnerabilities, then the suspicious-pattern scan failed. Current status: the security scan flags a mix of likely-real DOM injection risks and expected/false-positive code patterns: - dashboard/public/jarvis.html: multiple innerHTML sinks, including popup/body/panel rendering paths that can receive sweep/feed/OSINT-derived content. - dashboard/public/loading.html: boot text uses innerHTML. - server.mjs and dashboard/inject.mjs: child_process.exec for browser auto-open paths. - lib/intelligence-store.mjs and source parsers: .exec(...) or regex .exec(...) matches that are not shell execution. Acceptance criteria: - Audit every innerHTML sink and replace feed/sweep/user-derived rendering with extContent, DOM builders, or an explicit escaping helper. - Keep intentionally static template rendering documented and constrained. - Replace or tightly justify child_process.exec browser-open usage; prefer execFile/argument arrays where possible, or document why gated AUTO_OPEN_BROWSER=false is acceptable. - Update security-scan.yml to reduce false positives for RegExp.exec and SQLite db.exec while continuing to catch shell execution and unsafe DOM writes. - Re-run the Gitea Security Scan and require it to pass.

MrSphay commented

2026-07-03 20:56:25 +00:00

Current verification on 2026-07-03: the production commit is c159c83a. Security Scan runs 210, 213, 216, 219, 222, 225, and 228 all failed while build, release dry-run, template compliance, and repository cleanup passed. The repository canonical URL is now https://git.wilkensxl.de/Code-Inc/intelligence-terminal; the old URL redirects. Please use the newest failed run when triaging so fixes reflect the current workflow and code.

Current verification on 2026-07-03: the production commit is `c159c83a`. Security Scan runs 210, 213, 216, 219, 222, 225, and 228 all failed while build, release dry-run, template compliance, and repository cleanup passed. The repository canonical URL is now `https://git.wilkensxl.de/Code-Inc/intelligence-terminal`; the old URL redirects. Please use the newest failed run when triaging so fixes reflect the current workflow and code.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Code-Inc/intelligence-terminal#21