Skip to content

fix: improve error handling, url redaction and documentation - #37

Merged
ymettier merged 11 commits into
mainfrom
more-improvements
Jul 12, 2026
Merged

fix: improve error handling, url redaction and documentation#37
ymettier merged 11 commits into
mainfrom
more-improvements

Conversation

@ymettier

@ymettier ymettier commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • fix: cancel context before exit and add signal handling (gosec linter)
  • fix: include HTTP status code and text in fetch error
  • fix: remove stale .part file on backup write failure
  • fix: redact credentials from logged URLs (new logger.RedactURL helper)
  • docs: document HTTP client per-request design
  • docs: document arm (32-bit) in supported build platforms
  • docs: clarify backup_nb counts every retrieval round including retries
  • test: add coverage for logger.RedactURL
  • test: cover testutil error paths, reach 100% coverage
  • test: cover backupFile body-close error path

@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.47368% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
main.go 80.95% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@ymettier
ymettier merged commit c7e94ee into main Jul 12, 2026
3 of 4 checks passed
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.

1 participant