don't ban peers if the block timestamp is in the future (#8327)

near · Jan 17, 2023 · c578db6 · c578db6
1 parent 8a38b29
commit c578db6
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/chain/client/src/client.rs b/chain/client/src/client.rs
@@ -1072,7 +1072,12 @@ impl Client {
                 Ok(())
             }
             Err(e) if e.is_bad_data() => {
-                self.ban_peer(peer_id.clone(), ReasonForBan::BadBlockHeader);
+                // We don't ban a peer if the block timestamp is too much in the future since it's possible
+                // that a block is considered valid in one machine and invalid in another machine when their
+                // clocks are not synced.
+                if !matches!(e, near_chain::Error::InvalidBlockFutureTime(_)) {
+                    self.ban_peer(peer_id.clone(), ReasonForBan::BadBlockHeader);
+                }
                 Err(e)
             }
             Err(_) => {