Skip to content

[release/v1.18] jwt_authn: implementation of www-authenticate header (#16216)#17426

Merged
lizan merged 1 commit intoenvoyproxy:release/v1.18from
cyran-ryan:backport_to_1.18
Jul 28, 2021
Merged

[release/v1.18] jwt_authn: implementation of www-authenticate header (#16216)#17426
lizan merged 1 commit intoenvoyproxy:release/v1.18from
cyran-ryan:backport_to_1.18

Conversation

@cyran-ryan
Copy link
Copy Markdown
Contributor

Commit Message: Cherry-pick of 9fee580 of "jwt_authn: implementation of www-authenticate header (#16216)" to release 1.18
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@cyran-ryan cyran-ryan requested a review from lizan as a code owner July 20, 2021 19:50
Signed-off-by: Ryan Sutton <ryan.sutton@volunteers.acasi.info>

Co-authored-by: Lizan Zhou <lizan@tetrate.io>

Signed-off-by: cyran-ryan <58446431+cyran-ryan@users.noreply.github.com>
@dmitri-d
Copy link
Copy Markdown
Contributor

This lgtm, although I noticed that changes to test/common/tracing/http_tracer_impl_test.cc present in the original PR are not here. I don't think MetadataCustomTagReturnsDefaultValue test is related to main changes however. In the future, please mention in the PR description the changes made the original PR, it makes reviewing cherry-picks/ports easier. Thanks!

@lizan lizan merged commit 78ab00f into envoyproxy:release/v1.18 Jul 28, 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.

3 participants