Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

[Homematic] Restrict Bin/XmlRpcServer to specified callback host address #5928

Merged
merged 1 commit into from
Jul 18, 2018

Conversation

mdicke2s
Copy link
Contributor

Open Server on host specified in the property "callbackHost" instead of
opening it without specification. This will prevent conflicts on machines which provide multiple network interfaces.

Further improvements for stability of the communication between binding and gateway:

  • Improved error logging of ConnectionTrackerThread
  • Synchronized AbstractHomematicGateway's start/stopServers to prevent starting of new servers while previous operation is still blocking

REMARK:
Actually this change was already in-place on openhab (see openhab/openhab2-addons@357036c#diff-34d39d6e0ca3033b15f157f32c370cac),
but it was accidentially not included when moving the binding to the eclipse smarthome project

Signed-off-by: Michael Reitler michael.reitler@telekom.de

Open Server on host specified in the property "callbackHost" instead of
opening it without specification. This will prevent conflicts on machines which provide multiple network interfaces.

Further improvements for stability of the communication between binding and gateway:

* Improved error logging of ConnectionTrackerThread
* Synchronized AbstractHomematicGateway's start/stopServers to prevent starting of new servers while previous operation is still blocking

REMARK:
Actually this change was already in-place on openhab (see openhab/openhab2-addons@357036c#diff-34d39d6e0ca3033b15f157f32c370cac),
but it was accidentially not included when moving the binding to the eclipse smarthome project

Signed-off-by: Michael Reitler <michael.reitler@telekom.de>
@mdicke2s mdicke2s closed this Jul 18, 2018
@mdicke2s mdicke2s reopened this Jul 18, 2018
@mdicke2s mdicke2s closed this Jul 18, 2018
@mdicke2s mdicke2s reopened this Jul 18, 2018
Copy link
Contributor

@htreu htreu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for porting this one.

@htreu htreu merged commit ea77604 into eclipse-archived:master Jul 18, 2018
@mdicke2s
Copy link
Contributor Author

Thanks @htreu

@htreu htreu added this to the 0.10.0 milestone Oct 30, 2018
@kaikreuzer kaikreuzer changed the title [homematic] Reduce Bin/XmlRpcServer to specified callback host address [Homematic] Restrict Bin/XmlRpcServer to specified callback host address Dec 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants