fix(ploeg): reject tracker webhooks when no signing secret is set #142

Merged
ryangr0 merged 1 commit from ryangr0/ploeg-tracker-webhook-secret into development 2026-10-03 00:01:50 +00:00 AGit
Owner

Vikunja and ClickUp skipped signature verification when their secret was
empty, so anyone who could reach /webhooks/tracker/{provider} could assign
or withdraw work. They now reject every delivery without a configured
secret, the same as the Forgejo and GitLab forge providers, and ploegd
warns at startup when a tracker secret is missing.

Production already sets PLOEG_VIKUNJA_SECRET through the ploeg-webhook-secret ExternalSecret, so live Vikunja dispatch is unaffected. Deployments without it now reject every tracker webhook and log a warning at startup.

Verified with mise run verify on the pinned toolchain (all gates passed).

Ticket: https://vikunja.webgrip.dev/tasks/1716

🤖 Generated with Claude Code

Vikunja and ClickUp skipped signature verification when their secret was empty, so anyone who could reach /webhooks/tracker/{provider} could assign or withdraw work. They now reject every delivery without a configured secret, the same as the Forgejo and GitLab forge providers, and ploegd warns at startup when a tracker secret is missing. Production already sets PLOEG_VIKUNJA_SECRET through the ploeg-webhook-secret ExternalSecret, so live Vikunja dispatch is unaffected. Deployments without it now reject every tracker webhook and log a warning at startup. Verified with `mise run verify` on the pinned toolchain (all gates passed). Ticket: https://vikunja.webgrip.dev/tasks/1716 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(ploeg): reject tracker webhooks when no signing secret is set
Some checks failed
[Workflow] On Pull Request / checks (pull_request) Has been cancelled
[Workflow] On Pull Request / warnings (pull_request) Has been cancelled
[Workflow] On Pull Request / release-policy (pull_request) Has been cancelled
347bc775c4
Vikunja and ClickUp skipped signature verification when their secret was
empty, so anyone who could reach /webhooks/tracker/{provider} could assign
or withdraw work. They now reject every delivery without a configured
secret, the same as the Forgejo and GitLab forge providers, and ploegd
warns at startup when a tracker secret is missing.

VIK-1716

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ryangr0 merged commit 2b52b9ae19 into development 2026-10-03 00:01:50 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
webgrip/unfold!142
No description provided.