-
Notifications
You must be signed in to change notification settings - Fork 110
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
add new encryption method #165
Conversation
Hi, i've just tested this. I'm getting a few errors (using the old encryption):
|
Could you please check again? I have no version 1 unit to tests. |
Seems to work now. Few code questions. Let me know :) |
Hmmm. I cannot see any questions. |
You have to publish it to let others see. |
i think this should do the trick. can you now see it? |
I removed duplicated code. It should look better now. |
This also works for v1.08 (Vaillant Firmware) 👍 |
Works like a charm! Thanks :) |
Hi @JensVanhooydonck , i have also the same brand, but can't get it to work even with this new release.
|
I've added them to the app, and extracted the encryption keys from the database (i'm on a macbook m1 => Opened the Sql-lite database to get the keys)
|
Hi @JensVanhooydonck
|
@tomaszduda23 powieliliście ten sam błąd, który był w oryginalnym kodzie. |
I can confirm it is working flawlessly with:
@RobHofmann ; maybe it is possible to add this to the confirmed working units? (they are btw exact the same units as the saunier duval & vaillaint vivair series) Configured it via the ewpe smart app. (also working in the gree app) Only configuration needed in ha:
|
In my opinion, there is a bug in the code from the very beginning that should be removed before any changes are made in the future. Retrieving a device key, unless it is provided in the configuration, should not occur in the class constructor. If for some reason a connection to the device is not established during the creation of the object, HA will throw an error, the device will not be added to the registry and it becomes unusable until the HA is restarted, but even then there is no guarantee that the key will be downloaded and we have an endless loop. in my opinion, retrieving device key should be placed in async_added_to_hass as in #163 |
Add support for new version of encryption. It was mentioned tomikaa87/gree-remote#52