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

Ensure ssl is started before running connection and querying #37

Merged
merged 1 commit into from
Sep 1, 2024

Conversation

ghivert
Copy link
Contributor

@ghivert ghivert commented Aug 26, 2024

Hi!

Follow-up of a long bugfix on Discord that is finally fixed.

Unless I'm in error, that PR should fix it by ensuring ssl is running.

@ghivert ghivert force-pushed the fix/ensure-ssl-is-started branch 2 times, most recently from 1a16a42 to 94d4b0e Compare August 26, 2024 18:32
@lpil
Copy link
Owner

lpil commented Aug 26, 2024

Thanks! Why is this here rather than in pgo?

@ghivert
Copy link
Contributor Author

ghivert commented Aug 26, 2024

I've no real reason unless it was logical to me to push the fix here because it's the repo I'm mainly modifying/working with. I'll see if it can easily be pushed in pgo.

Copy link
Owner

@lpil lpil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thank you

@lpil lpil merged commit 1622c7e into lpil:main Sep 1, 2024
1 check passed
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