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 unsubscribe from optimistic using per subscription flags #276

Closed

FZambia wants to merge 2 commits into master from properly_unsubscribe_from_optimistic

Member

FZambia commented Apr 4, 2024

Relates #275 #274

ThisIsEsh and others added 2 commits

March 28, 2024 18:46


          fix missing unsubscribe frames for optimistic subscriptions if unsubs…

…cription happened between client and server connect frame


          per sub _optimisticallySent, disable optimistic for http_stream

2cec5b9

FZambia mentioned this pull request

Fix unsubscribe from optimistic inflight subscription #275

Closed

Member Author

FZambia commented Apr 6, 2024

Closing in favour of #278

FZambia closed this

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

Labels

None yet