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

Client ECDSA support #360

Merged
merged 2 commits into from
Oct 3, 2019
Merged

Client ECDSA support #360

merged 2 commits into from
Oct 3, 2019

Conversation

tomato42
Copy link
Member

@tomato42 tomato42 commented Sep 25, 2019

Add support for client certificates with ECDSA keys

depends on: #359
(second part of the full ECDSA support, #52, reworking of #196 to make review easier)


This change is Reviewable

@tomato42 tomato42 added the enhancement new feature to be implemented label Sep 25, 2019
@lgtm-com
Copy link

lgtm-com bot commented Sep 25, 2019

This pull request fixes 1 alert when merging d512283 into 401712c - view on LGTM.com

fixed alerts:

  • 1 for Membership test with a non-container

@lgtm-com
Copy link

lgtm-com bot commented Sep 30, 2019

This pull request fixes 1 alert when merging 24be2c9 into 401712c - view on LGTM.com

fixed alerts:

  • 1 for Membership test with a non-container

@tomato42 tomato42 force-pushed the client-ecdsa-support branch 3 times, most recently from 42147dd to 0d7f252 Compare September 30, 2019 20:43
@tomato42 tomato42 requested a review from nmav October 1, 2019 12:20
@nmav nmav self-assigned this Oct 1, 2019
tests/tlstest.py Show resolved Hide resolved
nmav
nmav previously approved these changes Oct 1, 2019
Copy link
Collaborator

@nmav nmav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added some questions and comments however it looks good to me as it is.

tests/tlstest.py Show resolved Hide resolved
tlslite/tlsconnection.py Show resolved Hide resolved
tlslite/tlsconnection.py Outdated Show resolved Hide resolved
@tomato42 tomato42 force-pushed the client-ecdsa-support branch 2 times, most recently from 1668165 to b1c9017 Compare October 2, 2019 18:26
@tomato42
Copy link
Member Author

tomato42 commented Oct 2, 2019

@nmav updated

@tomato42 tomato42 merged commit bbbdde7 into master Oct 3, 2019
@tomato42 tomato42 deleted the client-ecdsa-support branch October 3, 2019 08:57
@tomato42
Copy link
Member Author

tomato42 commented Oct 3, 2019

Thanks for review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement new feature to be implemented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants