Devices which only require converting to ASCII #53
Replies: 14 comments 10 replies
-
|
This should work in theory since the latest version: #35 (comment) But it's not working in practice for one user, so I would appreciate some help testing it in the real world. |
Beta Was this translation helpful? Give feedback.
-
|
Happy to help, Ive updated to latest version in HACS, enter the IP and get the following message on the integration Retrying setup: Error communicating with API: JSONDecodeError('Extra data: line 1 column 2 (char 1)') |
Beta Was this translation helpful? Give feedback.
-
|
Let me know if you want me to test anything |
Beta Was this translation helpful? Give feedback.
-
|
Can you try retrying the setup, but with debug logs enabled? Like so: logger:
default: warning
logs:
custom_components.candy: debug |
Beta Was this translation helpful? Give feedback.
-
|
Sure, here you go. Tired 2 time after new install 2021-12-14 20:00:03 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration candy which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant |
Beta Was this translation helpful? Give feedback.
-
|
Hello, Dumping a bit of information in case it might help. Using HASSIO on PI3
hassos: "7.0"
Washing machine is a Hoover HNFLS S684TAH-37 Dump of http://192.168.0.225/http-read.json?encrypted=1 Using HEX to ASCII Version 0.7.0 fails to add the machine. Adding integration I set the IP and I get the message: "Failed to detect encryption, check logs" /config/home-assistant.log During handling of the above exception, another exception occurred: Traceback (most recent call last): Thank you all for the effort you are putting into this, I really appreciate it. @ofalvai |
Beta Was this translation helpful? Give feedback.
-
|
I'm honestly out of ideas why the server disconnects during the second request. I'm just blindly guessing here, but what if a little delay before the second API call solves this? I created a PR with this workaround, can you please try running that branch and let me know if this works? |
Beta Was this translation helpful? Give feedback.
-
|
Hey @GozerBCN, please see my latest comment in this thread. Seems like your device also can't handle multiple requests in a short time window. I recommend applying the change manually from my PR branch, plus adding another sleep command that I detailed in the last message in this thread. Let me know if this helps! |
Beta Was this translation helpful? Give feedback.
-
|
Hello @ofalvai here's what I tried. So far homeassistant.log shows: 2022-01-04 15:39:34 INFO (MainThread) [custom_components.candy.client] Trying to get a response without encryption (encrypted=0)... The above exception was the direct cause of the following exception: Traceback (most recent call last): So I decided to try with a 15s delay. (@SwannyBFC if you try this remember to reboot HA each time you change the code, just in case for the changes to be applied). 2022-01-04 16:04:09 INFO (MainThread) [custom_components.candy.client] Trying to get a response without encryption (encrypted=0)... The above exception was the direct cause of the following exception: Traceback (most recent call last): During handling of the above exception, another exception occurred: Traceback (most recent call last): During handling of the above exception, another exception occurred: Traceback (most recent call last): |
Beta Was this translation helpful? Give feedback.
-
|
This is interesting, the root cause seems to be an entirely different error:
I usually get this error when my washing machine is completely turned off and can't even ping the device. Are you sure the machine is turned on, not in some sleep mode, and the IP is correct? I would ping the IP just to make sure. |
Beta Was this translation helpful? Give feedback.
-
|
Hello, async def detect_encryption(session: aiohttp.ClientSession, device_ip: str) -> (Encryption, Optional[str]): Then integration is added without further errors but when I check the logs I can see: 2022-01-06 11:19:12 DEBUG (MainThread) [custom_components.candy] Finished fetching candy data in 1.583 seconds (success: False) So back to square 1? |
Beta Was this translation helpful? Give feedback.
-
|
@GozerBCN back to square 1 because you removed every sleep from the code, so now the next request (the first update call of the newly added integration) is fired immediately :) |
Beta Was this translation helpful? Give feedback.
-
|
@ofalvai How do I pull this branch via HACS ? |
Beta Was this translation helpful? Give feedback.
-
|
I don't think you can do that unfortunately, you'll need to download the code as a ZIP or use Git to checkout the branch in the custom components folder.
… On 2021. Dec 18., at 14:28, SwannyBFC ***@***.***> wrote:
@ofalvai <https://github.com/ofalvai> How do I pull this branch via HACS ?
—
Reply to this email directly, view it on GitHub <#53 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAM52CSKHNBQXPJPSFF7OKDURSEBXANCNFSM5J4BWJ2A>.
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.
|
Beta Was this translation helpful? Give feedback.
-
|
Ok heres what I did, deleted the current installed integration, rebooted. Installed 0.70 and copied the code from your branch and pasted it over thee init.py within \config\custom_components\candy\client. Rebooted and still getting issues 2021-12-19 09:15:21 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration candy which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant |
Beta Was this translation helpful? Give feedback.
-
|
If its any help, when the washer is running if i send the curl command over and over it always returns the output I originally posted |
Beta Was this translation helpful? Give feedback.
-
This line tells me that the setup flow was successful, however...
You get the same disconnection error when the integration tries to update its state for the first time. We are back to the original problem that your device can't handle two API requests in a short window of time. Please try adding another |
Beta Was this translation helpful? Give feedback.
-
|
Hi @ofalvai , I'm experiencing the same issue ( see #167 ) I've added Any clue on this ? |
Beta Was this translation helpful? Give feedback.
-
|
Ive added the extra sleep and get the same issue. Have I put them both in the correct place ? |
Beta Was this translation helpful? Give feedback.
-
|
@ofalvai just tested latest update and the integration now works for my washer, thanks |
Beta Was this translation helpful? Give feedback.
-
|
I think your issue was fixed by #68. I forgot to create a release after merging that PR, sorry about not releasing it earlier. |
Beta Was this translation helpful? Give feedback.
-
|
sorry folks, I am having that same issue - can't detect encryption. Is the modified code now in the main branch or do I need to copy code from another branch? If so which branch is it? Add Sleep seems to be 13 months old. |
Beta Was this translation helpful? Give feedback.
-
|
Hello everyone 👋 I released version 0.8.0 that contains API rate-limiting, which is the long-term fix for those who had to add the The rate limiter is set up to only allow 1 request every 3 seconds, I think this will be a good start. If you want to tweak the value, you can find it here: |
Beta Was this translation helpful? Give feedback.
-
In the integration, is it possible you could add devices that only require you convert the json to ASCII.
i.e. for my washer if I do the following - curl -s http://[APPLIANCE-IP]/http-read.json?encrypted=1 | xxd -r -p
it returns the following
{
“statusLavatrice”:{
“WiFiStatus”:“1”,
“Err”:“255”,
“MachMd”:“1”,
“Pr”:“15”,
“PrPh”:“0”,
“SLevel”:“2”,
“Temp”:“90”,
“SpinSp”:“4”,
“Opt1”:“0”,
“Opt2”:“0”,
“Opt3”:“0”,
“Opt4”:“0”,
“Opt5”:“0”,
“Opt6”:“0”,
“Opt7”:“0”,
“Opt8”:“0”,
“Steam”:“0”,
“DryT”:“0”,
“DelVal”:“255”,
“RemTime”:“136”,
“RecipeId”:“0”,
“CheckUpState”:“0”
}
Running simplyfi tool returns the following output
./simplyfi [APPLIANCE-IP] getkey
Candy Simply-Fi tool by Melvin Groenendaal
error: find_xor_key_list, could not find key
Beta Was this translation helpful? Give feedback.
All reactions