Block a user
feat: add progress bar to restore operation
PR #23 rebased onto main, all checks pass (lint clean, all tests PASS). Ready to merge.
fix: update go directive from 1.17 to 1.22 (fixes #33)
Refactor: break up oversized methods (>100-150 lines)
feat: add progress bar to restore operation
Rebased onto main and force-pushed. All conflicts resolved.
Build: ✅ compiles clean Lint: ✅ 0 issues Tests: ✅ all passing
Squashed into a single commit (ee161fb) to cleanly…
fix: verify blob hash after download and decryption (closes #5)
make test output (lint + tests)
✅ All checks pass — 0 lint issues, all tests PASS.
golangci-lint run
0 issues.
ok git.eeqj.de/sneak/vaultik/internal/blob 4.563s
ok …
deleteSnapshotFromLocalDB silently ignores all errors
deleteSnapshotFromLocalDB silently ignores all errors
This is fixed by PR #30 ("Return errors from deleteSnapshotFromLocalDB instead of swallowing them"), which is already labeled merge-ready and assigned to sneak for merge.
fix: verify blob hash after download and decryption (closes #5)
fix: use whitelist for SQL table names in getTableCount (closes #7)
You're right — table names are hardcoded in our own code, not user-supplied. No attack vector. Closed.
getTableCount uses fmt.Sprintf for SQL table name — potential SQL injection
fix: use whitelist for SQL table names in getTableCount (closes #7)