New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix/cancellable-queries #80

Merged

einari merged 3 commits into main from fix/cancellable-queries

Jan 12, 2024

Contributor

einari commented Jan 10, 2024 •

edited

Loading

Fixed

Fixing so that if a client fetches from the same query multiple times and the response could potentialle come out of order, we make sure to cancel any ongoing requests.
Adding a cancellation token for ObservableClient queries that we pass along to every WebSocket operation in case the client disconnects.

einari added 2 commits

January 10, 2024 13:31


          Adding a cancellation token that we cancel when we disconnect, to avo…

b7841a0

…id sending things to the client after it is disconnected


          Making use of AbortController and its signal for the fetch operations…

60b9532

…, so that we guarantee order of result from query calls

einari added the patch label


          Adding using statement for the CTS

3cbcc3b

einari merged commit c662dc8 into main

5 checks passed

einari deleted the fix/cancellable-queries branch

January 12, 2024 12:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels