BlazeClient: requestTimeout is not enforced

Problem Description

requestTimeout in BlazeClient is not enforced on certain requests.

Expected behavior: A timeout exception is thrown after 1 second.

Actual behavior: The request seems to be stuck indefinitely.

Related issues: #2338 (almost certainly the same source of the problem)

Source

Main.scala

import cats.effect._
import org.http4s._
import org.http4s.client.blaze.BlazeClientBuilder

import scala.concurrent.duration._
import scala.concurrent.ExecutionContext

object Main extends IOApp {

  override def run(args: List[String]): IO[ExitCode] = for {
    c <- BlazeClientBuilder[IO](ExecutionContext.global)
        .withRequestTimeout(1.second)
        .resource
        .allocated
    (client, _) = c
    _ <- IO { println("Sending a request.") }
    _ <- client.fetch(Request[IO](Method.GET, Uri.unsafeFromString("http://1.2.3.4:12345/")))(_ => IO.unit)
    _ <- IO { println("Got a response.") }
  } yield ExitCode.Success
}

build.sbt

name := "http4s-client-request-timeout-bug"

version := "0.1"

scalaVersion := "2.12.8"

libraryDependencies ++= Seq(
  "org.typelevel" %% "cats-core" % "1.5.0",
  "org.typelevel" %% "cats-effect" % "1.1.0",
  "org.http4s" %% "http4s-blaze-client" % "0.20.0-M5"
)

[RafalSumislawski]

The fix for typelevel/cats-effect#499 is in master, but there was no release since it was merged. I’ll take care of removing the workaround code once a fixed casts-effect version is available.

[anilkumarmyla]

There is a new version 1.3.0 of cats-effect that was released today!