python-zulip-api icon indicating copy to clipboard operation
python-zulip-api copied to clipboard

Improve error handling when a request fails due to the persistent connection being expired

Open timabbott opened this issue 1 year ago • 10 comments

Some networks drop long-lived inactive TCP connections, which can lead to connection errors when using the Zulip API bindings for some persistent jobs. We should ideally either enable a keep-alive, or perhaps a transparent retry for this class of connection error.

See https://github.com/zulip/zulip/commit/ff647dff03f178c329240c0e938d3fff8fbc5ab0 for an example.

timabbott avatar Jul 30 '22 16:07 timabbott

@zulipbot claim

Beanpen avatar Nov 10 '22 23:11 Beanpen

Welcome to Zulip, @Beanpen! We just sent you an invite to collaborate on this repository at https://github.com/zulip/python-zulip-api/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

zulipbot avatar Nov 10 '22 23:11 zulipbot

Hi @timabbott I wonder if you mean to remove all the alias of "client = zulip.Client(...)" and instead directly call "zulip.Client(...)" in all files. Thank you!

Beanpen avatar Nov 11 '22 00:11 Beanpen

@Beanpen You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar Nov 21 '22 00:11 zulipbot

@zulipbot claim

ryansong13 avatar Dec 06 '22 19:12 ryansong13

Welcome to Zulip, @ryansong13! We just sent you an invite to collaborate on this repository at https://github.com/zulip/python-zulip-api/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

zulipbot avatar Dec 06 '22 19:12 zulipbot

@zulipbot abandon

ryansong13 avatar Dec 13 '22 19:12 ryansong13

@zulipbot claim

akshitadixit avatar Dec 25 '22 07:12 akshitadixit

Welcome to Zulip, @akshitadixit! We just sent you an invite to collaborate on this repository at https://github.com/zulip/python-zulip-api/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

zulipbot avatar Dec 25 '22 07:12 zulipbot

@akshitadixit You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar Jan 04 '23 12:01 zulipbot