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

Upgrade to rustls 0.20 #198

Merged
merged 2 commits into from Nov 15, 2021
Merged

Upgrade to rustls 0.20 #198

merged 2 commits into from Nov 15, 2021

Conversation

dnaka91
Copy link
Contributor

@dnaka91 dnaka91 commented Oct 18, 2021

Upgrading to rustls version 0.20, same as in snapview/tungstenite-rs#244.

Marked as draft as it depends on rustls-native-certs and the mentioned PR for tungstenite.

sdroege added a commit to sdroege/async-tungstenite that referenced this pull request Nov 6, 2021
Based on tokio-tungstenite PR:
  snapview/tokio-tungstenite#198

Co-authored-by: Sebastian Dröge <sebastian@centricular.com>
sdroege added a commit to sdroege/async-tungstenite that referenced this pull request Nov 6, 2021
Based on tokio-tungstenite PR:
  snapview/tokio-tungstenite#198

Co-authored-by: Sebastian Dröge <sebastian@centricular.com>
@yusdacra
Copy link

yusdacra commented Nov 10, 2021

Any reason why this is not merged yet? I would like to be able to use a new release with this change to not have duplicate dependencies 😅

@dnaka91
Copy link
Contributor Author

dnaka91 commented Nov 11, 2021

Ah sorry @yusdacra I haven't checked for new releases of tungstenite in the recent days. It's updated now to use the latest official release that contains required changes.

@dnaka91 dnaka91 marked this pull request as ready for review November 11, 2021 01:03
@daniel-abramov
Copy link
Member

Thanks guys! Going to release it soon.

@daniel-abramov daniel-abramov merged commit 7a520c2 into snapview:master Nov 15, 2021
@dnaka91 dnaka91 deleted the rustls-0.20 branch November 16, 2021 01:23
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