Add a 10 second max timeout to flush requests

[benjackwhite]

When shutting down, users are advised to call .flush to ensure all queued events are sent. Without a timeout this can block the shutdown, causing other issues

[neilkakkar]

Neil to check if timeouts on requests are sufficient to not block thread joins

[0xRaduan]

How long does flush take in general? I just read about using it here, and I have a bunch of short-lived tasks in Celery. However, what I've noticed, is that after putting flush at the end of my celery task, it's blocked(for >10 minutes), and never actually finishes / nor sends then request.

For now I am using sync client, but I want to use async one, because otherwise I have celery tasks, which can produce a lot of events, and I don't want to use sync calls for that.

Please advise.

[taobojlen]

I have a similar question.

In our Celery workers, we set sync_mode=True -- otherwise, some events don't get sent to PostHog. I just learned about posthog.flush() which seems like it'd be great to use in a Celery task_postrun callback or similar, but I'd like to know if it works first!