On server timeouts, it's possible for multiple minutes to elapse
between receiving the last pong on the old connection and attempting to
send the next ping on the new connection. If that time span exceeded 5
minutes, the binding went into a minutely reconnection loop, because it
never attempted to send a ping anymore, which led to lastPong not being
updated anymore.
To fix this, replace the timing dependent timestamp handling by a simple
counter which counts how many consecutive ping attempts didn't receive
an answer, and closing the connection after 5 unsuccessful ping
attempts. That new counter is now also reset whenever the connection is
restarted.
Fixes#14188
Signed-off-by: Danny Baumann <dannybaumann@web.de>
* Removed provider header, no longer necessary
* Fixed connection tracker not starting after some websocket restarts
Signed-off-by: Gerhard Riegler <gerhard.riegler@gmail.com>
* Adaptation of the login for API version 1.1
* Fixed logging statement
* Fixed logging message
* Reverted changes in the german translation
Signed-off-by: Gerhard Riegler <gerhard.riegler@gmail.com>
* Fix compiler warning
* Add basic test coverage for DataItem deserialization
* Add full prefixes to attributes variables
* Add missing newlines at end of test payload files
* Add full prefix to attributes variable
Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
* [gardena] Fix handling of websocket connection losses that causes memory leaks
* The binding no longer restarts websockets more than once if the connection is lost
Signed-off-by: Nico Brüttner <n@bruettner.de>
* New translations tradfri.properties (Italian)
* New translations mycroft.properties (French)
* New translations deutschebahn.properties (German)
* New translations gardena.properties (German)
* New translations luftdateninfo.properties (German)
* New translations network.properties (Finnish)
* New translations publictransportswitzerland.properties (Finnish)
* New translations mycroft.properties (Finnish)
* New translations pushover.properties (German)
* New translations yamahamusiccast.properties (Italian)
* New translations ecotouch.properties (German)
* New translations voicerss.properties (French)
* Add default translations for binding add-ons
This makes the texts used by these add-ons translatable with Crowdin.
To keep the PR simple, it only adds default translations for add-ons which do not yet have any default translations properties file.
We can do follow up PRs for adding missing key/values to add-ons that already have these files or to remove duplications.
There are several add-ons in this PR that do have non-English translation files, so I'll upload those to Crowdin when the PR is merged.
Signed-off-by: Wouter Born <github@maindrain.net>
* Switch properties files encoding from ISO-8859-1 to UTF-8
Crowdin will generate translations in this encoding, so this prevents a lot of diffs which makes it easier to review future Crowdin PRs.
Related to openhab/openhab-core#2298
Signed-off-by: Wouter Born <github@maindrain.net>
