Skip to content
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

[Release-1.6] Fix useragent in stackdriver access log (#3076) #3083

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

gargnupur
Copy link
Contributor

  • Fix useragent in stackdriver access log

Signed-off-by: gargnupur gargnupur@google.com

  • fix tests
    looks like user-agent was just getting ignored in tests before :(

Signed-off-by: gargnupur gargnupur@google.com

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

* Fix useragent in stackdriver access log

Signed-off-by: gargnupur <gargnupur@google.com>

* fix tests
looks like user-agent was just getting ignored in tests before :(

Signed-off-by: gargnupur <gargnupur@google.com>
@gargnupur gargnupur requested review from bianpengyuan and a team November 3, 2020 01:04
@google-cla google-cla bot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Nov 3, 2020
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 3, 2020
@gargnupur
Copy link
Contributor Author

/test test-tsan_proxy_release-1.6
/test check-wasm_proxy_release-1.6
/test release-test_proxy_release-1.6

@istio-testing istio-testing merged commit 980ec12 into istio:release-1.6 Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants