ukeeper / ukeeper-readability / 28681140645

78%

master: 78%

Build Type

Pull #85

github

Committed by paskal

Commit Message

Stream image sizing, share client, honor context, cap concurrency

getImageSize buffered every image fully into memory via io.ReadAll, built
a fresh http.Client per image, ignored the request context, and spawned
one unbounded goroutine per <img> on the page.

It now streams to io.Discard behind a 10MB io.LimitReader (reading the body
rather than trusting Content-Length, so a lying header can't inflate an
image's rank), shares a single lazily-built client, and propagates the
caller's context. extractPics caps concurrency at 8 probes per page and
bounds the whole probe phase with an overall budget (per-image timeout kept
below it) so a page full of slow image URLs can't hold the handler past the
write timeout; semaphore acquisition is context-aware. Lead-image selection
breaks size ties by URL so it's deterministic rather than scheduling-dependent.

Pull Request Pull Request #85: Stream image sizing, share client, honor context, cap concurrency

Coverage Stats

35 of 38 new or added lines in 2 files covered. (92.11%)

1 existing line in 1 file now uncovered.

773 of 993 relevant lines covered (77.84%)

9.51 hits per line