sandeepmistry / node-bleacon

A Node.js library for creating, discovering, and configuring iBeacons
MIT License
498 stars 88 forks source link

Estimote.discoverAll() leaks and misses discovering devices #34

Closed eric1138 closed 8 years ago

eric1138 commented 8 years ago

Estimote.discoverAll () leaks as it repeatedly discovers the same devices over and over again. Setting SCANDUPLICATES to false causes it to miss when a device goes out of range and returns.

Would be more useful if it returned an array of all currently visible estimote devices as some of the other noble functions do.

sandeepmistry commented 8 years ago

The leak is solved upstream by https://github.com/sandeepmistry/noble-device/pull/22.