Skip to content

Conversation

ruslandoga
Copy link
Contributor

@ruslandoga ruslandoga commented Apr 6, 2025

closes #4955

Some self-hosters are used to psql-like query strings and assume they would work in DATABASE_URL but they don't.
This PR adds support for sslmode and sslrootcert query strings in DATABASE_URL

@ruslandoga ruslandoga changed the title sslmode Support sslmode in DATABASE_URL Apr 6, 2025
@ruslandoga ruslandoga added the self-hosting Anything self-hosted label Apr 6, 2025
@ruslandoga ruslandoga changed the title Support sslmode in DATABASE_URL Support sslmode in DATABASE_URL in CE Apr 6, 2025
@ruslandoga ruslandoga requested review from ukutaht and cnkk April 7, 2025 06:39
@thomastvedt
Copy link

This PR is great news, any chance of getting this trough review soon @ukutaht or @cnkk ? :)
Our plausible instance just broke in production since we rely on root ca cert pinning, and Microsoft decided to change the root ca cert for postgresql flexible server again...

🙏

@cnkk cnkk added the preview label May 19, 2025
Copy link

Preview environment👷🏼‍♀️🏗️
PR-5280

@cnkk cnkk enabled auto-merge May 20, 2025 13:20
@cnkk cnkk added this pull request to the merge queue May 20, 2025
Merged via the queue into master with commit 9744961 May 20, 2025
8 checks passed
@cnkk cnkk deleted the sslmode branch May 20, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview self-hosting Anything self-hosted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Postgres connection rejected with sslMode=full in DATABASE_URL, and empty DATABASE_CACERTFILE
3 participants