-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Connect / Disconnect workaround #33
Connect / Disconnect workaround #33
Conversation
Please rebase and double check the appveyor or open an issue to track this bot which can be disabled in next change |
068b73e
to
56c6c89
Compare
56c6c89
to
6740d14
Compare
is this easy to fix ? : https://ci.appveyor.com/project/rzr/node-bluetooth-hci-socket/builds/41824764 |
@rzr yes, drop support for node 10 (it's out of support anyway). FYI I'm working on a complete overhaul of this workaround to resolve memory leaks present in both this version and the original (they leak the l2cap sockets). My first attempt wasnt successful as it breaks on certain BT implementations (e.g Broadcom) that output reconnections as:
A re-engineering of the workaround is required to handle cases like this |
This is a re-submission of PR #32