Skip to content

Fix delegated routing streaming#235

Open
byo wants to merge 3 commits intomainfrom
fix-delegated-routing-streaming
Open

Fix delegated routing streaming#235
byo wants to merge 3 commits intomainfrom
fix-delegated-routing-streaming

Conversation

@byo
Copy link
Copy Markdown

@byo byo commented Apr 22, 2026

No description provided.

byo added 2 commits April 22, 2026 22:13
Processing does not end when `doFindStreaming` method returns
because the output channel will be populated with data by
additional goroutines. It's the responsibility of the caller
to ensure the context is canceled when no more data is needed.

The caller method  no need for additional cancelation in the caller
because it is already a context bound to the http request.
That context is automatically canceled shortly after the request
handler returns.
@byo byo requested review from gammazero and willscott April 22, 2026 20:15
@byo byo marked this pull request as ready for review April 22, 2026 21:50
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.

1 participant