OpenAPITools / OpenAPITools/openapi-generator
[REQ] Add basic retries to asyncio Python REST client
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 26.8k
- Forks
- 7.7k
- PR merge metrics
- PR metrics pending
Description
Is your feature request related to a problem? Please describe.
The synchronous version of REST client supports basic retries through urllib3 library using the retries configuration parameter. The asyncio version should support the same behaviour such that the retries parameter can be honoured in this version.
I attempted a change to the template files that would appear to handle the case using the aiohttp-retry library. There were no tests for retry in the synchronous version so I have not added any to this change. Please feel free to improve upon this, the feature would be useful.
https://github.com/rad-pat/openapi-generator/tree/add-basic-retry-asyncio
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the asyncio client template changes in the linked add-basic-retry-asyncio branch and compare them with the synchronous client's urllib3 retry configuration. Confirm how the retries parameter should be honored with aiohttp-retry, then verify that a generated asyncio REST client exhibits the expected retry behavior. The issue does not name specific files or tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api, tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100