Skip to content

fix: deduplicate retried webhook deliveries#59

Merged
darthfork merged 1 commit intomainfrom
fix/issue-42-dedupe-deliveries
May 1, 2026
Merged

fix: deduplicate retried webhook deliveries#59
darthfork merged 1 commit intomainfrom
fix/issue-42-dedupe-deliveries

Conversation

@darthfork
Copy link
Copy Markdown
Owner

Summary

  • add an in-memory delivery deduper keyed by X-GitHub-Delivery with TTL-based retention and bounded size
  • drop retried webhook deliveries after signature validation and expose duplicate seen/dropped counters
  • add integration coverage that posts real webhook requests and verifies duplicate deliveries do not inflate exported metrics

Closes #42

Testing

  • make test
  • make lint

@darthfork darthfork force-pushed the fix/issue-42-dedupe-deliveries branch from 710e948 to c0f8136 Compare May 1, 2026 06:35
@darthfork darthfork merged commit 9c0f983 into main May 1, 2026
2 checks passed
@darthfork darthfork deleted the fix/issue-42-dedupe-deliveries branch May 1, 2026 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deduplicate retried GitHub webhook deliveries

1 participant