Skip to content

Add webhook delay support#627

Merged
axllent merged 2 commits intoaxllent:feature/webhook-delayfrom
rurbanovich:develop
Jan 31, 2026
Merged

Add webhook delay support#627
axllent merged 2 commits intoaxllent:feature/webhook-delayfrom
rurbanovich:develop

Conversation

@rurbanovich
Copy link
Contributor

This PR adds configurable delay support for webhook requests.

New Features:

--webhook-delay flag to set delay in seconds (default: 0)

MP_WEBHOOK_DELAY environment variable support

Delay is applied before rate limiting in webhook processing

Why:

Provides better control over webhook timing, useful for coordinating with external system processing.

No breaking changes - default behavior remains immediate webhook delivery.

@m1sterXD
Copy link

Yeah, we've encountered the same problem in our team, so having such feature would be nice!

@axllent
Copy link
Owner

axllent commented Jan 24, 2026

I am curious to understand the reason a delay is required for processing. Are either/both of you able to explain the use-case for this?

@rurbanovich
Copy link
Contributor Author

In my case, I'm using an asynchronous system, which, when using mailpit, will receive the delivery status before it stores the sending data, which leads to conflicts.

@axllent
Copy link
Owner

axllent commented Jan 26, 2026

OK, thank you for the explanation. I will review this and hopefully get it included in the next release (at the earliest this weekend) 👍

@axllent axllent changed the base branch from develop to feature/webhook-delay January 31, 2026 10:03
@axllent axllent merged commit 493d2c5 into axllent:feature/webhook-delay Jan 31, 2026
7 checks passed
axllent pushed a commit that referenced this pull request Feb 1, 2026
@axllent
Copy link
Owner

axllent commented Feb 1, 2026

This feature has now been included in v1.29.0 👍 Thank you for your contribution and feedback.

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.

3 participants