drogon icon indicating copy to clipboard operation
drogon copied to clipboard
verified publisher icon drogonframework

Add HttpClient pool

Open fantasy-peak opened this issue 1 year ago • 3 comments

fantasy-peak avatar Oct 18 '24 03:10 fantasy-peak

@hwc0919 @marty1885 please check

nqf avatar Oct 21 '24 05:10 nqf

@nqf Sorry Things cam up and I'm very busy at work RN. Will check sometime this week, most likely the weekends. From an initial look, I've concerns of this interacting unfavorably with our HTTP/2 client (you shouldn't pool HTTP/2).

marty1885 avatar Oct 22 '24 02:10 marty1885

@nqf Sorry Things cam up and I'm very busy at work RN. Will check sometime this week, most likely the weekends. From an initial look, I've concerns of this interacting unfavorably with our HTTP/2 client (you shouldn't pool HTTP/2).

yes, this is a client connection pool for http1.1, and i think http2 should not need it. http2 has not been merged into master yet

nqf avatar Oct 22 '24 02:10 nqf