DependencyTrack / DependencyTrack/dependency-track

Provide signatures for webhook alerts

Open
#1,555 5 comments 2 reactions 0 assignees View on GitHub
enhancement good first issue p2 size/S
Dominant language
Java
Stars
4.2k
Forks
811
Avg merge
8h 39m
Merged PRs (30d)
237

Description

### Current Behavior:

Alerts / notifications sent via Webhook are neither authenticated nor signed in any way.
This makes it hard for receiving parties to verify whether a given notification was sent by DT.

### Proposed Behavior:

For the *Webhook* alert type, allow for an optional shared secret to be provided.
Before sending the webhook request, calculate an HMAC for the JSON payload, and include the resulting value in a request header (e.g. `X-Webhook-Signature`).

For reference, this is also how GitHub is doing it: https://docs.github.com/en/developers/webhooks-and-events/webhooks/securing-your-webhooks

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.