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

feat(http): expose tls mod and add custom error to allow using other tls implementation #1199

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joelwurtz
Copy link
Contributor

I am currently trying to implement a way to have automatic certificate with acme and xitca and this need to have a specific tls acceptor (to allow lazy config acceptor like it's done in tokio rustls : cf https://tg-rs.github.io/carapax/tokio_rustls/struct.LazyConfigAcceptor.html )

Goal is to make a tls connection waiting when a certificate is being created (so there is no insecure / dropped connection on start)

There is already the with_tls method on the HttpServiceBuilder but this need to expose a TlsError which is not public, this PR fix that by making it public and allow any errors in it

@joelwurtz joelwurtz force-pushed the feat/allow-custom-tls branch from 91863a7 to 2a06671 Compare February 5, 2025 14:47
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.

1 participant