Skip to content

WPB-23816 log wire client and wire client verison headers in nginz#5123

Open
battermann wants to merge 3 commits intodevelopfrom
WPB-23816-log-wire-client-and-wire-client-verison-headers-in-nginz
Open

WPB-23816 log wire client and wire client verison headers in nginz#5123
battermann wants to merge 3 commits intodevelopfrom
WPB-23816-log-wire-client-and-wire-client-verison-headers-in-nginz

Conversation

@battermann
Copy link
Contributor

@battermann battermann commented Mar 16, 2026

https://wearezeta.atlassian.net/browse/WPB-23816

Tested manually:

127.0.0.1 - - [16/Mar/2026:15:37:52 +0000] "GET /v15/system/settings/unauthorized HTTP/1.1" 200 64 "-" "-" "-" - 1 0.002 0.002 - - - f6557d05ef0c8a71c211e44e5ee1d1e8 - "Web" "1.2.3" "5d41402abc4b2a76b9719d911017c592

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 16, 2026
@battermann battermann marked this pull request as ready for review March 16, 2026 15:44
@battermann battermann requested review from a team as code owners March 16, 2026 15:44
Copy link
Contributor

@pcapriotti pcapriotti left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Paolo Capriotti <paolo@capriotti.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants