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 access to SSL peer certificate #965

Merged
merged 3 commits into from Feb 23, 2015

Conversation

Projects
None yet
3 participants
@rix0rrr
Contributor

rix0rrr commented Jan 25, 2015

Provide access to a client's certificate information, which makes client
cert based auth/cert SSO a lot more useful.

rix0rrr added some commits Jan 25, 2015

Add access to SSL peer certificate
Provide access to a client's certificate information, which makes client
cert based auth/cert SSO a lot more useful.
Use DictionaryList instead of associative array
This should not invoke the GC as much.
@s-ludwig

View changes

Show outdated Hide outdated source/vibe/stream/openssl.d Outdated
@s-ludwig

View changes

Show outdated Hide outdated source/vibe/stream/openssl.d Outdated
@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 21, 2015

Member

Looks good to merge, except that lines are indented using spaces instead of tabs. Depending on what editor you use, there might be an EditorConfig plugin, which is very useful for such formatting issues.

Member

s-ludwig commented Feb 21, 2015

Looks good to merge, except that lines are indented using spaces instead of tabs. Depending on what editor you use, there might be an EditorConfig plugin, which is very useful for such formatting issues.

@rix0rrr

This comment has been minimized.

Show comment
Hide comment
@rix0rrr

rix0rrr Feb 23, 2015

Contributor

Changed spaces to tabs

Contributor

rix0rrr commented Feb 23, 2015

Changed spaces to tabs

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 23, 2015

Member

Thanks! Merging.

Member

s-ludwig commented Feb 23, 2015

Thanks! Merging.

s-ludwig added a commit that referenced this pull request Feb 23, 2015

Merge pull request #965 from rix0rrr/ssl-peer-info
Add access to SSL peer certificate

@s-ludwig s-ludwig merged commit d8b7a04 into vibe-d:master Feb 23, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@SerialVelocity

This comment has been minimized.

Show comment
Hide comment
@SerialVelocity

SerialVelocity Jun 8, 2017

@s-ludwig @rix0rrr Is there a D client library which allows setting the client certificate? (WebSocket 2-way SSL is actually what I'm looking for)

SerialVelocity commented Jun 8, 2017

@s-ludwig @rix0rrr Is there a D client library which allows setting the client certificate? (WebSocket 2-way SSL is actually what I'm looking for)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment