Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove requests dependency #152

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Remove requests dependency #152

wants to merge 11 commits into from

Commits on Dec 18, 2023

  1. Start the messy path to removing requests 

    This starts the path toward removing the requests package by rejiggering *most* of our exception handling. There is a huge amount more to do and this doesn't even begin to start using urllib3 to make requests, so basically everything should be completely broken with this commit.

Part of #58.
    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    620a64f View commit details
    Browse the repository at this point in the history

  2. A horrifying implementation that passes tests 

    We can refine from here. Lots of bad things that need cleanup, and, well, maybe just too much stuff in general.
    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    5963af2 View commit details
    Browse the repository at this point in the history

  3. Fix flake8 errors and URL problem

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    9ce638a View commit details
    Browse the repository at this point in the history

  4. Rename ok to is_success on InternalHttpResponse 

    Taking some inspiration from httpx here, which I think had good reasoning for this change from requests.
    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    b3abbf9 View commit details
    Browse the repository at this point in the history

  5. Fix tests on urllib3 v1

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    34effec View commit details
    Browse the repository at this point in the history

  6. Move urllib3 mocking into separate module

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    a78d180 View commit details
    Browse the repository at this point in the history

  7. Create new _http module for HTTP stuff 

    This also moves the crazy gzip hack there for a start.
    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    193ee16 View commit details
    Browse the repository at this point in the history

  8. Fix lint issues/cleanup local plugin config

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    99c34f0 View commit details
    Browse the repository at this point in the history

  9. Unify session.request/session.send

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    1464230 View commit details
    Browse the repository at this point in the history

  10. Move other new HTTP stuff into _http.py

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    358e036 View commit details
    Browse the repository at this point in the history

  11. Move Session into _http.py

    @Mr0grog
    Mr0grog committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    8775f2c View commit details
    Browse the repository at this point in the history