Skip to content

ssl: more fine-grained test for cert validation failure#18424

Merged
davinci26 merged 2 commits intoenvoyproxy:mainfrom
Shikugawa:ssl_test
Oct 12, 2021
Merged

ssl: more fine-grained test for cert validation failure#18424
davinci26 merged 2 commits intoenvoyproxy:mainfrom
Shikugawa:ssl_test

Conversation

@Shikugawa
Copy link
Copy Markdown
Member

Signed-off-by: Shikugawa rei@tetrate.io

Commit Message: ssl: more fine-grained test for cert validation failure
Additional Description:
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Shikugawa <rei@tetrate.io>
Copy link
Copy Markdown
Member

@rojkov rojkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Added one nit.

Signed-off-by: Shikugawa <rei@tetrate.io>
Copy link
Copy Markdown
Member

@rojkov rojkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Copy Markdown
Member

@davinci26 davinci26 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, really nice addition

@davinci26 davinci26 merged commit 288c47f into envoyproxy:main Oct 12, 2021
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.

4 participants