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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sasadangelo
Copy link
Contributor

Hi Team,

I opened the following issue:
#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:
#629
#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
Copy link
Contributor Author

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
Copy link
Contributor Author

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

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.

1 participant