-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[onkyo] Support for more audio streams through the HTTP audio servlet #15117
Conversation
@dalgwen for information and feedback |
791d2bc
to
99b30a0
Compare
3d914ce
to
950dedf
Compare
LGTM : the new capabilities are used as intended. |
Do we set the timeout for 5 or to 10 for more security ? |
By security, I finally set the timeout to 10s. @kaikreuzer : this is ready for a final review. |
Related to openhab#15113 Signed-off-by: Laurent Garnier <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo found.
...nhab.binding.onkyo/src/main/java/org/openhab/binding/onkyo/internal/OnkyoHandlerFactory.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Laurent Garnier <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks!
…openhab#15117) * [onkyo] Support for more audio streams through the HTTP audio servlet Related to openhab#15113 Signed-off-by: Laurent Garnier <[email protected]> Signed-off-by: Jørgen Austvik <[email protected]>
Related to #15113
Signed-off-by: Laurent Garnier [email protected]