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

feat: Add custom header options for webhook notification #2044

Conversation

bunseokbot
Copy link
Contributor

Description

Add support for custom header options in webhook notifications. This functionality would be particularly useful for scenarios requiring Authorization headers.

Related issues

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@bunseokbot
Copy link
Contributor Author

FYI) There seems to be an issue on the codecov.
codecov/codecov-action#1280

Copy link
Collaborator

@chen-keinan chen-keinan left a comment

Choose a reason for hiding this comment

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

@bunseokbot lgtm 🚀 thank you for the contribution

@chen-keinan chen-keinan merged commit 96a010b into aquasecurity:main Apr 28, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add custom header option for webhook notification
2 participants