Skip to content

Add an auth-config for local connections in testing #195

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

Merged
merged 1 commit into from
Nov 21, 2019

Conversation

bzikarsky
Copy link

When the test suite is run with the testcontainers backend, the
integration tests were failing with newer versions of the postgres
docker image. The failures were caused by the docker-container not
properly starting which itself was caused by a missing entry in the
pg_hba.conf for local connections.

This commit adds the missing configuration for local connections.

When the testsuite is run with the testcontainers backend, the
integration tests were failing with newer versions of the postgres
docker image. The failures were caused by the docker-container not
properly starting which itself was caused by a missing entry in the
`pg_hba.conf` for local connections.
@pivotal-issuemaster
Copy link

@bzikarsky Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@mp911de
Copy link
Collaborator

mp911de commented Nov 21, 2019

No need to sign the CLA. @pivotal-issuemaster this is an obvious fix

@pivotal-issuemaster
Copy link

@bzikarsky This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.

@mp911de mp911de added this to the 0.8.0.RELEASE milestone Nov 21, 2019
@mp911de mp911de added the type: bug A general bug label Nov 21, 2019
@mp911de mp911de merged commit 257f8c8 into pgjdbc:master Nov 21, 2019
@mp911de
Copy link
Collaborator

mp911de commented Nov 21, 2019

Thanks a lot, that's merged now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants