reqwest icon indicating copy to clipboard operation
reqwest copied to clipboard

Hope to support "Max Conntions Per Host”.

Open lsk569937453 opened this issue 1 year ago • 3 comments

Hop to support the max_connections_per_host in the config of pool. The following is the implementation of golang https://github.com/golang/go/blob/go1.23.2/src/net/http/transport.go#L218

lsk569937453 avatar Oct 15 '24 09:10 lsk569937453

That would be useful for a use case of mine as well. Are there any plans to support it in the near future?

yonipeleg33 avatar Jan 04 '25 20:01 yonipeleg33

It would be useful for me too

mabiondo avatar Jan 09 '25 09:01 mabiondo

I think this is closed with https://seanmonstar.com/blog/hyper-util-composable-pools/ and:

.connector_layer(tower::limit::concurrency::ConcurrencyLimitLayer::new(100))

Right?

arcanis avatar Dec 17 '25 13:12 arcanis

I think this is closed with https://seanmonstar.com/blog/hyper-util-composable-pools/ and:

.connector_layer(tower::limit::concurrency::ConcurrencyLimitLayer::new(100))

Right?

I have no time rn to check the details, but the PR is really clear, so for me it's a yes.

mabiondo avatar Dec 24 '25 13:12 mabiondo

Note, we haven't yet replaced the pool in reqwest, that will come soon.

seanmonstar avatar Dec 24 '25 13:12 seanmonstar

Haven't had the opportunity to check that, but looks promising. Thank you all for the hard work!

yonipeleg33 avatar Dec 24 '25 13:12 yonipeleg33