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 rustls family to 0.23 #4557

Closed
sunng87 opened this issue Aug 14, 2024 · 5 comments · Fixed by #4732
Closed

Upgrade rustls family to 0.23 #4557

sunng87 opened this issue Aug 14, 2024 · 5 comments · Fixed by #4732
Labels
C-enhancement Category Enhancements help wanted Extra attention is needed

Comments

@sunng87
Copy link
Member

sunng87 commented Aug 14, 2024

What type of enhancement is this?

Other

What does the enhancement do?

Upgrade the entire protocol tier to rustls 0.23.

Implementation challenges

No response

@sunng87 sunng87 added C-enhancement Category Enhancements help wanted Extra attention is needed labels Aug 14, 2024
@leaf-potato
Copy link
Contributor

Maybe I can give it a try.

@leaf-potato
Copy link
Contributor

Curious about what issues rustls 0.23 solves and why greptime needs to be upgraded.

@sunng87
Copy link
Member Author

sunng87 commented Aug 16, 2024

It's not an urgent issue. We just want to keep our dependencies up-to-date.

@leaf-potato
Copy link
Contributor

leaf-potato commented Aug 16, 2024

@sunng87 The greptime-proto code: GreptimeTeam/greptime-proto#186.

@sunng87
Copy link
Member Author

sunng87 commented Aug 17, 2024

@leaf-potato Thank you! It may take some time to get the proto merged and you don't have to be blocked by this. Just use your git branch for now to upgrade in greptimedb.

@sunng87 sunng87 linked a pull request Sep 19, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category Enhancements help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants