Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deal with timeouts because sending mail takes too long in production #254

Closed
Jumpy-Squirrel opened this issue Jan 26, 2025 · 1 comment · Fixed by #255 or #256
Closed

deal with timeouts because sending mail takes too long in production #254

Jumpy-Squirrel opened this issue Jan 26, 2025 · 1 comment · Fixed by #255 or #256
Assignees

Comments

@Jumpy-Squirrel
Copy link
Contributor

We are seeing timeouts in production in two situations:

  • While performing status changes. Here the solution is probably to increase the (currently rather small) timeout for the endpoint.
  • While receiving the payments-changed webhook. Here we cannot increase the endpoint timeout further because our payment provider that originally sends the request that causes this webhook has a hard timeout of 10 seconds. Solution is probably to send this email asynchronously.
    • Note that for other processing, async sending of emails has some undesirable downsides. We WANT the admin to see an error if a "you have been cancelled" mail cannot be sent.
@Jumpy-Squirrel
Copy link
Contributor Author

Logs from async processing not shown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant