-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does this project support urllib3 directly? #981
Comments
No, requests-cache doesn't support urllib3 directly. Most of this library's public API is very specific to Here is what you do have to work with, though:
I've considered making some shared utilities that could work with multiple HTTP clients, but my sense so far is that trying to make that work would be messier than just duplicating code. For aiohttp, for example, I opted to make a completely separate library, although the main factor there was the difficulty of reusing sync and async logic, which wouldn't apply to urllib3. While I'm probably not interested in a massive refactor of requests-cache to add multi-client support, I may be open to some smaller incremental changes, if you want to make it easier to add urllib3 support in either a fork or a separate library that depends on requests-cache. |
Background
I've been investigating persistent cache options for urllib3 and cake across the following issue urllib3/urllib3#2166
The Question
So I'm wondering if this lib supports urllib3 directly and if not would you accept a pr to add this functionality? Currently I'm not sure on how this would be implemented but can have a play around
Other options
I could make some urllib3-cache library however this would result in a bunch of duplicated work. I'd rather spend that time contributing to an existing lib
Thanks for your time and an awesome project :)
The text was updated successfully, but these errors were encountered: