Skip to content

fix(security): complete allowlist input validation audit - #233

Open
TFT444 wants to merge 2 commits into
devfrom
fix/201-input-validation-audit
Open

fix(security): complete allowlist input validation audit#233
TFT444 wants to merge 2 commits into
devfrom
fix/201-input-validation-audit

Conversation

@TFT444

@TFT444 TFT444 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add centralized allowlist, type, format, and size validation for API inputs
  • validate scan/findings paths, query parameters, AI payloads, request IDs, auth headers, and Sentinel ingestion data
  • document every reviewed boundary and update the OpenSSF Silver evidence register
  • add security regression coverage for malformed, oversized, duplicated, and injection-style inputs

Validation

  • ruff check .
  • ruff format --check .
  • 511 passed, 3 skipped with 84.42% API/scanner coverage
  • frontend lint, i18n, accessibility, and production build
  • website URL security tests
  • Bandit: no medium/high findings
  • pip-audit and both npm audits: 0 vulnerabilities

Closes #201

Signed-off-by: Tanvir Farhad <tamimtarafder12@gmail.com>
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Comment thread api/routes/ai.py Fixed
Comment thread api/routes/ai.py Fixed
Comment thread api/routes/ai.py Fixed
Comment thread api/routes/ai.py Fixed
Comment thread api/routes/ai.py Fixed
Comment thread api/routes/findings.py Fixed
Comment thread api/routes/findings.py Fixed
Comment thread api/routes/scans.py Fixed
Comment thread api/routes/scans.py Fixed
Comment thread api/routes/scans.py Fixed
@TFT444
TFT444 requested review from m-khan-97 and removed request for parthrohit22 August 6, 2026 11:35
Signed-off-by: Tanvir Farhad <tamimtarafder12@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

security: complete allowlist-based input validation audit

2 participants