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
[magentatv] Adapt to new Telekom OAuth flow (required to retrieve the userId) #10267
Conversation
Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Markus Michels <markus7017@gmail.com>
aca4a09
to
f72d99e
Compare
@openhab/add-ons-maintainers Could someone pick-up this small PR? Telekom changed the auth flow, so new users are no longer able to signin with the current version included with the distro. This PR fixes the problem and should be easy to review |
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.
Thanks @markus7017, just some small comments.
...magentatv/src/main/java/org/openhab/binding/magentatv/internal/handler/MagentaTVHandler.java
Outdated
Show resolved
Hide resolved
...g.magentatv/src/main/java/org/openhab/binding/magentatv/internal/network/MagentaTVOAuth.java
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.magentatv/src/main/resources/OH-INF/binding/binding.xml
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.magentatv/src/main/resources/OH-INF/thing/thing-types.xml
Show resolved
Hide resolved
Signed-off-by: Markus Michels <markus7017@gmail.com>
@kaikreuzer review changes applied |
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.
Perfect, thank you!
… userId) (openhab#10267) Signed-off-by: Markus Michels <markus7017@gmail.com> Signed-off-by: John Marshall <john.marshall.au@gmail.com>
… userId) (openhab#10267) Signed-off-by: Markus Michels <markus7017@gmail.com>
… userId) (openhab#10267) Signed-off-by: Markus Michels <markus7017@gmail.com>
… userId) (openhab#10267) Signed-off-by: Markus Michels <markus7017@gmail.com>
This PR adapted to the new OAuth flow, current version is not able to retrieve the userId from the authenticated account.