zalando / spilo

Highly available elephant herd: HA PostgreSQL cluster using Docker
Apache License 2.0
1.55k stars 382 forks source link

Fix ssl.SSLError: [SSL: HTTP_REQUEST] http request (_ssl.c:997) in Sp… #909

Open sasadangelo opened 1 year ago

sasadangelo commented 1 year ago

Hi Team,

I opened the following issue: https://github.com/zalando/spilo/issues/889

that explain a problem I found on Spilo 3.0-p1 code. In September 2021 I submitted two PRs to add SSL support to Patroni: https://github.com/zalando/spilo/pull/629 https://github.com/zalando/spilo/pull/625

however, I found out that the file postgres-appliance/scripts/patroni_wait.sh doesn't manage SSL variable so an error message appear at startup if patroni is configured in SSL (see my issue above). I had a discussion with Alexander Kukushkin on Slack and we agreed a possible solution. This solution, at the moment, is already running in my company dev, staging and production systems with an external patch I applied to Spilo 3.0-p1 during our build.

However, I would like to have this fix in the official code.

sasadangelo commented 1 year ago

Hi Team. Any chance to have someone looking at this PR? It’s OK for me even to know a possible date when it could be included.

sasadangelo commented 1 year ago

Hi Team. Any chance to have someone looking at this PR?