Skip to content
This repository has been archived by the owner on Jan 5, 2025. It is now read-only.

Certificate warnings #23

Closed
tofuSCHNITZEL opened this issue Oct 26, 2023 · 2 comments
Closed

Certificate warnings #23

tofuSCHNITZEL opened this issue Oct 26, 2023 · 2 comments

Comments

@tofuSCHNITZEL
Copy link

Hi,
thanks for creating this integration!
I started getting these errors in the home-assistant log:
InsecureRequestWarning: Unverified HTTPS request is being made to host 'nucleo.ksecosys.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings

I tried to bump the pybotvac to version 0.0.24 but it contains an old certificate and so the communication stops entirely...
I created a pull request with the new cert:
stianaske/pybotvac#82
but I wonder if there would be a better way? to just enable the urllib3 cert validation like suggested in the link above?

@tofuSCHNITZEL
Copy link
Author

I would suggest this:
stianaske/pybotvac#83
is there a reason you added the cert? instead of just setting it to True so requests validates the cert (chain) itself?

@trunneml
Copy link
Owner

No. This was at that point in time the easiest way. If true is working, that's the way to go. I close this ticket, because it is related to pybotvac.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants