reworked the connect workaround

abandonware / node-bluetooth-hci-socket

MIT License

42 stars 47 forks source link

reworked the connect workaround #34

Closed splitice closed 2 years ago

splitice commented 2 years ago

This version of the connect/disconnect workaround uses C++ to track ownership of the L2CAP handle and includes a expiration of connections that do not complete (hard coded 60s)

Fixes a fd leak on re-binding / binding close and a fd leak on re-use of a handle.
Includes support for out of order connect/disconnect events (previously worked only due to leak, my broadcom devices do this)

splitice commented 2 years ago

Support for unsupported version of node e.g node 10 is not within my specification.

Anyway we have been running this in production for months now. It works well on broadcom chips and unlike the original doesnt rely on leaked sockets.