Update test cert generation script for postgresql#1163
Update test cert generation script for postgresql#1163cyli merged 2 commits intonotaryproject:masterfrom
Conversation
Signed-off-by: Ashwini Oruganti <ashwini.oruganti@gmail.com>
|
This addresses @cyli's comment from #1160 (comment) |
|
|
||
| # Postgresql keys for testing server/client auth | ||
|
|
||
| # TODO: test if cfssl is installed, if not, cleanly exit with a message |
There was a problem hiding this comment.
I've used something like this before:
command -v cfssljson >/dev/null 2>&1 || {
echo >&2 "Installing cfssl tools"; go get -u github.com/cloudflare/cfssl/cmd/...;
}There was a problem hiding this comment.
Thank you kindly. Will update! :)
Signed-off-by: Ashwini Oruganti <ashwini.oruganti@gmail.com>
|
Thanks for working on this! LGTM with the caveat that we might want to create the dir |
|
Nope that works for me! |
|
jenkins, test this please |
riyazdf
left a comment
There was a problem hiding this comment.
code LGTM pending green on jenkins tests, and also we should open an issue to migrate the other cert generation bits in regnerateTestingCerts.sh to also use cfssl - it's a little odd to only use cfssl for certain certs and openssl for others
|
There were no previous changes to that file, and seem to be no conflicts, so merging anyway :) Thanks @ashfall! |
This can be used to regenerate testing certs for postgres db connections.
Depends on #1160