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

Fix for issue 4998 - Web Hook Rules check http headers in case sensitive manner #5038

Merged
merged 11 commits into from
Jan 28, 2022

Conversation

Rand01ph
Copy link
Contributor

@Rand01ph Rand01ph commented Sep 10, 2020

I have add a payload['headers_lower'] for webhook and add a unittest for this.
Fixes #4998

@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines. Very easy to review. label Sep 10, 2020
@stale
Copy link

stale bot commented Dec 12, 2020

Thanks for contributing to this issue. As it has been 90 days since the last activity, we are automatically marking is as stale. If this issue is not relevant or applicable anymore (problem has been fixed in a new version or similar), please close the issue or let us know so we can close it. On the contrary, if the issue is still relevant, there is nothing you need to do, but if you have any additional details or context which would help us when working on this issue, please include it as a comment to this issue.

@stale stale bot added the stale label Dec 12, 2020
@cognifloyd
Copy link
Member

FYI: we just merged a change that reformats all the code with black. Please rebase on master (or merge master into your branch) and reformat your code with black as well. You can use pre-commit install to have black run on your commits. Thanks!

@CLAassistant
Copy link

CLAassistant commented Sep 6, 2021

CLA assistant check
All committers have signed the CLA.

@stale stale bot removed the stale label Sep 6, 2021
Copy link
Member

@cognifloyd cognifloyd left a comment

Choose a reason for hiding this comment

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

@Rand01ph could you please sign the CLA? (If you signed it before, you'll need to resign it now that it says "Linux Foundation" instead of one of the previous StackStorm stewards).

Also, I just resolved the conflicts, merged master into your branch, and reformatted with black.

I have one style suggestion.

I would like to merge your PR fairly quickly if you can help me at least with the CLA. Hopefully there's not much left before we can merge this.

st2api/st2api/controllers/v1/webhooks.py Outdated Show resolved Hide resolved
@cognifloyd cognifloyd added API status:needs cla A manual tag to note PRs that are otherwise ready, but the CLA bot says the CLA has not been signed. labels Oct 5, 2021
@cognifloyd cognifloyd removed the status:needs cla A manual tag to note PRs that are otherwise ready, but the CLA bot says the CLA has not been signed. label Oct 7, 2021
@pull-request-size pull-request-size bot added size/M PR that changes 30-99 lines. Good size to review. and removed size/S PR that changes 10-29 lines. Very easy to review. labels Oct 7, 2021
@cognifloyd cognifloyd added this to the 3.7.0 milestone Oct 7, 2021
@arm4b arm4b added this to In progress in StackStorm v3.7.0 via automation Jan 28, 2022
@arm4b arm4b added the bug fix label Jan 28, 2022
@arm4b arm4b merged commit d333638 into StackStorm:master Jan 28, 2022
StackStorm v3.7.0 automation moved this from In progress to Done Jan 28, 2022
@arm4b
Copy link
Member

arm4b commented Jan 28, 2022

Merged! Thanks, everyone! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API bug fix enhancement size/M PR that changes 30-99 lines. Good size to review.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Web Hook Rules check http headers in case sensitive manner
4 participants