Skip to content
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

Siri times out after three seconds #85

Open
AlexGustafsson opened this issue Jan 9, 2020 · 4 comments
Open

Siri times out after three seconds #85

AlexGustafsson opened this issue Jan 9, 2020 · 4 comments

Comments

@AlexGustafsson
Copy link
Owner

AlexGustafsson commented Jan 9, 2020

In version 4.0.0+ everything is asynchronous and waited for. This includes the grace times. When Siri is used to invoke a command to start a computer, she will time out after three seconds responding that the accessory did not respond. The plugin still works as intended however.

Unfortunately there are only two options, since Siri cannot be configured to wait for longer:

  1. Let the accessory respond immediately which lets Siri work as intended but she may respond with the wrong status. There may be an increased possibility of race conditions resulting in weird behavior.

  2. Let the plugin work as intended and wait for asynchronous code to finish, which will allow the Home app to work as intended, but in turn result in Siri responding that the accessory is not responding. One can play with the grace times in order to get around the Siri issue.

This issue will likely not be fixed (only Apple can change things), rather mitigated by being able to configure which of the above two solutions one wants to use, with the second being the default due to it technically being more correct in terms of plugin function.

@AlexGustafsson
Copy link
Owner Author

AlexGustafsson commented Jan 9, 2020

Version 4.1.1 comes with a configurable value returnEarly which, if set to true, will result in behavior 1 and if set to false (default) will result in behavior 2.

@AlexGustafsson
Copy link
Owner Author

Could this be used to resolve this issue?

@AlexGustafsson
Copy link
Owner Author

Repository owner deleted a comment from flocorbab Apr 11, 2021
@AlexGustafsson
Copy link
Owner Author

Moved a comment to #155.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant