Skip to content

fix: bound and cancel Nominatim requests - #179

Merged
atomantic merged 3 commits into
mainfrom
claim/issue-160
Aug 30, 2026
Merged

fix: bound and cancel Nominatim requests#179
atomantic merged 3 commits into
mainfrom
claim/issue-160

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

  • Bound every Nominatim attempt, including response-body parsing, to a 15-second deadline so stalled upstream connections cannot poison the shared queue.
  • Propagate SSE disconnect cancellation through batch geocoding and the active fetch, without persisting cancelled work or emitting a false completion event.
  • Preserve the existing 429 backoff/retry behavior and add focused lifecycle and stream-disconnect tests.

Verification

  • npx vitest run tests/unit/services/geocode.service.spec.ts tests/unit/routes/mapGeocodeStream.spec.ts
  • npm run build

Closes #160

@atomantic
atomantic merged commit 5fc95a1 into main Aug 30, 2026
4 checks passed
@atomantic
atomantic deleted the claim/issue-160 branch August 30, 2026 23:38
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.

Time-bound and cancel queued Nominatim requests

1 participant