search

deltachat / mailadm

mail account administration tool for temporary and other account creation/modification
https://mailadm.readthedocs.io/
Mozilla Public License 2.0
14 stars 1 forks source link

set timeout for adding users in mailcow to 30 seconds #41

Closed missytake closed 2 years ago

missytake commented 2 years ago

Hopefully we get rid of the weird timeout errors this way and get verbose timeout errors now <.<

missytake commented 2 years ago

the failing tests are fixed by #40.

link2xt commented 2 years ago

Documentation says there is no default timeout btw: https://requests.readthedocs.io/en/latest/user/quickstart/#timeouts

You can tell Requests to stop waiting for a response after a given number of seconds with the timeout parameter. Nearly all production code should use this parameter in nearly all requests. Failure to do so can cause your program to hang indefinitely.

gunicorn then timeouts probably because all workers hanged and no other requests can be processed.

missytake commented 2 years ago

the others are not web-facing and not executed by gunicorn, so we should just get a proper traceback when we call it through a CLI. This has the advantage that we can better react to these errors and debug them. They are a very different area

missytake commented 2 years ago

rebased on master, let's see if CI succeeds now :) (I don't see a reason why not)