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

Wipe webhook credentials from messages that are passed into the flow #28

Open
jhorbulyk opened this issue Feb 28, 2020 · 1 comment
Open
Labels
Ready For ToDo Task is ready for development

Comments

@jhorbulyk
Copy link
Contributor

Currently, if you create a flow where the first component is a webhook component with authentication all received headers are passed to the next step in the flow (in the body). Ideally, the authentication headers should be removed. Ex:

  • If authentication type is No Auth -> Nothing to do
  • If authentication type is Basic Auth -> Remove Authorization header
  • If authentication type is API Key auth -> Remove the specific key from the header
  • If authentication type is HMAC -> remove whatever header contains the HMAC value.
@jhorbulyk
Copy link
Contributor Author

For the sake of backwards compatibility, we may need to add the above functionality toggle-able with a config field.

@andrewreshitko andrewreshitko added the Ready For ToDo Task is ready for development label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For ToDo Task is ready for development
Projects
None yet
Development

No branches or pull requests

2 participants