Ignore AppResponse timeouts for benching (#2066)

ava-labs · Sep 22, 2023 · aad3067 · aad3067
1 parent e8c32b3
commit aad3067
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/snow/networking/timeout/manager.go b/snow/networking/timeout/manager.go
@@ -125,8 +125,11 @@ func (m *manager) RegisterRequest(
 	timeoutHandler func(),
 ) {
 	newTimeoutHandler := func() {
-		// If this request timed out, tell the benchlist manager
-		m.benchlistMgr.RegisterFailure(chainID, nodeID)
+		if requestID.Op != byte(message.AppResponseOp) {
+			// If the request timed out and wasn't an AppRequest, tell the
+			// benchlist manager.
+			m.benchlistMgr.RegisterFailure(chainID, nodeID)
+		}
 		timeoutHandler()
 	}
 	m.tm.Put(requestID, measureLatency, newTimeoutHandler)