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
[homematic] Add Authentication #16196
Conversation
Signed-off-by: Christian Kittel <ckittel@gmx.de>
f3f9fa5
to
e4e0fa2
Compare
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/authentication-for-api-access-to-ccu/152635/1 |
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.
Left some comments, nothing big. Thanks.,
...matic/src/main/java/org/openhab/binding/homematic/internal/common/AuthenticationHandler.java
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/i18n/homematic_de.properties
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/thing/bridge.xml
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/thing/bridge.xml
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/thing/bridge.xml
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/thing/bridge.xml
Outdated
Show resolved
Hide resolved
...matic/src/main/java/org/openhab/binding/homematic/internal/common/AuthenticationHandler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Christian Kittel <ckittel@gmx.de>
Thanks for your remarks. Changed the error handling a little bit |
5314c18
to
bdd75ac
Compare
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.
My last comments, would be nice if @gerrieg can also have a look as codeowner.
...matic/src/main/java/org/openhab/binding/homematic/internal/common/AuthenticationHandler.java
Outdated
Show resolved
Hide resolved
...tic/src/main/java/org/openhab/binding/homematic/internal/handler/HomematicBridgeHandler.java
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.homematic/src/main/resources/OH-INF/i18n/homematic.properties
Outdated
Show resolved
Hide resolved
324cdc3
to
6af7885
Compare
…eption Signed-off-by: Christian Kittel <ckittel@gmx.de>
6af7885
to
b3ef54d
Compare
all done |
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, ping @gerrieg.
* Add Authentication --------- Signed-off-by: Christian Kittel <ckittel@gmx.de> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Description
This PR enable authorization (resolves #8625 ) when accessing Homematic Server (like CCU). I've added three new binding configuration values to enable this. Now it's possible to disable unauthorized accesses to the CCU. I added some documentation which is relevant for testing.
Testing
I tested the new feature with raspberrymatic and OH running in a K8s environment. I thinks it's necessary to test it with other settings.