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.

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

feat: Add a Error::is_connect() method #95

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

jakubadamw
Copy link
Contributor

It will be useful for reqwest's upgrade to hyper v1.

jakubadamw added a commit to grafbase/reqwest that referenced this pull request Jan 29, 2024
jakubadamw added a commit to grafbase/reqwest that referenced this pull request Jan 29, 2024
@seanmonstar seanmonstar merged commit 32166f6 into hyperium:master Jan 29, 2024
16 checks passed
seanmonstar pushed a commit to seanmonstar/reqwest that referenced this pull request Jan 31, 2024
…onstar/reqwest#hyper-v1 (#2115)

* Implement `size_hint()` for `async_impl::body::Body`.

* Add the `Sync` bound to `S` in `async_impl::body::Body::wrap_stream()`.

* Install a `TokioTimer` for all targets other than wasm32.

* Implement `Accepts::none()`.

* Uncomment and fix `impl<T: Into<Body>> From<http::Response<T>> for Response`.

* Other fixes.

* Patch `hyper-util` until hyperium/hyper-util#95 is merged.

* A few fixes to the tests.
seanmonstar pushed a commit to seanmonstar/reqwest that referenced this pull request Jan 31, 2024
…onstar/reqwest#hyper-v1 (#2115)

* Implement `size_hint()` for `async_impl::body::Body`.

* Add the `Sync` bound to `S` in `async_impl::body::Body::wrap_stream()`.

* Install a `TokioTimer` for all targets other than wasm32.

* Implement `Accepts::none()`.

* Uncomment and fix `impl<T: Into<Body>> From<http::Response<T>> for Response`.

* Other fixes.

* Patch `hyper-util` until hyperium/hyper-util#95 is merged.

* A few fixes to the tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants