-
Notifications
You must be signed in to change notification settings - Fork 418
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
header key, case-insensitive match #460
Conversation
24bbbc5
to
43db89a
Compare
/test pull-tekton-triggers-build-tests |
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.
Please add a little documentation in the TriggerBinding.md saying that the header comparison is case-insensitive.
/test pull-tekton-triggers-build-tests |
c8d287f
to
a1a3253
Compare
I added a statement and an example in the triggerbindings.md |
a1a3253
to
cf12041
Compare
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 Aki!
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ncskier The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Changes
This resolve #458
Issue:
The keys in the request header are converted by
CanonicalMIMEHeaderKey
before they are passed to the event sink code. So the cases of the key string are not same as sent from the source. This causes failure of matching header key. Theheader.X-GitHub-Event
was not found because it was converted toX-Github-Event
.Change:
The header keys specified in the TriggerBinding are converted with the same function before it is searched in the headers.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide for more details.
Release Notes