koush / scrypted

Scrypted is a high performance video integration and automation platform
https://scrypted.app
Other
3.9k stars 237 forks source link

homekit: fix late 2way setup #1461

Closed xdissent closed 1 month ago

xdissent commented 1 month ago

When my ring doorbell sets up a streaming session, the first message on the audio return channel isn't a request to setup 2way. In that case, the "once" handler would fire (and bail) and when the real 2way setup message would arrive, the setup listener was no longer registered and nothing would happen. This PR moves the 2way setup into the always-on listener, and creates a guard to prevent multiple setup requests from running concurrently. It should also allow retrying the setup if something temporarily went wrong on a previous attempt.