Feature Request: Optional retries on HTTP requests #80
DamianHeard
started this conversation in
Ideas
Replies: 2 comments
-
Hey @DamianHeard appreciate your feedback! I'm going to convert this to a discussion topic for the community to gauge temperature as it's not something we've seen requested previously. If you would like to put up a PR for the change we'd welcome the contribution! |
Beta Was this translation helpful? Give feedback.
0 replies
-
@rdemarco-xero can we implement @DamianHeard 's suggestion above? This relates to the timeout issue and could fix that. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Background
xero-python
usesurllib3
under the hood. By default, urllib3 will retry requests 3 times which is not necessarily desirable, particularly if rate limiting is in play. In some case it might make sense to the API user to handle retries themselves.Proposed solution
xero_python.api_client.configuration.Configuration
should acceptretries
as an argument which in turn would be passed down to theurllib3.PoolManager
orurllib3.ProxyManager
during initialisation.I'm happy to put a PR up if you feel the change has merit.
Beta Was this translation helpful? Give feedback.
All reactions