From 9c39d064167d742133b90ec1729eea8dda9b638f Mon Sep 17 00:00:00 2001
From: Roxanne Skelly <roxanne@highfidelity.io>
Date: Thu, 23 May 2019 11:42:51 -0700
Subject: [PATCH] Don't set permissions / is authenticated when
 DomainServerList checkin should be ignored.

---
 libraries/networking/src/NodeList.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libraries/networking/src/NodeList.cpp b/libraries/networking/src/NodeList.cpp
index 31576ff8d9..e269499938 100644
--- a/libraries/networking/src/NodeList.cpp
+++ b/libraries/networking/src/NodeList.cpp
@@ -641,11 +641,9 @@ void NodeList::processDomainServerList(QSharedPointer<ReceivedMessage> message)
     // pull the permissions/right/privileges for this node out of the stream
     NodePermissions newPermissions;
     packetStream >> newPermissions;
-    setPermissions(newPermissions);
     // Is packet authentication enabled?
     bool isAuthenticated;
     packetStream >> isAuthenticated;
-    setAuthenticatePackets(isAuthenticated);
 
     quint64 connectRequestTimestamp;
     quint64 now = usecTimestampNow();
@@ -689,6 +687,9 @@ void NodeList::processDomainServerList(QSharedPointer<ReceivedMessage> message)
         return;
     }
 
+    setPermissions(newPermissions);
+    setAuthenticatePackets(isAuthenticated);
+
     // when connected, if the session ID or local ID were not null and changed, we should reset
     auto currentLocalID = getSessionLocalID();
     auto currentSessionID = getSessionUUID();