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

[Breaking changes] Update rustls from 0.21 to 0.23 #125

Closed
5 of 6 tasks
junkurihara opened this issue Dec 8, 2023 · 6 comments
Closed
5 of 6 tasks

[Breaking changes] Update rustls from 0.21 to 0.23 #125

junkurihara opened this issue Dec 8, 2023 · 6 comments
Labels
dependencies Pull requests that update a dependency file help wanted Extra attention is needed

Comments

@junkurihara
Copy link
Owner

junkurihara commented Dec 8, 2023

rpxy heavily relies on rustls and its dependents. to upgrade rpxy, we need to wait for updates of the following crates based on rustls.

- [x] hyper-rustls (waiting for rustls/hyper-rustls#234 with hyper-1.0 support, alternative: hyper-tls could be workaround)

@junkurihara junkurihara added help wanted Extra attention is needed dependencies Pull requests that update a dependency file labels Dec 8, 2023
@junkurihara junkurihara pinned this issue Dec 8, 2023
This was referenced Dec 8, 2023
@junkurihara
Copy link
Owner Author

hyper-rustls v0.26.0 is available independently from other updates.

@junkurihara
Copy link
Owner Author

updated hyper-rustls to v0.26 in #137

@junkurihara junkurihara changed the title [Breaking changes] Update rustls from 0.21 to 0.22 (along with hyper-1.0) [Breaking changes] Update rustls from 0.21 to 0.22 Jan 14, 2024
@junkurihara junkurihara changed the title [Breaking changes] Update rustls from 0.21 to 0.22 [Breaking changes] Update rustls from 0.21 to 0.23 Apr 10, 2024
@yerke
Copy link

yerke commented May 23, 2024

hyperium/h3#238 was merged.

@junkurihara
Copy link
Owner Author

@yerke Thanks! Now waiting for the release ofs2n-quic-rustls. (It already supports rustls-0.23 in its unreleased version.)

@junkurihara
Copy link
Owner Author

Almost done in the draft PR #156. In my environment, it works flawlessly. Moreover, in the PR, we completely redefined the certificate manager as a separated crate that can support multiple types of certificate sources in addition to the static files in the current version. Thus in the future, we can extended the function of the cert manager to support other types of certificate management, e.g., ACME-based dynamic fetching of certificates.

NOTE:s2n-quic-0.38 supporting rustls-0.23 is not released yet. Currently, we are directly using the HEAD of s2n-quic repo.

@junkurihara
Copy link
Owner Author

Resolved by #126

@junkurihara junkurihara unpinned this issue Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants