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

Update rust-native-tls and dev-dependencies to latest #32

Merged
merged 2 commits into from
Mar 1, 2023
Merged

Update rust-native-tls and dev-dependencies to latest #32

merged 2 commits into from
Mar 1, 2023

Conversation

nickbp
Copy link
Contributor

@nickbp nickbp commented Feb 11, 2022

rust-native-tls 0.23 is pretty old (2019) and lacks more recent ALPN support, needed for HTTP/2.

Updates some dev-dependencies to latest as well.

Lastly cleans up a compile warning that was seen when running cargo test (probably a new check in recent rustc?)

rust-native-tls 0.23 is pretty old (2019) and lacks more recent [ALPN support](sfackler/rust-native-tls#194), needed for HTTP/2.

Updates some dev-dependencies to latest as well.

Lastly cleans up a compile warning that was seen when running `cargo test` (probably a new check in recent rustc?)
@nickbp
Copy link
Contributor Author

nickbp commented Dec 12, 2022

hello

@nickbp
Copy link
Contributor Author

nickbp commented Feb 25, 2023

It's been over a year, the project looks dead.

I've created a separate async-native-tls-alpn crate containing this patch. That crate is built based on a crate branch in my fork repo.

I'll leave the PR open in case the project comes back to life, but in the meantime, anyone who is hitting ALPN issues can use that crate.

@link2xt
Copy link
Contributor

link2xt commented Mar 1, 2023

@nickbp I think nobody got the notification, looking into the PR now. It usually helps to @ some contributors, I guess nobody was watching the repo.

@link2xt link2xt merged commit dd2c885 into async-email:master Mar 1, 2023
@link2xt link2xt mentioned this pull request Mar 1, 2023
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

2 participants