-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[Backport 1.21] TLS: Allow specification of both typed and non-typed san matchers in config (20529) #21170
Conversation
…san matchers in config (20529) Signed-off-by: Pradeep Rao <pcrao@google.com>
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
/retest |
Retrying Azure Pipelines: |
/assign @ggreenway |
/assign @adisuissa |
CI is failing |
Signed-off-by: Pradeep Rao <pcrao@google.com>
Can't reproduce the build failure locally. Doing a merge to kick off tests again. |
…1_21 Signed-off-by: Pradeep Rao <pcrao@google.com>
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.
Overall LGTM. Left a small nit
/lgtm api
/wait
* cryptomb: remove RSA PKCS1 v1.5 padding support. | ||
* perf: ssl contexts are now tracked without scan based garbage collection and greatly improved the performance on secret update. |
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.
These changes don't seem related to the current backport PR.
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.
Aah, this happened because of this: https://github.com/envoyproxy/envoy/pull/21226/files
Fixing.
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, thanks!
Signed-off-by: Pradeep Rao pcrao@google.com
Commit Message:
Additional Description:
Backport #20529 to 1.21
Risk Level: Low
Testing: Modified test.
Docs Changes:
Release Notes:
Platform Specific Features: