From 30d5d5a8921d1740ac0b981c0c6e32292a437e39 Mon Sep 17 00:00:00 2001 From: Adam Ierymenko Date: Mon, 20 Sep 2021 22:05:49 -0400 Subject: [PATCH] Revert "Don't assume roots validated the identity, just in case they did not." This reverts commit 39b97f91633dd8dca4a1c0834d49b7f172e9b935. --- node/IncomingPacket.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/node/IncomingPacket.cpp b/node/IncomingPacket.cpp index ae6c1a849..5a2a94642 100644 --- a/node/IncomingPacket.cpp +++ b/node/IncomingPacket.cpp @@ -511,10 +511,7 @@ bool IncomingPacket::_doOK(const RuntimeEnvironment *RR,void *tPtr,const SharedP case Packet::VERB_WHOIS: if (RR->topology->isUpstream(peer->identity())) { const Identity id(*this,ZT_PROTO_VERB_WHOIS__OK__IDX_IDENTITY); - // Good idea to locally validate here even if roots are doing so. In a truly distributed - // system there should not be single points of failure for global trust assertions. - if (id.locallyValidate()) - RR->sw->doAnythingWaitingForPeer(tPtr,RR->topology->addPeer(tPtr,SharedPtr(new Peer(RR,RR->identity,id)))); + RR->sw->doAnythingWaitingForPeer(tPtr,RR->topology->addPeer(tPtr,SharedPtr(new Peer(RR,RR->identity,id)))); } break;