Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Updated openssl example command #1763

Merged
merged 2 commits into from Apr 23, 2020
Merged

Updated openssl example command #1763

merged 2 commits into from Apr 23, 2020

Conversation

bvanderveen
Copy link
Contributor

This updates the documentation about how to connect using SSL and the key format requirements.

  • reflect the filenames the user likely needs
  • adds -nocrypt flag to prevent driver from attempting to ask the user for a password

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@bvanderveen
docs: update openssl key conversion example command
dc59346 
update openssl example command to reflect the filenames the user likely needs, and to be
consistent with the naming requirements described in the previous sentence. add -nocrypt flag to
the example command. this prevents driver from attempting to ask the user for a password for the
resulting key file (even when the input key did not have a password).
@bvanderveen
Copy link
Contributor Author

This is in reference to: #1364 (comment)

@bvanderveen bvanderveen mentioned this pull request Apr 23, 2020
Closed
2 tasks
@davecramer
Copy link
Member

You do not need the -nocrypt if you provide an sslpwd. I'll push yours if you can update that bit.

@bvanderveen
Copy link
Contributor Author

I propose a note just below the command like:

If your key has a password, provide it using the sslpassword connection parameter described below. Otherwise, you can add the flag -nocrypt to the above command to prevent the driver from requesting a password.

If that sounds good I have a commit standing by.

@davecramer
Copy link
Member

Seems reasonable enough.

@bvanderveen
docs: add note explaining usage of sslpassword with respect to sslkey
17d6cf7 
Explain that key passwords are provided using sslpassword. Move the mention of the `-nocrypt`
flag to the note for use with password-less key files.
@bvanderveen
Copy link
Contributor Author

Done, thanks for your effort and support!

@davecramer davecramer merged commit e19d458 into pgjdbc:master Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bvanderveen @davecramer