Skip to content
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.

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

Added check for Stateview, added socket cancel to finish all connect,… #419

Open
wants to merge 1 commit into
base: minor
Choose a base branch
from

Conversation

SaltySnail
Copy link
Collaborator

Fix for some race condition related crashes of the server.

By creating this pull request, I understand that code that is AI generated or otherwise automatically generated may be rejected without further discussion.
I declare that I fully understand all code I pushed into this PR, and wrote all this code myself and own the rights to this code.

@lionkor
Copy link
Contributor

lionkor commented Jan 25, 2025

Question from the sidelines; boost docs say that close() already cancels

Use the close() function to simultaneously cancel the outstanding operations and close the socket.

What does this do internally, as we already close() right after?

Wondering since I use this API in other projects lol

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.

2 participants