Conversation
f2a4219 to
b34eed4
Compare
|
Linked to: |
sgotti
left a comment
There was a problem hiding this comment.
@jwnl88 Thanks for the PR.
Can you also update the tests (since this impacts also pg_hba logic) to also test that everything works using this new auth method? (I'll just add another flavor of TestFailover in tests/integration/ha_test.go that uses the new auth method).
Also the documentation should be updated.
@sgotti Yes, I would like to update the tests + documentation, so this PR can be merged. However, I did run into some errors while trying to run the tests (with Does the test script currently fail on the Then I fixed this with And ran the test script again: |
Fixes issue #776 as far as I can tell. These changes allow us to use SCRAM-SHA-256 authentication on the clusters we're running.