Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Better support for secure database connections #2945

Merged
merged 4 commits into from
Jul 5, 2024

Conversation

sandhose
Copy link
Member

@sandhose sandhose commented Jul 2, 2024

Fixes #2799

This adds new configuration parameters to set the CA, client certificate and key, as well as the SSL mode to use

This also adds support in syn2mas for using reading certificates for both connecting to the Synapse and the MAS databases

@sandhose sandhose added the A-Configuration Related on what is configurable and how it can be configured label Jul 2, 2024
@sandhose sandhose requested a review from reivilibre July 2, 2024 15:21
Copy link

cloudflare-workers-and-pages bot commented Jul 2, 2024

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7aa2e9b
Status: ✅  Deploy successful!
Preview URL: https://89d6ffbf.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-postgres-certs.matrix-authentication-service-docs.pages.dev

View logs

Copy link
Contributor

@reivilibre reivilibre left a comment

Choose a reason for hiding this comment

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

I admittedly have a slight preference for just leaving this to SQLx and having users configure it through the connection URI but on the other hand this looks fine

@sandhose sandhose merged commit ba58195 into main Jul 5, 2024
16 checks passed
@sandhose sandhose deleted the quenting/postgres-certs branch July 29, 2024 12:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Configuration Related on what is configurable and how it can be configured
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for secure database connections using TLS certs
2 participants