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.

Sign up for GitHub

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

Jump to bottom

feat: update rustls to 0.23 #186

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

feat: update rustls to 0.23 #186

wants to merge 1 commit into from

Conversation

leaf-potato
Copy link

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

GreptimeTeam/greptimedb#4557

What's changed and what's your intention?

  • Update prost from 0.12 to 0.13.
  • Update tonic from 0.11 to 0.12.

Checklist

  • I have written the necessary comments.
  • I have added the necessary unit tests and integration tests.

@leaf-potato leaf-potato mentioned this pull request Aug 16, 2024
Closed
@discord9
Copy link
Contributor

discord9 commented Nov 14, 2024 •
edited
Loading

What's the status of this PR? By the way rustls 0.23 default to use aws-lc-rs which have many build problems with different toolchain and stuff, as main repo(greptimedb patched it's rustls depend to use ring again it shouldn't be a problem for now) but would it affect greptime-proto when using it as a library?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fengys1996 fengys1996 fengys1996 approved these changes

@sunng87 sunng87 sunng87 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@leaf-potato @discord9 @sunng87 @fengys1996