Remove expectTimeout Await.result() (#4957)

This commit is contained in:
Chris Stewart 2023-01-23 12:30:36 -06:00 committed by GitHub
parent 4e4c6d03f1
commit dfe92d2ba4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -9,8 +9,7 @@ import org.bitcoins.node.networking.P2PClient
import org.bitcoins.node.networking.peer.PeerMessageReceiverState._
import org.bitcoins.node.{Node, P2PLogger}
import scala.concurrent.duration.DurationInt
import scala.concurrent.{Await, Future}
import scala.concurrent.Future
/** Responsible for receiving messages from a peer on the
* p2p network. This is called by [[org.bitcoins.rpc.client.common.Client Client]] when doing the p2p
@ -305,9 +304,14 @@ class PeerMessageReceiver(
case good: Normal =>
logger.debug(s"Handling expected response for ${msg.commandName}")
val expectedResponseCancellable =
system.scheduler.scheduleOnce(nodeAppConfig.queryWaitTime)(
Await.result(node.peerManager.sendResponseTimeout(peer, msg),
10.seconds))
system.scheduler.scheduleOnce(nodeAppConfig.queryWaitTime) {
val responseTimeoutF =
node.peerManager.sendResponseTimeout(peer, msg)
responseTimeoutF.failed.foreach(err =>
logger.error(
s"Failed to timeout waiting for response for peer=$peer",
err))
}
val newState = Waiting(
clientConnectP = good.clientConnectP,
clientDisconnectP = good.clientDisconnectP,