-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[http codec] case-insensetive transfer-encoding checks #10055
Conversation
…oxy#10041 Signed-off-by: Oleg Guba <oleg@dropbox.com>
Can you merge master? caseCompare just got removed in place of an absl utility function. Otherwise LGTM. Thank you! /wait |
Signed-off-by: Oleg Guba <oleg@dropbox.com>
Signed-off-by: Oleg Guba <oleg@dropbox.com>
done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
) (#218) Signed-off-by: Oleg Guba <oleg@dropbox.com> Co-authored-by: Yuchen Dai <silentdai@gmail.com> Co-authored-by: Oleg Guba <oleg@dropbox.com>
@mattklein123 do you know if this fix will be merged into |
@andrijaperovic |
@howardjohn I see tags |
We didn't tag istio/envoy until recently. You can look at the git logs though |
Signed-off-by: Oleg Guba oleg@dropbox.com
Description: Per RFC7230: Hypertext Transfer Protocol (HTTP/1.1): Message Syntax and Routing transfer-coding names are case-insensitive:
Risk Level: Low
Testing: unit test
Docs Changes: None
Release Notes: None
Fixes #10041