search

CDCgov / phdi

https://cdcgov.github.io/dibbs-site/
Creative Commons Zero v1.0 Universal
35 stars 14 forks source link

Enable Pool Client SSL Configuration for tefca #2641

Closed robertandremitchell closed 2 months ago

robertandremitchell commented 2 months ago

PULL REQUEST

Summary

Does 2 things:

  1. Enables SSL to remove it as a potential confound for AWS connection per @alismx recommendation
  2. Restores tefca.env and amends npm run dev to work correctly for local development

Related Issue

Fixes SSL issue that appears to be impact

Acceptance Criteria

Please copy the acceptance criteria from your ticket and paste it here for your reviewer(s)

Additional Information

I would test both docker compose up and npm run dev to confirm both are still able to connect.

Checklist

DanPaseltiner commented 2 months ago

@robertandremitchell why have we checked in an entire Postgres DB? I could very well be missing something here.

robertandremitchell commented 2 months ago

@robertandremitchell why have we checked in an entire Postgres DB? I could very well be missing something here.

I've been having trouble getting the playwright tests to pass, so was just trying to chase down that error.

I went from:

[WebServer] db-1            | postgres: could not access the server configuration file "/var/lib/postgresql/data/postgresql.conf": No such file or directory

to:

[WebServer] db-1            | 2024-10-01 14:50:01.851 UTC [11] FATAL:  data directory "/var/lib/postgresql/data" has wrong ownership
db-1            | 2024-10-01 14:50:01.851 UTC [11] HINT:  The server must be started by the user that owns the data directory.

with this change, so hoping that once I figure that out, I can then remove pgdata/ or limit it to just pgdata.conf.