Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
oneclick: Do not use a stale Zulip client.
Initializing the Zulip client opens a long-lived TCP connection due to connection pooling in urllib3. In Github Actions, the network kills such requests after ~270s, making the later `send_message` call fail. Use a singular call to `zulip.Client()` early on to verify the credentials, and do not cache the resulting client object. Instead, re-create it during the final step when it is needed, so we do not run afoul of bad TCP connection state. This would ideally be fixed via connection keepalive or retry at the level of the Zulip module.
- Loading branch information