-
Notifications
You must be signed in to change notification settings - Fork 41
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
Integration stopped working with HA 2022.7.0 #130
Comments
|
I do get the same error message, since I have installed 2022.7. |
I also see this error in my logs
|
Looks like the error is not in the freeathome part but in a required package. I wonder if updating the "requirements" to a later version (slixmpp==1.8.2) will fix the issue or raise a couple new ones. |
I have created a PR as a first try to fix this issue. Upgraded slixmpp and made a small code change. Please go to #131 to verify the change. |
Sadly new errors:
|
I also had this error. If you apply the PullRequest #131 from @jeroen84 in the code and run the integration in the Docker image (latest), then everything is found correctly and free@home integration is working fine. As a workaround, I can only recommend the following (for "Home Assistant Operation System"-Users):
|
Another error:
|
Has the complete PR #131 also been applied? |
Same here, its working again (Had to reboot the Busch Jaeger / Free@Home). How do I git clone the fix because when I do a git clone I get the old files without the fix. :( Anyway, thank you for the fix and help! |
If you just want to get it working quickly again you can do a rollback to HA version 2022.6.7 with
|
Or if using docker, set the tag to 2022.6
Apologies for the introduced errors by autocorrection on my mobile device.
…On Sat, 9 Jul 2022, 11:58 Stefan Stadlberger ***@***.***> wrote:
If you just want to get it working quickly again you can do a rollback to
HA version 2022.6.7 with
ha core update --version 2022.6.7
—
Reply to this email directly, view it on GitHub
<#130 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHT3H75P3DIGGLM7SNKUITVTFEM3ANCNFSM5243YSTQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
As I'm lazy to repeat those steps each time after an update, I've create a simple script, that can be triggered via automation. For all other lazy people, here are my scripts and settings respectively: Shell-Script (/config/custom_components/freeathome/scripts/fah_requirements.sh):
EDIT Entry in global configuration.yaml:
Entry in global automations.yaml:
Short explanation: When Home Assistant is started/restarted, the automation 'fah_system_start' is triggered, this executes the shell command 'fah_requirements' which triggers our script. The script itself first checks whether the apk is already installed and only installs it if this is not the case. |
I can also confirm applying the PR and apk adding libsodium brings FAH integration back to life and functionality |
Thanks for this. As I tend to be a lazy person as well I also added that. Cheers mate. |
Hi Guys, I don't know what i'm doing wrong but i can't get it working. |
Looks like this change breaks the integration: home-assistant/docker@e926373 This removed libsodium from the build of the Home Assistant base image. Home Assistant OS and Docker images are affected. Not sure how to make a structural solution for this, other than adding libsodium as explained above. |
Yes but when I try to do it like above, then I get this response back from the logbook: Logger: homeassistant.components.shell_command Error running command: |
After a brief search a potential solution is switching to https://github.com/pyca/pynacl/ as a replacement for libnacl. This package is a standard dependency in Home Assistant https://github.com/home-assistant/core/blob/3429a75cc5d9be8560dfef5c2619d21273e2bffe/requirements_all.txt#L27 |
I also don't see an easy way to solve this. As @jeroen84 mentioned, the library was removed from the docker image, and since no other official component relies on it, chances are low that it will be re-added by the core team. My workaround is to build a custom Docker image on my machine, however this may not be possible for everyone: FROM homeassistant/home-assistant:2022.7.2
RUN apk add libsodium The cleanest solution would be to replace |
Seems that you don't have the correct permissions on "fah_requirements.sh"...
|
In fact, they have removed a number of libs. |
I give now all permits to fah_requirements.sh
I tried to add the libsodium manually: But when I restart home assistant I get again faillures;
|
I had an update for freeathome yesterday, everything still worked afterwards. Now 2022-07.3 came up and it's broken again... I guess HA guys don't make too many friends this time. I read that Homematic seems to be broken as well... |
Update: Script stopped working for me as well now. manually apk adding in shell and restarting HA brought integration back to life again |
Was broken for me after update to 2022-07.3. I did a restart of HA only and fah is working again. My script tries to add the apk after every HA restart, but this trigger is handled after integration tries to start, that's why I have to do a 2nd restart after each update! |
Check your path and try running the command via terminal or ssh. |
I tried to do that, but when I login via SHH and following the instructions I get:
|
Have you disabled safe-mode of your ssh-plugin? |
It looks like you use the "ssh" addon from the official repository. Remove it and use the "ssh" addon from the community store. This has docker installed. |
Thanks, This was the problem. Now everything is working again ;) |
Also with the update to 2022.6.7 freeathome does not work... |
I also run into this issue, I have installed a fresh 2022.7.6 and are not able to get free@home running :( |
Also with the update to 2022.7.7 freeathome does not work... |
Hi, as I see a lot of "F@H isn't working" messages in this issue I wanted to share my experience with it. Perhaps it helps ... Main thanks goes to @bezibaerchen and the following comment #130 (comment), which helped me to get F@H working in 2022.7 (from 2022.7.1 to 20.22.7.7). Pre-Info:
Root cause:
Pre-Step (for troubleshooting and so on):
Test-Step:
So if you come to this point your F@H integration should work as expected. You can restart HA and still F@H should work. The drawback of this is, that, because of the nature of a docker-image, each update of HA will override this modification and you have to do the steps above again. Making it permanent:
(Replace the directory and filename with your setup you collected in step 8)
Final consideration:
I hope this write-down helps somebody to get the F@H-integration working again. For sure, this is not a permanent fix (I don't have the technical knowledge to propose a permanent fix), but it is working [at least for me]. I successfully upgraded HA all the way from 2022.7.1 till the current 2022.7.7 with just this one additional restart required. Regards |
Thanks for the summary, I started from scratch and followed your instructions, and now it works for me too. |
The guide from @derjoerg also works for me. Thank you so much. |
|
As a permanent solution I think we have two options:
For 1, this is complex as outlined by @Tho85 . For 2, this is a similar approach as how pynacl is build - it is dependent on libsodium as well - and the reason why this package works with HA. Any other thoughts? |
2 sounds for me like the way, which would result in less effort. I would be happy to test :) |
The guide worked for me too. Thanks a lot! |
Also with version 2022.8 free@home does not work. I'm seriously concerned now. |
Workaround still working though, just confirmed here |
I thought that was a bit complicated.. and wasn't sure it would work for me.. |
Workaround with 2022.8.1 still working |
Could you please help with the error I get after fully following your guide? Error running command: My full path is mentioned above, though I tried only pointing to |
All, I've look at the two options: #130 (comment) Option 2 is very complex and not sustainable as it requires - next to time to develop and test - significant future maintenance. Option 1 should be considered to make this integration future proof. I've made a first try, but this is not working yet. It cannot do a successful key exchange, perhaps because pynacl lacks an important libsodium function crypto_onetimeauth. If anybody has time to investigate and would like to contribute: https://github.com/jeroen84/freeathome/tree/move-pynacl A third option we could consider is moving to the REST API. I notice that this option was considered ~2 years ago in #80 There were limitations of the API back then. Perhaps there are developments in due course that makes this option feasible? |
I have just pushed a PR for replacing libnacl with pynacl. This should fix the issue structurally. Please test on your setup: #134 |
It works fine here! Well done! |
Super that it is fixed, when can we expect a new release? Just ask; no pressure :) |
Real nice guys! this works like a charm! |
We do not currently use GitHub releases for this custom component. Do you use HACS? I'm not an HACS user myself, but you should be able to update the component there somewhere. |
Thx, I will try that. I am very new to HomeAssistant and HACS. I don't know anything about the internal release process. |
I updated using HACS just fine. It updates to the latest commit. |
Works for me too! |
I read about changes in Python.
Status as of now: "Failed to set up: Import error
The text was updated successfully, but these errors were encountered: