Skip to content

Commit

Permalink
add warning for docker build TLS certificate in readme
Browse files Browse the repository at this point in the history
  • Loading branch information
arska committed Dec 16, 2023
1 parent 54a71d3 commit 530c5fb
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,16 @@

Python API for ControlMySpa.com cloud-controlled of Balboa spa control systems for hot tubs.

* https://www.balboawatergroup.com/ControlMySpa
* https://controlmyspa.com
- https://www.balboawatergroup.com/ControlMySpa
- https://controlmyspa.com

## 2023-12-13: iot.controlmyspa.com missing intermediate certificate

Since approximately June 2023 iot.controlmyspa.com has a new TLS certificate. This certificate is signed by digicert, but the intermediate certificate chain is not served by iot.controlmyspa.com and is also missing in the python certifi trust store. Instead of disabling the TLS certificate validation, we download the intermediate certificate from digicert over a successfully verified TLS connection and add it to the local trust store on first run. This does, however, not work for read-only runtimes like Docker containers. See https://github.com/arska/controlmyspa-porssari/blob/main/Dockerfile and https://github.com/arska/controlmyspa-porssari/blob/main/get_certificate.py for an example how to download the certificate at Docker image build time instead.

## Usage

see example.py for runnable example
see example.py for a runnable example

```python
from controlmyspa import ControlMySpa
Expand Down

0 comments on commit 530c5fb

Please sign in to comment.