Fix Possible Race Condition

[scottweaver]

fixes #3268

Refactor 'refreshIdleTimeoutHandler' so that 'ChannelPipeline.replace' does not crash the fiber on unhandled exceptions.

[kyri-petrou]

This "breaks" the current logic for retrying channels from the pool until we succeed. I think a better solution would be to keep the existing code as is and wrap it in a try-catch block so that we don't fail requests when this happens

  private def refreshIdleTimeoutHandler(
    channel: JChannel,
    timeout: Duration,
  ): Boolean = {
  try {
    channel
      .pipeline()
      .replace(
        Names.ReadTimeoutHandler,
        Names.ReadTimeoutHandler,
        new ReadTimeoutHandler(timeout.toMillis, TimeUnit.MILLISECONDS),
      )
    channel.isOpen
  } catch {
    case _: NoSuchElementException => false
  }