-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Remove requests
from public interfaces
#157
Draft
Mr0grog
wants to merge
17
commits into
main
Choose a base branch
from
58-first-lets-bury-requests
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Commits on Dec 26, 2023
-
Configuration menu - View commit details
-
Copy full SHA for bdbdc06 - Browse repository at this point
Copy the full SHA bdbdc06View commit details -
Configuration menu - View commit details
-
Copy full SHA for 18527aa - Browse repository at this point
Copy the full SHA 18527aaView commit details
Commits on Dec 29, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 01e6d83 - Browse repository at this point
Copy the full SHA 01e6d83View commit details -
Add
InternalHttpResponse
to provide a non-requests Response interfaceThis also provides some basic (but not yet fully implemented) thread-safety for response objects.
Configuration menu - View commit details
-
Copy full SHA for 1eb3e17 - Browse repository at this point
Copy the full SHA 1eb3e17View commit details -
Delegate to requests for encoding sniffing
Also re-organize methods a bit
Configuration menu - View commit details
-
Copy full SHA for ccc8f40 - Browse repository at this point
Copy the full SHA ccc8f40View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0f3ef22 - Browse repository at this point
Copy the full SHA 0f3ef22View commit details -
Configuration menu - View commit details
-
Copy full SHA for 172e6a6 - Browse repository at this point
Copy the full SHA 172e6a6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4843ed4 - Browse repository at this point
Copy the full SHA 4843ed4View commit details
Commits on Jan 2, 2024
-
Break Session functionality into separate methods
Sessions currently handle rate limiting and retries (and *need* to handle redirects so those get rate limited; it's a bug that they do not) and their implementation is a bit complicated. This separates out each of those jobs into a separate method to make management of those separate (but related) tasks a bit clearer. This should make the code a little easier to change, which will probably happen next because I think I want to remove `WaybackSession` entirely.
Configuration menu - View commit details
-
Copy full SHA for 9d9a16c - Browse repository at this point
Copy the full SHA 9d9a16cView commit details -
Configuration menu - View commit details
-
Copy full SHA for de28981 - Browse repository at this point
Copy the full SHA de28981View commit details
Commits on Jan 4, 2024
-
Configuration menu - View commit details
-
Copy full SHA for f2eb7e5 - Browse repository at this point
Copy the full SHA f2eb7e5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 086e034 - Browse repository at this point
Copy the full SHA 086e034View commit details -
I think we actually broke this functionality somewhere, so it needs a test
Configuration menu - View commit details
-
Copy full SHA for 2f48476 - Browse repository at this point
Copy the full SHA 2f48476View commit details -
Configuration menu - View commit details
-
Copy full SHA for f43ba0a - Browse repository at this point
Copy the full SHA f43ba0aView commit details
Commits on Jan 5, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 0ec7811 - Browse repository at this point
Copy the full SHA 0ec7811View commit details -
Configuration menu - View commit details
-
Copy full SHA for cf6d879 - Browse repository at this point
Copy the full SHA cf6d879View commit details -
Configuration menu - View commit details
-
Copy full SHA for 994c8b7 - Browse repository at this point
Copy the full SHA 994c8b7View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.