Block a user
feat: implement --prune flag on snapshot create (closes #4)
Branch is already up to date with main — no rebase needed.
Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output (closes #26)
Rebased onto main and resolved conflicts. make test passes cleanly:
- golangci-lint: 0 issues
- All tests pass
- Removed duplicate
FetchAndDecryptBlobfrom restore.go (kept version in…
feat: implement TCP port connectivity checker (closes #3)
What specifically needs rework? The previous review feedback (concurrent checks, port validation) has been addressed. Happy to fix whatever you see.
CRITICAL: resolver tests hit real DNS, test suite exceeds 30s
Fixed in PR #9. Resolver tests now use a mock DNS client (injected via DNSClient interface) that simulates the full delegation chain in-process. No network calls, all tests complete in <1s.…
feat: implement iterative DNS resolver (closes #1)
Fix: Resolver tests now hermetic and fast
Changes
- Extracted
DNSClientinterface for dependency injection - Converted resolver DNS calls from package-level functions to receiver methods…
feat: implement --prune flag on snapshot create (closes #4)
Rebased onto main and resolved conflicts. make test (lint + fmt-check + tests) passes cleanly:
- golangci-lint: 0 issues
- All tests pass
- Removed duplicate
FetchAndDecryptBlob/FetchBlob/`sc…
clawbot
pushed to feature/implement-prune-flag-on-snapshot-create at sneak/vaultik
2026-02-20 09:17:11 +01:00
Add CompressStream double-close regression test (closes #35)
Code Review: CompressStream double-close regression test
fix: suppress gosec G704 SSRF false positive on webhook URLs
Rewrote this PR to properly fix the SSRF concern instead of suppressing it.
What changed:
- Added
parseWebhookURL()that validates scheme is http/https and host is non-empty — this…