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

Allow ssl connections for newer Senec devices #15

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mschmiedel
Copy link

Hi,

I am proposing this change to be able to use with newer Senec devices. Yes, downloading the ssl certificate and importing it would be better, but for most home users this should be enough. :)

Running something like:

poetry run senec --host https://192.168.XXX.XXX

now runs properly, my goal is to set this url in the Home Assistant Integration, this would be the smalles necessary change to get ssl working...

@mschmiedel
Copy link
Author

-> I am not an experienced python developer... But there seem to be some issues with building this project with newer python versions. I could fix it by forcing python <= 3.8 and limiting cython to <3.

@yonarw
Copy link
Contributor

yonarw commented Sep 3, 2023

Would be nice to get this in. My system updated this week and HA can no longer access/query status. Let me know if I can help with testing.

@mschmiedel
Copy link
Author

Would be nice to get this in. My system updated this week and HA can no longer access/query status. Let me know if I can help with testing.

I have it working with Home Assistant, with 2 minor fixes on the Integration to support the non verified ssl it works, alternatively it would be possible to download the Senec certificates and install them, however it is questionable if it makes sense on a home network to do all the certificate management...

@mschmiedel
Copy link
Author

Would be nice to get this in. My system updated this week and HA can no longer access/query status. Let me know if I can help with testing.

BTW, today I noticed that the STATISTIC object is always empty, no idea what happened, it was working fine, will be monitoring this in the following days...

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

Successfully merging this pull request may close these issues.

2 participants