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
[tls] Add set min/max TLS version APIs to TLS credentials APIs. #34861
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor comments.
Also, once #34859 is merged, could we add to the e2e test where we force TLS 1.2 and force TLS 1.3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Will added e2e tests as a follow up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, one minor comment
…#34861) Address grpc#28382. This is a recreation of grpc#31368 except e2e tests are not handled here (yet). Closes grpc#34861 COPYBARA_INTEGRATE_REVIEW=grpc#34861 from rockspore:tls_version f9a1215 PiperOrigin-RevId: 589847110
…#34861) Address grpc#28382. This is a recreation of grpc#31368 except e2e tests are not handled here (yet). Closes grpc#34861 COPYBARA_INTEGRATE_REVIEW=grpc#34861 from rockspore:tls_version f9a1215 PiperOrigin-RevId: 589847110
…#34861) Address grpc#28382. This is a recreation of grpc#31368 except e2e tests are not handled here (yet). Closes grpc#34861 COPYBARA_INTEGRATE_REVIEW=grpc#34861 from rockspore:tls_version f9a1215 PiperOrigin-RevId: 589847110
Co-authored-by: Luwei Ge <lwge@google.com>
Co-authored-by: Luwei Ge <lwge@google.com>
Backport of #34861 to 1.49. --------- Co-authored-by: Luwei Ge <lwge@google.com> Co-authored-by: Esun Kim <veblush@google.com>
Address #28382. This is a recreation of #31368 except e2e tests are not handled here (yet).