-
-
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
[denonmarantz] Add HTTP protocol support for newer receivers #16747
Labels
enhancement
An enhancement or new feature for an existing add-on
Comments
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 11, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 11, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 13, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 13, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 13, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 14, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 14, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 15, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 23, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 24, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 24, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 24, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
jlaur
added a commit
to jlaur/openhab-addons
that referenced
this issue
May 25, 2024
Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
lolodomo
pushed a commit
that referenced
this issue
May 25, 2024
* Add HTTP protocol support for newer receivers Resolves #16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
LeeC77
pushed a commit
to LeeC77/openhab-addons
that referenced
this issue
May 27, 2024
…#16748) * Add HTTP protocol support for newer receivers Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk> Signed-off-by: LeeC77 <lee.charlton00@gmail.com>
psmedley
pushed a commit
to psmedley/openhab-addons
that referenced
this issue
Jun 15, 2024
…#16748) * Add HTTP protocol support for newer receivers Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk> Signed-off-by: Paul Smedley <paul@smedley.id.au>
pgfeller
pushed a commit
to pgfeller/openhab-addons
that referenced
this issue
Sep 29, 2024
…#16748) * Add HTTP protocol support for newer receivers Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk> Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
joni1993
pushed a commit
to joni1993/openhab-addons
that referenced
this issue
Oct 15, 2024
…#16748) * Add HTTP protocol support for newer receivers Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
matchews
pushed a commit
to matchews/openhab-addons
that referenced
this issue
Oct 18, 2024
…#16748) * Add HTTP protocol support for newer receivers Resolves openhab#16747 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I recently purchased a Denon AVC-X4800H as replacement for my 17 year old Denon AVR-3808.
Previously I had not been using the binding, since I need the telnet port for other purposes (mainly Denon/Marantz-plugin for Android).
Now, I thought I'd give the binding a try since I assumed the new receiver would have proper HTTP protocol support.
I quickly discovered that the protocol was accessible through port 8080, but also noticed this remark:
openhab-addons/bundles/org.openhab.binding.denonmarantz/src/main/java/org/openhab/binding/denonmarantz/internal/handler/DenonMarantzHandler.java
Lines 235 to 236 in 7a32f4e
After having added a new Thing, the next discovery was:
Trace logs
I loaded an old app project and found the HTTP implementation I did with help from a user (based on Wireshark captures/debug versions back and forth) almost 10 years ago, and located the interesting part for getting main zone information:
POST http://denon.local:8080/goform/AppCommand.xml
Request payload:
Response:
Using this information it will be possible to adapt the method for retrieving main zone information for at least receivers since 2015. However, the tricky part will be to figure out which receivers supports the current binding's method, and which needs adaptation. There is also this small hint in the README:
I have an old database of models and features, but it's very lacking:
https://vindvejr.dk/apps/denon-plugin/
I will also try to poke my old receiver and verify if it actually does support any of this.
I plan to create an enhancement for supporting my receiver. In the first draft version I imagine I'll make it conditional depending on the configured port number. As end result it should probably be the other way around. If it can be determined that port 8080 is directly related to the new method, and that this is actually a change that happened more than 10 years ago, we could make a more direct option.
During my research so far I also noticed that the binding is quite well written. 👍 Thanks @idserda, @jwveldhuis and others.
Your Environment
The text was updated successfully, but these errors were encountered: