Skip to content
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 Binding] Search for things failed under Windows 10 for most devices #1244

Closed
MHerbst opened this issue Sep 19, 2016 · 9 comments
Closed
Assignees

Comments

@MHerbst
Copy link
Contributor

MHerbst commented Sep 19, 2016

Tested with openHAB 2.0b4

When I search for Homematic devices on my Windows 10 PC (Java 1.8.102 64 Bit) I get a these error messages:
20:05:22.954 [WARN ] [ternal.communicator.HomematicGateway] - Can't load device with address 'MEQ0798777' from gateway 'MHBridge': Ung³ltiges Byte 1 von 1-Byte-UTF-8-Sequenz.
In english: Invalid byte 1 of 1-byte-utf-8-sequence.

Only my door contact devices are found. Other devices like temperature sensor are not found.

On my Raspberry Pi everything works fine.

@kaikreuzer
Copy link
Member

@gerrieg Could you comment?

@itn3rd77
Copy link

Hi @MHerbst ,

please have a look at the Homematic binding readme. Scroll down to the end and see if this solves your problem.

Ung?ltiges Byte 1 von 1-Byte-UTF-8-Sequenz
Your LOCALE is wrong, make sure it is set to UTF-8

@MHerbst
Copy link
Contributor Author

MHerbst commented Sep 20, 2016

Hi @itheiss ,
thank you for pointing me to the hint in the README file. But unfortunately it is a bit short. In a first attempt I tried to change the locale setting in Paper UI. But as UTF-8 is neither valid as language nor as country I got an error message in the console (BTW: no error message in Paper UI).
Exactly speaking UTF-8 is an encoding and not a locale. I have solved it by setting this environment variable:

set JAVA_TOOL_OPTIONS=-Dfile.encoding=UTF8

I would be a good idea to add this information to the readme.

As far as I can see UTF-8 is required for the communication with the Homematic CCU. So the best solution would be to set UTF-8 as encoding within the Homematic binding.

@kaikreuzer
Copy link
Member

@gerrieg @itheiss Is this something that can be set within the Homematic binding?
Or shall we consider to set JAVA_TOOL_OPTIONS in the distro launch scripts directly? It is the first time, I come across this variable, so I don't know what negative effects this might have...

@MHerbst
Copy link
Contributor Author

MHerbst commented Sep 20, 2016

In order to avoid side-effects on other bindings I would prefer a binding internal solution.
There is a larger discussion on Stackoverflow https://stackoverflow.com/questions/361975/setting-the-default-java-character-encoding

Maybe I can find some time the next days to look into the binding's implementation and test whether it is possible inside the binding.

@maggu2810
Copy link

IIRC: The quality tools I use for my tools complains if I use a function that relies on the system locale and there is also another function provided by the Java API that could handle different character sets.
As I don't know your code I cannot grant that there is a function, but if some code relies on UTF-8 you should set this explicit.
If there is another bundle that relies on another character encoding...
I would prefer to fix the code and not rely on the global character encoding at all.

@gerrieg
Copy link
Contributor

gerrieg commented Sep 21, 2016

I also prefer to fix it in the binding. I tried to start the binding on my development environment with -Dfile.encoding=Cp1250 and i can now reproduce the problem!
Currently i have no idea whats causing this but i can play with it now and i'll try to fix it.

@gerrieg
Copy link
Contributor

gerrieg commented Sep 22, 2016

Can you try this please: https://github.com/openhab/openhab2-addons/pull/1256

This works for me regardless of the file.encoding settings

@MHerbst
Copy link
Contributor Author

MHerbst commented Sep 22, 2016

I have tested in in my Windows 10 environment and works correctly now. Problem solved.

Thanks for the fast fix.

