Skip to content

fix(webhook): retry logic#581

Merged
wolf31o2 merged 1 commit intomainfrom
fix/webhook-retry
Jan 18, 2026
Merged

fix(webhook): retry logic#581
wolf31o2 merged 1 commit intomainfrom
fix/webhook-retry

Conversation

@wolf31o2
Copy link
Copy Markdown
Member

@wolf31o2 wolf31o2 commented Jan 18, 2026

Closes #334


Summary by cubic

Adds retry with exponential backoff for webhook delivery to improve reliability and prevent lost events. Addresses #334 by handling transient failures and reporting exhausted retries.

  • Bug Fixes

    • Retries failed webhook sends with exponential backoff and a max backoff cap.
    • Logs attempts and final failure; sends a non-blocking error to errorChan.
  • New Features

    • Configurable retry via WithRetryConfig(maxRetries, initialBackoff, maxBackoff).
    • Defaults: 3 retries, 1s initial backoff, 30s max backoff, factor 2.0.

Written for commit 4a95a79. Summary will update on new commits.

Signed-off-by: Chris Gianelloni <wolf31o2@blinklabs.io>
@wolf31o2 wolf31o2 requested a review from a team as a code owner January 18, 2026 00:22
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@blinklabs-io blinklabs-io deleted a comment from coderabbitai bot Jan 18, 2026
@wolf31o2 wolf31o2 merged commit 59d398d into main Jan 18, 2026
11 checks passed
@wolf31o2 wolf31o2 deleted the fix/webhook-retry branch January 18, 2026 16:39
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.

Handle errors in webhook output

2 participants