Skip to content

Refactor entropy scanning to use os.walk#298

Open
Sejusahu011 wants to merge 1 commit into
ionfwsrijan:mainfrom
Sejusahu011:patch-1
Open

Refactor entropy scanning to use os.walk#298
Sejusahu011 wants to merge 1 commit into
ionfwsrijan:mainfrom
Sejusahu011:patch-1

Conversation

@Sejusahu011

Copy link
Copy Markdown

Before opening: make sure there is an issue tracking this work, and link it below. PRs without a linked issue may be closed without review.

Linked issue

Closes #38

What this PR does

Adds a SQLite persistence layer for scan findings. Previously, findings were stored only in memory and were lost after server restarts. This PR introduces SQLite integration, a findings schema, CRUD helper functions, and persistent storage keyed by job_id.

Type of change

  • Bug fix
  • New feature
  • ML model / training pipeline
  • Refactor (no behaviour change)
  • Documentation
  • Tests only

ML tier (if applicable)

  • Tier 1 — Triage
  • Tier 2 — Predictive
  • Tier 3 — Autonomous
  • Not ML-related

Stack affected

  • Backend
  • Frontend
  • Both

Changes

Backend

  • Integrated SQLite database.
  • Added schema for storing scan findings.
  • Implemented CRUD helper functions.
  • Persisted findings using job_id.
  • Stored scanner type, severity, title and timestamps.

Frontend

  • No frontend changes.

New dependencies

  • SQLite driver.

Database / schema changes

  • Added SQLite schema for persistent scan findings storage.

Testing

How did you test this?

  • Ran the application locally.
  • Executed a scan.
  • Verified findings were written to SQLite.
  • Retrieved findings using job_id.
  • Restarted the server and confirmed findings persisted.

Checklist

  • Tested locally end-to-end (upload ZIP or GitHub URL → scan → findings returned correctly)
  • New ML model falls back gracefully when model file is absent
  • No new console.error or unhandled Python exceptions introduced
  • Added or updated tests where applicable
  • requirements.txt / package.json updated if new dependencies added
  • New model files (.pkl, .pt, etc.) are gitignored, not committed

Anything reviewers should focus on

Please review the SQLite schema, CRUD implementation, and integration with the scan workflow to ensure findings are persisted and retrieved correctly.

Screenshots (if UI changed)

N/A (Backend-only changes)

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🎉 Thank you @Sejusahu011 for submitting a Pull Request!

We're excited to review your contribution.

Before Review

✅ Ensure all CI checks pass
✅ Complete the PR template
✅ Link the related issue

Want faster reviews and contributor support?

Join our Discord community:

🔗 https://discord.gg/FcXuyw2Rs

Maintainers and mentors are active there and can help resolve blockers quickly.

Happy Contributing! 🚀

@github-actions github-actions Bot added backend Backend issues feature New feature frontend Frontend issues SSoC26 labels Jul 8, 2026
@github-actions github-actions Bot requested a review from arpit2006 July 8, 2026 18:06
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

PR template check passed!

@arpit2006 this PR is ready for your review. 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend issues feature New feature frontend Frontend issues SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add SQLite persistence layer for scan findings

1 participant