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

Add TLS support #45

Open
davissp14 opened this issue Oct 15, 2021 · 3 comments
Open

Add TLS support #45

davissp14 opened this issue Oct 15, 2021 · 3 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@davissp14
Copy link
Contributor

davissp14 commented Oct 15, 2021

Neither the Fly proxy nor HAProxy understands pgsql, which means we are not able to handle TLS termination in the same way we do for other apps. We need to decide whether to use something like Stunnel to handle termination or work to inject certificates as secrets and just have Postgres handle it.

@davissp14 davissp14 added bug Something isn't working enhancement New feature or request labels Oct 15, 2021
@mrkurt
Copy link
Contributor

mrkurt commented Oct 15, 2021

If it's possible, I'd prefer to just show people how to setup a pgbouncer that does TLS termination and points at their postgres cluster. It makes a lot of sense to isolate public ports to a special pgbouncer vm!

We can also just not do this for quite some time.

@davissp14
Copy link
Contributor Author

Yeah, I think that could work. I think the big thing would be to ensure pgbouncer runs within the same regions as their Postgres app in order to accommodate reads.

@tv42
Copy link

tv42 commented May 26, 2022

Is this a duplicate of #4 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants