-
Notifications
You must be signed in to change notification settings - Fork 654
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
fix(headers): Metadata requestHeaders lookup should be case insensitive #863
Conversation
@ezimanyi I assume this will need to be cherry picked into a release? |
Yup, for sure, in general feel free to just request the backport from mergify; I review the list of outstanding cherry-picks every day and merge them in. |
@Mergifyio backport release-1.19.x |
Command
|
Ah, very cool! |
In #790 I removed
HttpHeaders
so thatecho-api
could be dependency free (basically, just Java) -- this is important for the ongoing extensibility / plugin work we are doing.However, I overlooked that
HttpHeaders
builds a case insensitive map. This resolves that issue.Resolves: spinnaker/spinnaker#5645
Relates to: #840