chore: remove h2 from dependencies when http2 feature is off

[mladedav]

Motivation

Closes #2386

h2 does not need to be a dependency when the feature http2 is not enabled.

Solution

I've changed the features of hyper-util we depend on conditionally.

cargo tree -i h2 --features default,macros,multipart,ws --edges no-dev prints nothing so the dependency is not there anymore.

[xfmoulet]

v 0.1.3 hyper-util changelog seems to further separation of http1 and http2, would it be wise to upgrade to this version ?

[w-flo]

Does axum even build on hyper-util 0.1.2 without enabling its "server-auto" feature? I believe this is only possible since hyper-util 0.1.3?

[mladedav]

@xfmoulet You're right, as @w-flo pointed out, it does not even compile with the old version. Thank you both.

Since there is no Cargo.lock, cargo helpfully provided me with the new version so I didn't notice it fails to build with the version specified in Cargo.toml.

[xfmoulet]

built a project in http1 with this version, h2 is not pulled anymore.

[jplatte]

Thanks for the PR, and sorry for the delay!

[jplatte]

Actually, could you add a changelog entry (might cause conflicts so rebase recommended) before I merge?

[mladedav]

I've added the changelog entry