@MHerbst MHerbst closed this as completed Sep 22, 2016
Flole998 pushed a commit to Flole998/openhab-addons that referenced this issue Dec 30, 2021
markus7017 pushed a commit to markus7017/openhab-addons that referenced this issue Aug 12, 2023
* Update README (2.5.x) (openhab#1153)

Change branch name.

Signed-off-by: Yannick Schaus <[email protected]>

* Update items.md (openhab#1156)

* Added var and VA units to UoM (openhab#1146)

VA (Volt-Ampere - apparent power) and var (Volt-Ampere reactive) are used to measure power and energy consumption in AC circuits.


Signed-off-by: Nagy Attila Gabor <[email protected]>

* Fix filepath to keystore (openhab#1148)

Default openHAB userdata environment variable should be `$OPENHAB_USERDATA`, not `$USER_DATA` shouldn't it? At least, this is the default on my fresh openHABian and also the most popular variant to find in the docs.

* Slight language corrections (openhab#1150)

I think it reads better this way

Signed-off-by: Richard Davies <[email protected]>

* additional example for non default persistence service (openhab#1152)

For me it was confusing how to pass on the serviceId into methods that already had an argument. An extra example is always good.

Signed-off-by: jaco <[email protected]>

* Adding 12 new logos for OH Add-Ons page on website (openhab#1158)

Signed-off-by: bracklanna [email protected]

* Added missing preset variables (openhab#1104)

* Added missing preset variables

Signed-off-by: Scott Rushworth <[email protected]>

* Cleaned up blank lines, fixed table, and added file name for SimpleRule

Signed-off-by: Scott Rushworth <[email protected]>

* Fix broken link (openhab#1165)

* Added Hotlink from "label" section to "state presentation" (openhab#1167)

* Added note about broken action (openhab#1164)

* Added note about broken action

See openhab/openhab-core#1374

Signed-off-by: Christoph Weitkamp <[email protected]>

* Incorporated changes from review

Signed-off-by: Christoph Weitkamp <[email protected]>

* Incorporated changes from review

Signed-off-by: Christoph Weitkamp <[email protected]>

* Update index.md (openhab#1170)

Link appears to be wrong and does not work when I click on it in Edge. Loads the same page again instead of loading the correct new page from the hyperlink.

https://www.openhab.org/docs/developer/guidelines.html

* Added Airthings logo (openhab#1171)

* typo in exambp (openhab#1172)

`Temperature.averageSince(now.minusMinutes(5),"influxdb")`

* file.encoding=UTF-8 (openhab#1173)

* Update demo URL and add demo.rules URL (openhab#1174)

Based on: https://community.openhab.org/t/demo-setup-missing/94850
Old Link is broken leading to 404.
The link to the demo.rules on github is an extra :)

* Replace outdated zulu.org link. (openhab#1177)

* Replace outdated zulu.org link.

As of 3/23/2020 zulu.org has an SSL cert that expired on 9/28/2019. Changed link to azul.com/downloads, since that appears to be the new official source.

Signed-off-by: Billy Stevens <[email protected]>

* Changed all http links to https for installation/index.md.

All changed links working, tested on 3/24/2020.

Signed-off-by: Billy Stevens <[email protected]>

* Minor language tweak (openhab#1178)

* Ending an active scan/stopScan (openhab#1179)

Signed-off-by: Mark Theiding <[email protected]>

* Add files via upload (openhab#1184)

* Update persistence.md (openhab#1185)

Clarify return objects for max/min rules extensions.

Signed-off-by: Ross Kennedy [email protected]

* Update things.md (openhab#1186)

Amended example code to include using label and location when defining a Thing with a bridge that is defined elsewhere.

* Correct typos (openhab#1190)

* Correct usage of its/it's

"It's" is always a contraction of "it is" or "it has".  "Its" is a
possessive.  Correct a few places where they were used backwards.

Signed-off-by: Bjorn Helgaas <[email protected]>

* Correct "Z-Wave" spelling

Per https://www.z-wave.com/, the canonical spelling appears to be "Z-Wave".
Most places use "Z-Wave" already; change the remaining references to match.

Signed-off-by: Bjorn Helgaas <[email protected]>

* Correct typos and grammatical errors

Correct some typos and grammatical errors.

Signed-off-by: Bjorn Helgaas <[email protected]>

* Update sitemap.md section charts (openhab#1191)

I observed that the unique first word in the labels of items charted in a group isn't causing an empty chart anymore. I'm on openHAB 2.5.1.

Signed-off-by: Juergen Baginski [email protected]

* Add image for insteon binding (openhab#1196)

Signed-off-by: Rob Nielsen <[email protected]>

* typo (openhab#1198)

Signed-off-by: Mark Theiding <[email protected]>

* Installation details (openhab#1197)

Added more details around the installation and configuration process.
Fixed that engine no longer logs "Activated scripting support..."

Signed-off-by: Mark Theiding <[email protected]>

* Update sitemaps.md (openhab#1202)

Added full item definition for usage of visibility. See https://community.openhab.org/t/sitemap-visibility-basic-ui/97304/9

* Updated ecobee logo (https://brand.ecobee.com/) (openhab#1203)

Signed-off-by: Rob Nielsen <[email protected]>

* tutorial: Fix description of sitemap 'type' (openhab#1204)

In the tutorial, the generic sitemap description says that ItemType has
to be the same as the type defined in default.items.
Looking at
https://www.openhab.org/docs/configuration/items.html#type and
https://www.openhab.org/docs/configuration/sitemaps.html#element-types
this is incorrect as they take different values.
The example is even mislading as `Switch` is one of the only types which
is common between items and sitemaps. Might be better to describe
`Default` instead.

Signed-off-by: Christophe Fergeau <[email protected]>

* Added information about DateTime Group functions LATEST/EARLIEST (openhab#1206)

Signed-off-by: Christoph Weitkamp <[email protected]>

* Add section for documentation contributions (openhab#1205)

Hopefully this will lower the hurdle for people to submit documentation contributions. I know from myself that I didn't submit various documentation improvements, because I didn't know git and thought it would be a much more involved process. 
Ideally there would be a separate documentation section, but submitting this under the development contribution page for now (as per discussion with @Confectrician in openhab/openhab-docs#1179 (comment)).
Note that I am addressing the issue of DCO failures wrt specifying the full name that I ran into myself in openhab/openhab-docs#1197 (comment). I found a good discussion of the issue at dcoapp/app#43.

Signed-off-by: Mark Theiding <[email protected]>

* fix typo (openhab#1209)

* add description of Ephemeris localization support (openhab#1210)

Add a new section to describe the localization support and how-to steps

Signed-off-by: Michael Roßner [email protected]

* Line 115 broken link - should be: (openhab#1217)

* Line 115 broken link - should be:

({{base}}/docs/configuration/sitemaps.html#element-types)

was:
({{base}}/configuration/configuration/sitemaps.html#element-types)

* Removed diplicated docs breadcrumb

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* add missing space between words (openhab#1212)

* Update configuration.md (openhab#1215)

I'm a beginner myself. Though I liked this tutorial very much, it took me some time trying and erroring and finally reading forum posts to get behind this. I didn't even know there was something like a more modern ping. So maybe others are happy to learn this right from the beginning.

* Remove architecture from Docker tags (openhab#1220)

Docker automatically detects the architecture and downloads the appropriate image (openhab/openhab-docker#213).
BuildKit will no longer generate new tags having the architecture (openhab/openhab-docker#293).

Signed-off-by: Wouter Born <[email protected]>

* slight readability improvements (openhab#1221)

* slight readability improvements

* Update introduction.md

* Update introduction.md

* minor wording update

* Update eclipse.md (openhab#1225)

Clarifying that it's no longer possible to make changes in the Core Framework for 2.5.x.

Signed-off-by: Mark Theiding <[email protected]>

* [fmiweather] logo for FMI Weather binding (openhab#929)

Signed-off-by: Sami Salonen <[email protected]>

* Update eclipse.md (openhab#1226)

Added additional structure around install, run, debug and update steps. Provided more pointers to interactions with Eclipse, Maven and Git.

Signed-off-by: Mark Theiding <[email protected]>

* Update contributing.md (openhab#1227)

Need to escape \< and \> in the sign off message format so users see them explicitly in the Contributing to the Documentation section. 

Signed-off-by: Mark Theiding <[email protected]>

* Update contributing.md (openhab#1228)

Small refinement on documentation change submission flow. 

Signed-off-by: Mark Theiding <[email protected]>

* Add doc folder to the binding directory structure (openhab#1230)

Signed-off-by: Fabian Wolter <[email protected]>

* Make Subheadings Use Proper Subheading Syntax (openhab#1234)

This way they render out as proper markdown and don't look weird on the website

Signed-off-by: Stefan Zabka <[email protected]>

* Remove unnecessary isCancelled() from code example (openhab#1235)

Cancelling an already canceled task has no effect. IMHO this check is not necesssary and removal would simplify the code. I came to this because I saw this pattern in many bindings during reviewing.

Signed-off-by: Fabian Wolter <[email protected]>

* Update thing-xml.md (openhab#1236)

Signed-off-by: Christoph Weitkamp <[email protected]>

* Fix broken ESH links (openhab#1231)

Signed-off-by: Wouter Born <[email protected]>

* Update logging.md (openhab#1238)

Add information on how to find out the symbolic names of the bundles

* Remove Apache Commons from Default Libraries (openhab#1229)

See openhab#7722
Signed-off-by: Fabian Wolter <[email protected]>

* Update introduction.md (openhab#1239)

* Update introduction.md

Signed-off-by: Markus Storm [email protected]

* Update introduction.md

* Revise Java recommendations (openhab#1240)

* Revise Java recommendations

* Delete pine.md

Do not recommend PINE, it's not supported any longer by openHABian.

* Removed sidebar link in config

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* Update security.md (openhab#1241)

Been using FreeDNS for many years (ever since all these companies got rid of their free tiers) and never an issue!

* Fix DecimalType hex conversion example (openhab#1243)

See: openhab/openhab-core#1526

Signed-off-by: Wouter Born <[email protected]>

* Fix typo (openhab#1244)

Signed-off-by: Wouter Born <[email protected]>

* Update persistence.md (openhab#1246)

Fixes link to quartz docs page.

* Revision. (openhab#1187) (openhab#1237)

* Revision. (openhab#1187)

- Update of screenshots, removal of old screenshots
- Chapters for better formatting
- Removal of ZWave chapter (one example of adding things should be enough IMHO)
- Adding items in simple mode and in "manual" mode

Signed-off-by: Sascha Billian <[email protected]>

* Use one line per sentence
Signed-off-by: Sascha Billian <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* Add notes for configuring Synology Diskstation (openhab#1219)

* Add notes for configuring Synology Diskstation

I have a working set up for SSL enabled remote access on a Synology diskstation, taking advantage of the GUI as much as possible, to ensure automatic renewal of certs from Let's Encrypt, etc. It took me about 8 hours to suss it all out, but it could be achieved in about 30 mins if you knew exactly what to do... may not be widely useful, but since Synology is officially supported, I figured this might be a good addition.

There's also a minor error in the 'allow' masks - these should be 192.168.0.0/24 to allow access to anything in the 192.168.0.xxx range.

* Updated to use one line per sentence

Updated to use one line per sentence - sorry for the delay!

* Update security.md

* Updated for one line per sentence

Updated for one line per sentence

Signed-off-by: Andrew Mills [email protected]

* Bad subnet (openhab#1245)

Nginx warns about low address bits of `192.168.0.1/24` because they are meaningless.
The correct subnet mask should be `192.168.0.0/24`

Signed-off-by: Olivier Béraud <[email protected]>

* Fixed broken images. (openhab#1247)

* Fixed broken images.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix image path

Signed-off-by: Jerome Luckenbach <[email protected]>

* [documentation] clarification of representation property (openhab#1248)

* [documentation] clarification of representation property

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] typo

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] adopt suggestions of reviewers

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] commas

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] typo

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] addopted suggestions of @bobadair

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] typo

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentaion] example added back

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentaion] simplified text

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] typo

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* [documentation] adopted reviewer comment

Signed-off-by: Andrew Fiddian-Green <[email protected]>

* Add Alexa mapping along side a channel mapping (openhab#1249)

* Add Alexa mapping along side a channel mapping

It took me a while to find this https://community.openhab.org/t/tagging-devices-for-alexa-support/98155/3 on the Forum and its not clearly documented in the openHAB Amazon Alexa Smart Home Skill or here in Item Metadata.
I originally suggested this as an update to the openHAB Amazon Alexa Smart Home Skill documentaion, but it fits better here, then other integrations using metadata (e.g. HomeKit or Google Assistant) could refer to it as well.

* Update items.md

* Mention defaults for element type setpoint. (openhab#1250)

Mention defaults for min, max and step value for element type setpoint.

Signed-off-by: Thomas Weiler <[email protected]>

* Update index.md (openhab#1251)

I thought 'workl' was probably intended to be 'work'.

* Items - Bedroom_Light written as Light_Bedroom (openhab#1252)

Fix small error which might mislead some readers.

* Added example for time-weighted averages (openhab#1253)

Signed-off-by: Christoph Weitkamp <[email protected]>

* Remove deprecated UIs, Eclipse Marketplace from sidebar

Signed-off-by: Yannick Schaus <[email protected]>

* Update branch name in README

Signed-off-by: Yannick Schaus <[email protected]>

Co-authored-by: Markus Storm <[email protected]>
Co-authored-by: Nagy Attila Gábor <[email protected]>
Co-authored-by: Christoph Thiede <[email protected]>
Co-authored-by: Richard Davies <[email protected]>
Co-authored-by: jwaes <[email protected]>
Co-authored-by: bracklanna <[email protected]>
Co-authored-by: Scott Rushworth <[email protected]>
Co-authored-by: cpmeister <[email protected]>
Co-authored-by: Ross Kennedy <[email protected]>
Co-authored-by: Christoph Weitkamp <[email protected]>
Co-authored-by: Skinah <[email protected]>
Co-authored-by: pali <[email protected]>
Co-authored-by: ljsquare <[email protected]>
Co-authored-by: PatrikG <[email protected]>
Co-authored-by: Elias H <[email protected]>
Co-authored-by: Billy Stevens <[email protected]>
Co-authored-by: theiding <[email protected]>
Co-authored-by: jadcx <[email protected]>
Co-authored-by: Bjorn Helgaas <[email protected]>
Co-authored-by: Jürgen Baginski <[email protected]>
Co-authored-by: robnielsen <[email protected]>
Co-authored-by: GumbyMan82 <[email protected]>
Co-authored-by: Christophe Fergeau <[email protected]>
Co-authored-by: Paulo "JCranky" Siqueira <[email protected]>
Co-authored-by: Michael Rossner <[email protected]>
Co-authored-by: BugSmurF <[email protected]>
Co-authored-by: Jerome Luckenbach <[email protected]>
Co-authored-by: josefscript <[email protected]>
Co-authored-by: Wouter Born <[email protected]>
Co-authored-by: Sami Salonen <[email protected]>
Co-authored-by: Fabian Wolter <[email protected]>
Co-authored-by: Stefan Zabka <[email protected]>
Co-authored-by: TRS-80 <[email protected]>
Co-authored-by: sihui <[email protected]>
Co-authored-by: Andrew Mills <[email protected]>
Co-authored-by: Olivier Béraud <[email protected]>
Co-authored-by: Andrew Fiddian-Green <[email protected]>
Co-authored-by: LeeC77 <[email protected]>
Co-authored-by: Thomas Weiler <[email protected]>
Co-authored-by: garretcook <[email protected]>
Co-authored-by: Michael Fielding <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants