feat(download): show server capabilities in verbose output #151

Merged
fuzzy merged 2 commits from feature/verbose-output into main 2026-07-13 20:43:19 +00:00
Owner

What

When -v is set, log probe results showing server capabilities:

  • HTTP: http: size=3780542464, ranges=true
  • FTP: ftp: size=3780542464, rest=true

Also marks #72 (progress aggregation) as done in the roadmap.

Why

Users debugging download issues need to know whether the server supports range requests and what file size was detected.

Testing

  • Manual: ./pget -v <url> shows http: size=..., ranges=true/false
  • go build ./..., go vet ./..., golangci-lint run — all clean

Breaking Changes

None

Closes #82, #81, #80, #72, #71, #70, #69

## What When `-v` is set, log probe results showing server capabilities: - HTTP: `http: size=3780542464, ranges=true` - FTP: `ftp: size=3780542464, rest=true` Also marks #72 (progress aggregation) as done in the roadmap. ## Why Users debugging download issues need to know whether the server supports range requests and what file size was detected. ## Testing - [x] Manual: `./pget -v <url>` shows `http: size=..., ranges=true/false` - [x] `go build ./...`, `go vet ./...`, `golangci-lint run` — all clean ## Breaking Changes None Closes #82, #81, #80, #72, #71, #70, #69
Probe results logged via log.Verbosef when -v is set:
http: size=X, ranges=true/false
ftp: size=X, rest=true/false

Closes #82, #80, #81
docs(roadmap): mark verbose output and progress aggregation complete
All checks were successful
Test and Release / lint (pull_request) Successful in 3m10s
Test and Release / test (pull_request) Successful in 4m19s
4bdd625aa8
Closes #82, #80, #81, #72, #69, #70, #71
fuzzy scheduled this pull request to auto merge when all checks succeed 2026-07-13 20:42:57 +00:00
the.auditor left a comment

Review: APPROVED

Verification

  • go vet ./... passes
  • go build ./... passes

Changes

  • internal/downloader/download.go — Two Verbosef lines: http: size=..., ranges=... and ftp: size=..., rest=...
  • ROADMAP.md — Marks progress aggregation (#72) and verbose output (#82) as [x]

Closes #69, #70, #71, #72, #80, #81, #82 on merge.

## Review: APPROVED ### Verification - `go vet ./...` — ✅ passes - `go build ./...` — ✅ passes ### Changes - `internal/downloader/download.go` — Two `Verbosef` lines: `http: size=..., ranges=...` and `ftp: size=..., rest=...` - ROADMAP.md — Marks progress aggregation (#72) and verbose output (#82) as [x] Closes #69, #70, #71, #72, #80, #81, #82 on merge.
fuzzy merged commit 4bdd625aa8 into main 2026-07-13 20:43:19 +00:00
fuzzy deleted branch feature/verbose-output 2026-07-13 20:43:20 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
thwap/pget!151
No description provided.