fix: add a CE whitelist for remote logging (fix None type)

martynia · Jan 23, 2024 · 61e640e · 61e640e
1 parent 9a1fb3c
commit 61e640e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/Pilot/pilotTools.py b/Pilot/pilotTools.py
@@ -1224,12 +1224,13 @@ def __initJSON2(self):
         # logger CE white list
         loggerCEsWhiteList = pilotOptions.get("RemoteLoggerCEsWhiteList", None)
         # restrict remote logging to a whitelisted CEs ([] => no restriction)
-        if not isinstance(loggerCEsWhiteList, list):
-            loggerCEsWhiteList = [elem.strip() for elem in loggerCEsWhiteList.split(",")]
         self.log.debug("JSON: Remote logging CE white list: %s" % loggerCEsWhiteList)
-        if loggerCEsWhiteList is not None and self.ceName not in loggerCEsWhiteList:
-            self.pilotLogging = False
-            self.log.debug("JSON: Remote logging disabled for this CE: %s" % self.ceName)
+        if loggerCEsWhiteList is not None:
+            if not isinstance(loggerCEsWhiteList, list):
+                loggerCEsWhiteList = [elem.strip() for elem in loggerCEsWhiteList.split(",")]
+            if self.ceName not in loggerCEsWhiteList:
+                self.pilotLogging = False
+                self.log.debug("JSON: Remote logging disabled for this CE: %s" % self.ceName)
         pilotLogLevel = pilotOptions.get("PilotLogLevel", "INFO")
         if pilotLogLevel.lower() == "debug":
             self.debugFlag = True