mitodl / mitodl/release-script
Use httpx in ClientWrapper
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 16
- Forks
- 8
- Avg merge
- 11h 47m
- Merged PRs (30d)
- 2
Description
We should replace our current use of requests in ClientWrapper with httpx. We did this before but it was removed due to unexplained problems. However they say there is a 1.x release due this year so it may be a good time to give it a second try.
Currently, each HTTP request from requests blocks other async work, which can cause delays in Doof's responsiveness. Using httpx would help us prevent blocking communication
Contributor guide
No contributing guide indexed for this repository
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 locating ClientWrapper and tracing its current requests usage and how HTTP calls interact with async work. Review the earlier removal of httpx if that history is available, then verify that the replacement no longer blocks other async work and avoids the unexplained problems described in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100