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

ECC private keys support for tls feature #1048

Merged

Conversation

joseluisq
Copy link
Contributor

This PR adds support for ECC private keys when enabling the tls feature.
It is particularly useful when generating private keys using some client tools like for example Lego ACME which defaults to EC256 keys.

Changelog:

  • ECC private keys support
  • Refactor TlsConfigError to reflect intended changes
  • tokio-rustls 0.24 and subsequent client_auth updates

It adds support for `ECC` private keys when enabling the `tls` feature.
This is particularly useful when generating private keys for example
using clients like `Lego ACME` which defaults to `EC256` keys

changelog:
- ecc private keys support
- refactor `TlsConfigError` to reflect intended changes
- tokio-rustls 0.24 and subsequent `client_auth` updates
src/tls.rs Show resolved Hide resolved
@fdeantoni fdeantoni mentioned this pull request Aug 2, 2023
@mohe2015
Copy link

mohe2015 commented Oct 8, 2023

Is there anything for the author of this PR to do (maybe resolve conflicts) or would it be possible to maybe get this reviewed and merged please?

@mohe2015
Copy link

mohe2015 commented Oct 8, 2023

Oh no please don't say this is a breaking change because of the dependency bump...

Copy link
Owner

@seanmonstar seanmonstar left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks!

@seanmonstar
Copy link
Owner

Looks like there's merge conflicts, if you resolve and ping me, I'll click merge :)

@joseluisq
Copy link
Contributor Author

Looks like there's merge conflicts, if you resolve and ping me, I'll click merge :)

Conflicts gone. Let me know if there is something else to do :).

@seanmonstar seanmonstar merged commit efe8548 into seanmonstar:master Oct 9, 2023
8 checks passed
@joseluisq joseluisq deleted the tls-ecc-private-keys-support branch October 9, 2023 01:51
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.

None yet

3 participants