This repository has been archived by the owner on Jan 15, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 6
Implement a proper queue for handling BT handshakes #23
Comments
@evabishchevich / @vasilevskayaem I've always wondered if this was a problem. Should we handle handshakes serially? |
@evabishchevich / @vasilevskayaem I've been waiting 2 months for an answer. Can I please get one? |
@evabishchevich / @vasilevskayaem please check the latest @yaronyg comment. |
When library starts handshake it's already have bluetooth sockets created. So client just sends massage to server and waits response. A problem here can be only in speed of sending message (because speed when we have several connections become lower), but I don't think we should worry about it in this case because handshakes messages are not long. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
This PR #14 enabled multiple handshakes to be handled simultaneously. It has been a question mark can this make the overall BT connection process more unreliable in case of many devices.
An improvement that could be tried out would be to implement a proper queue to handle the handshakes.
The text was updated successfully, but these errors were encountered: