sentry-rust icon indicating copy to clipboard operation
sentry-rust copied to clipboard

Hyper Transport

Open kvnvelasco opened this issue 2 years ago • 3 comments
trafficstars

Reqwest adds quite a bit of time to compile times in projects where it isn't needed. Removing reqwest from our project reduced compile times by about 30s. We already have hyper in the dependency tree via axum so it was mostly free.

Willing to put up an implementation of this if it's supported.

kvnvelasco avatar Feb 07 '23 06:02 kvnvelasco

I believe this is a good idea, yes. What exactly would be the advantages feature wise? As reqwest is based on hyper itself, but I assume you have to implement a ton of stuff yourself.

Swatinem avatar Feb 07 '23 08:02 Swatinem

It would definitely be a feature reduction actually. Proxy support is something that we've intentionally left out of scope (because we don't use one). The reduction in feature scope reduces compile times and we find the trade off advantageous in that way.

kvnvelasco avatar Feb 13 '23 01:02 kvnvelasco

@kvnvelasco Is your code on a public branch somewhere?

fenollp avatar Apr 05 '24 14:04 fenollp