Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Negotiage client certificate#134

Merged
nayato merged 1 commit intoAzure:devfrom
mtuchkov:tlsserver
Jun 18, 2016
Merged

Negotiage client certificate#134
nayato merged 1 commit intoAzure:devfrom
mtuchkov:tlsserver

Conversation

@mtuchkov
Copy link
Contributor

Motivation:
SslStream's paramter required client certificate is misleading. In fact in suggests to client to provide the certificate and client may or may not provide it.
If client has not provided the client certificate - it assumes that client is not authenticated from TLS prospective and continues the handshake

Modification:
Expose this setting as a parameter, so that users could set it if needed.

Results:
Allows users to do a client certificate authentication on server.

Motivation:
SslStream's paramter required client certificate is misleading. In fact in suggests to client to provide the certificate and client may or may not provide it.
If client has not provided the client certificate - it assumes that client is not authenticated from TLS prospective and continues the handshake

Modification:
Expose this setting as a parameter, so that users could set it if needed.

Results:
Allows users to do a client certificate authentication on server.
@nayato
Copy link
Member

nayato commented Jun 18, 2016

LGTM

@nayato nayato merged commit 8a42c72 into Azure:dev Jun 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants