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 AWS Parameter Store config setting for "require database ssl" #4309

Closed
codyebberson opened this issue Apr 3, 2024 · 0 comments · Fixed by #4319
Closed

Add AWS Parameter Store config setting for "require database ssl" #4309

codyebberson opened this issue Apr 3, 2024 · 0 comments · Fixed by #4319
Labels
self-host Features and fixes related to self hosting

Comments

@codyebberson
Copy link
Member

Our database connection implementation supports Postgres SSL.

Our config setting implementation supports Postgres SSL.

Unfortunately, the AWS Parameter Store implementation of config setting loading does not work, because AWS Secrets Manager does not support arbitrary JSON in the secret.

To enable customers to "require SSL", we need to either modify the AWS Secrets Manager loading, or add an additional setting for "require SSL".

As pre work, let's first research what AWS recommends here.

@reshmakh reshmakh added the self-host Features and fixes related to self hosting label Apr 4, 2024
medplumbot added a commit that referenced this issue Apr 4, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
codyebberson pushed a commit that referenced this issue Apr 4, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
github-merge-queue bot pushed a commit that referenced this issue Apr 5, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
@reshmakh reshmakh added this to the April 30th, 2024 milestone Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
self-host Features and fixes related to self hosting
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants