Skip to content

access log: fix %UPSTREAM_CLUSTER% in upstream http access logs#17453

Merged
htuch merged 3 commits intoenvoyproxy:mainfrom
ggreenway:upstream-cluster-access-log
Jul 23, 2021
Merged

access log: fix %UPSTREAM_CLUSTER% in upstream http access logs#17453
htuch merged 3 commits intoenvoyproxy:mainfrom
ggreenway:upstream-cluster-access-log

Conversation

@ggreenway
Copy link
Copy Markdown
Member

Signed-off-by: Greg Greenway ggreenway@apple.com

Commit Message:
Additional Description: This has been broken for several versions, possibly by
commit 980c84d.
Risk Level: Low
Testing: Added new test
Docs Changes: none
Release Notes: added
Platform Specific Features: none
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

This has been broken for several versions, possibly by
commit 980c84d.

Signed-off-by: Greg Greenway <ggreenway@apple.com>
Signed-off-by: Greg Greenway <ggreenway@apple.com>
Signed-off-by: Greg Greenway <ggreenway@apple.com>
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@htuch htuch merged commit d977ce3 into envoyproxy:main Jul 23, 2021
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
…yproxy#17453)

This has been broken for several versions, possibly by commit 980c84d.

Risk Level: Low
Testing: Added new test


Signed-off-by: Greg Greenway <ggreenway@apple.com>
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.

2 participants