Freegle / Iznik / 25989

72%

master: 72%

Build Type

Pull #1027

circleci

Committed by invalid-email-address

Commit Message

fix(remap): don't abort nightly postcode remap on a spatial-server blip

PostcodeRemapService::findNearestArea handled bad HTTP status codes
gracefully (log + return null => skip the postcode) but not connection
exceptions. A ~1s blip while spatial-knn was mid-rebuild made Http::get
throw an uncaught ConnectionException that propagated through the
chunkById loop and killed the whole nightly remap (Sentry BATCH-60,
01:34 2026-07-10). No data corruption — the caller only writes on a
non-null area — but the run died partway.

Wrap the Http::get in try/catch (ConnectionException) and return null,
matching the existing "skip this postcode, keep going" contract. Add a
regression test for the connection-refused path.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D2mZBRnmwiaucKihbjgHwb

Pull Request Pull Request #1027: fix(remap): don't abort nightly postcode remap on a transient spatial-server blip

Coverage Stats

12939 of 17048 branches covered (75.9%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 1 file covered. (100.0%)

340 existing lines in 10 files now uncovered.

135413 of 188518 relevant lines covered (71.83%)

37.97 hits per line