-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
(Re-) discovered devices should reset peripheral status? #19
Comments
Please provide steps to reproduce this issues you have created are very lazy. Will re-open when proper steps to reproduce are provided |
Thats not that easy ... I easiely could start a new scanning for device and such and when the device was then rediscovered the status stayed "connecting". But I was - after the rediscovery - able to connect to the device. After this the state was updated. I mainly refer to https://github.com/stoprocent/noble/blob/main/lib/noble.js#L227-L245 When the peripheral which is discovered is already known some values get updated, but the peripheral state stays untouched. So my main thing was if my assumptions above are correct (about the only connectable or discoverable) then also setting peripheral.status="disconnected" should be ok here in the case of existing peripherals I mainly had this issue with the Shelly device where I did not correctly got the Matter handshake response (other issue) and so my retry mechanisms lead to this sometimes |
Ahh check the log in the main post at #13 ... there at the very end is such a "connect" that never resolved ... after this the peripheral has this "connecting" state and will also stay that ways when a new scan re-discovers the device because the state is not "re-set" on discovery and the peripheral is already known |
Can you clean up issues because im lost a bit. You dropped suddenly 7 issues and somehow they are all related. Please clean it up |
When I read corrrectly a peripheral which is connected can not send "connectable announcements". If this is true then a re-discovery of a device which is connectable should reset the peripheral state to "disconnected". Or!?
This could clean strange intermediate states like from #17
The text was updated successfully, but these errors were encountered: