ukeeper / ukeeper-readability / 24314599977

78%

master: 78%

Build Type

Pull #73

github

Committed by paskal

Commit Message

fix: address review feedback on Cloudflare routing PR

- TestGetContentCustom: pass the rule directly to getContent so it actually
  exercises the custom-rule path; the RulesMock.GetFunc setup was dead code
  after getContent stopped looking up rules
- CloudflareRetriever.MaxRetries: remove default substitution for the zero
  value — 0 now means "no retries" as expected. Callers opt into retries by
  setting MaxRetries explicitly; main.go uses the exported CFDefaultMaxRetries
  constant (2)
- README: add cf-route-all to the config table and rewrite the Cloudflare
  section to reflect the opt-in routing model + 429 retry behaviour
- rest.Server.Run: expand the WriteTimeout comment to explain why the 150s
  ceiling is server-wide rather than per-route via http.TimeoutHandler

Pull Request Pull Request #73: Modularise URL retrieval with Cloudflare Browser Rendering support

Coverage Stats

207 of 250 new or added lines in 5 files covered. (82.8%)

1 existing line in 1 file now uncovered.

767 of 984 relevant lines covered (77.95%)

9.39 hits per line