RFC2217 improvements

[wborn]

• Rework workaround so binding can use undiscovered RFC2217 ports
• Catch UnsupportedCommOperationException

See also:

• community
• Workaround causes serial port discovery issues #1332
• RFC2217 URI does not work in OH 2.5 openhab-core#1462

[openhab-bot]

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/solved-zwave-zigbee-rfc2217-remote-serial-port-howto/81633/69

[bodiroga]

The code looks good! I will try to pick up a Raspberry Pi and test it in the Eclipse IDE!

Thank you @wborn!

[bodiroga]

Hi @wborn!

In my own fork I have created a branch with even more improvements for the RFC2217 connection, here you have a link to it: https://github.com/bodiroga/org.openhab.binding.zwave/tree/rfc2217-improvements

How can we merge both changes? Can I make a pull request to your own branch? Can you manually add my changes?

@cdjackson, I can confirm that this improvements work great, and with my additions, it works even better. Finally we have a RFC2217 native implementation 😉

Thanks guys!

[RafalLukawiecki]

I am interested in any improvements to the RFC2217 reconnection logic, many thanks for your work.

[cdjackson]

Thanks @bodiroga . I don't have any way to test this so I'll merge based on your tests. What I'd suggest is that if you've created changes on top of this PR, then you now create a new PR (adding this also-by signature to reference @wborn and we close this.

Thanks for looking at this - I know there are a few people that will be happy :)

[bodiroga]

Done! #1341

[cdjackson]

Thanks @bodiroga. I'll close this and we can complete the discussion in #1341.