Skip to content

CID-4118: Validate webhook signature before processing#135

Merged
mohamedlajmileanix merged 1 commit intomainfrom
feature/CID-4118/Validate-webhook-signature-before-removing-it
Jul 29, 2025
Merged

CID-4118: Validate webhook signature before processing#135
mohamedlajmileanix merged 1 commit intomainfrom
feature/CID-4118/Validate-webhook-signature-before-removing-it

Conversation

@mohamedlajmileanix
Copy link
Contributor

@mohamedlajmileanix mohamedlajmileanix commented Jul 23, 2025

🛠 Changes made

Validate webhook signature before processing

✨ Type of change

Please delete the options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

🧪 How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • should throw InvalidEventSignatureException when signature does not start with sha256=

🏎 Checklist:

  • My code follows the style guidelines
  • I have performed a self-review of my own code
  • My commit message clearly reflects the changes made
  • Assigned the appropriate labels (version, PR type, etc.)

@mohamedlajmileanix mohamedlajmileanix requested a review from a team as a code owner July 23, 2025 14:38
@mohamedlajmileanix mohamedlajmileanix added the bug Something isn't working label Jul 23, 2025
@github-actions
Copy link

LeanIX GitHub Agent Code Coverage

File Coverage [100%] 🍏
GitHubWebhookService.kt 100% 🍏
Total Project Coverage 64.71% 🍏

@mohamedlajmileanix mohamedlajmileanix merged commit fd6a9a5 into main Jul 29, 2025
4 checks passed
@mohamedlajmileanix mohamedlajmileanix temporarily deployed to ghcr:leanix-github-agent July 29, 2025 07:00 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

2 participants