Results 131 comments of Daniel Esik

Should I clarify something to get this PR viewed?

@SethTisue thank you for your review!

@SethTisue will this PR be included in the forthcoming 2.13 release? I can do a backport if it's needed.

@lrytz oh, didn't see it, great!

I'll say a super obvious and a little annoying thing, but did you try to bump the blaze-client version (at least to 0.21.33 if changing the major version isn't an...

I can't be sure on 100%, but I bet that forward compatibility is working ok for the 2.13 series. Though I'd give one more unsolicited piece of advice - it's...

Recently was did http4s/http4s#6198. So we hope the next releases will reduce some error logs. Especially about `AlreadyShutdownException`.

My two cents about this – `responseHeaderTimeout` works intentionally in that way. > `BlazeServerBuilder[F].withResponseHeaderTimeout(5.minutes)`, but this is not a good solution since it will let other request timeout too late....

I've chosen `groupByNelMap` because the underlying method `groupByMap` name is constructed like `groupBy` + `map`. In that sense, `groupByNelMap` is `groupByNel` (that exists in syntax) + `map`. I agree with...