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.

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

Add Postgres support to s_client -starttls #683

Closed
wants to merge 1 commit into from

Conversation

vvidic
Copy link
Contributor

@vvidic vvidic commented Feb 15, 2016

The patch sends a SSLRequest packet and checks for a S in response.
Useful for checking certificate validity of a PostgreSQL server.

@richsalz
Copy link
Contributor

+1

@vvidic vvidic changed the title Add Postgres support to -starttls Add Postgres support to s_client -starttls Feb 16, 2016
@richsalz richsalz added the issue: feature request The issue was opened to request a feature label Mar 19, 2016
@mattcaswell mattcaswell added this to the Post 1.1.0 milestone May 16, 2016
@t-j-h t-j-h added the approval: done This pull request has the required number of approvals label Oct 5, 2016
@greuff
Copy link

greuff commented Oct 17, 2016

+1

@mattcaswell mattcaswell added the hold: cla required The contributor needs to submit a license agreement label Oct 17, 2016
@mattcaswell
Copy link
Member

@vvidic We don't currently have a CLA on file for you so I am unable to merge this at the moment. Please would you be able to supply one? Details here:

https://www.openssl.org/policies/cla.html

Please make sure the email you use in the CLA matches the one used to make your commits.

Thanks!

@vvidic
Copy link
Contributor Author

vvidic commented Oct 17, 2016

On Mon, Oct 17, 2016 at 05:18:36AM -0700, mattcaswell wrote:

@vvidic We don't currently have a CLA on file for you so I am unable to merge
this at the moment. Please would you be able to supply one? Details here:

https://www.openssl.org/policies/cla.html

Please make sure the email you use in the CLA matches the one used to make your commits.

No problem, sent the PDF CLA to legal@opensslfoundation.org.

Valentin

@mattcaswell
Copy link
Member

CLA received. Thank you!

@mattcaswell mattcaswell added CLA OK and removed hold: cla required The contributor needs to submit a license agreement labels Oct 17, 2016
@mattcaswell
Copy link
Member

Pushed.

PeterMosmans added a commit to PeterMosmans/openssl that referenced this pull request Dec 9, 2016
Minor adaptation from openssl#683 - original author Valentin Vidic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals issue: feature request The issue was opened to request a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants