-
-
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
[ipcamera] Fix discovery crashes when networks have access rights issues in docker #15059
Conversation
Signed-off-by: Matthew Skinner <matt@pcmus.com>
Signed-off-by: Matthew Skinner <matt@pcmus.com>
Signed-off-by: Matthew Skinner <matt@pcmus.com>
Signed-off-by: Matthew Skinner <matt@pcmus.com>
This pull request has been mentioned on openHAB Community. There might be relevant details there: |
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! Only one small comment about info logging.
...nding.ipcamera/src/main/java/org/openhab/binding/ipcamera/internal/onvif/OnvifDiscovery.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Matthew Skinner <matt@pcmus.com>
All requested changes have been made. |
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
…ues in docker (openhab#15059) * Fix discovery crashes if network use is blocked by docker. * Change log levels to be more useful. * add better reolink discovery check Signed-off-by: Matthew Skinner <matt@pcmus.com>
…ues in docker (openhab#15059) * Fix discovery crashes if network use is blocked by docker. * Change log levels to be more useful. * add better reolink discovery check Signed-off-by: Matthew Skinner <matt@pcmus.com> Signed-off-by: Matt Myers <mmyers75@icloud.com>
…ues in docker (openhab#15059) * Fix discovery crashes if network use is blocked by docker. * Change log levels to be more useful. * add better reolink discovery check Signed-off-by: Matthew Skinner <matt@pcmus.com> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
This PR fixes the issue reported here that most likely is an access issue caused by the use of docker. Closes #14500
It restricts the binding to only search the network that openHAB has setup as the primary address so if one of your networks causes issues the binding will no longer try to search using all internal networks, but only the selected one.
The PR also improves the discovery of cameras that do not reply with an ONVIF xaddress and will try to discover the brand from the cameras login page.
This pull request will automatically be built and available under the following links if anyone wants to test:
https://openhab.jfrog.io/ui/native/libs-pullrequest-local/org/openhab/addons/bundles/
https://openhab.jfrog.io/ui/native/libs-pullrequest-local/org/openhab/addons/bundles/org.openhab.binding.ipcamera/4.0.0-SNAPSHOT/org.openhab.binding.ipcamera-4.0.0-SNAPSHOT.jar
Signed-off-by: Matthew Skinner matt@pcmus.com