search

hyperium / hyper

An HTTP library for Rust
https://hyper.rs
MIT License
14.08k stars 1.55k forks source link

feat: relax connection IO trait bounds #3595

Closed ehaydenr closed 3 months ago

ehaydenr commented 3 months ago

I can't tell if this was intentional. hyper 0.14.x didn't have this requirement and as far as I can tell it's not actually needed.

dswij commented 3 months ago

For context, I believe that this is here for https://github.com/hyperium/hyper/blob/4d0c184f935adf93ed05b86ec3a16e432aa81706/src/server/conn/http1.rs#L180.

But the Connection impl itself, and Dispatch does not require this bound, so it's safe to remove this from impl Future.

dswij commented 3 months ago

Thank you! @ehaydenr