Spencer/bugfix 171 docker pg credentials#172

Closed

spencertipping wants to merge 21 commits into

morphik-org:mainfrom

spencertipping:spencer/bugfix-174-docker-pg-credentials

Commits on May 26, 2025

Pulling scripts into separate files, parsing PG credentials from POSTGRES_URI
spencertipping
committed
Preserving stderr to help debug connection issues
spencertipping
committed
update npm package version

Adityav369
authored and
spencertipping
committed
Trying awk parse to fix edge cases
spencertipping
committed
Split postgres parser into its own script so I can test jazzberry's alleged failure cases
spencertipping
committed
More edge case fixes
spencertipping
committed
Postgres need not use a password, especially when dealing with obscured or corrupt URIs
spencertipping
committed
Allowing postgres URI parse failures to accommodate the value "***localhost:5432/morphik", which happens during docker build
spencertipping
committed
Modified script to work with github awk version, which is non-GNU and has no strtonum function
spencertipping
committed
You're killing me github
spencertipping
committed
OK, doing everything in python since we know the container has that
spencertipping
committed
Log PG values only on failure
spencertipping
committed
Spinning up DB container during test
spencertipping
committed
Using async PG driver
spencertipping
committed
Trying longer timeout
spencertipping
committed
Added redis container
spencertipping
committed
Fixed redis container name
spencertipping
committed
Fixed postgres credentials to not collide with primary bind addr/port
spencertipping
committed
Using test toml file instead of repo-default
spencertipping
committed
Create test file if it doesn't exist (avoids mounting an empty dir)
spencertipping
committed
Properly locating the test file, which must exist since we created it earlier
spencertipping
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Spencer/bugfix 171 docker pg credentials#172

Spencer/bugfix 171 docker pg credentials#172
spencertipping wants to merge 21 commits into
morphik-org:mainfrom
spencertipping:spencer/bugfix-174-docker-pg-credentials

Commits on May 26, 2025

Pulling scripts into separate files, parsing PG credentials from POSTGRES_URI

Preserving stderr to help debug connection issues

update npm package version

Trying awk parse to fix edge cases

Split postgres parser into its own script so I can test jazzberry's alleged failure cases

More edge case fixes

Postgres need not use a password, especially when dealing with obscured or corrupt URIs

Allowing postgres URI parse failures to accommodate the value "***localhost:5432/morphik", which happens during docker build

Modified script to work with github awk version, which is non-GNU and has no strtonum function

You're killing me github

OK, doing everything in python since we know the container has that

Log PG values only on failure

Spinning up DB container during test

Using async PG driver

Trying longer timeout

Added redis container

Fixed redis container name

Fixed postgres credentials to not collide with primary bind addr/port

Using test toml file instead of repo-default

Create test file if it doesn't exist (avoids mounting an empty dir)

Properly locating the test file, which must exist since we created it earlier