Set okta_api_key to None when not configured. Corrected the log for C…

…RXcavator (#293)
cartography-cncf · Apr 29, 2020 · 68abae2 · 68abae2
1 parent 970ab87
commit 68abae2
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/cartography/cli.py b/cartography/cli.py
@@ -251,10 +251,12 @@ def main(self, argv):
         if config.okta_org_id and config.okta_api_key_env_var:
             logger.debug(f"Reading API key for Okta from environment variable {config.okta_api_key_env_var}")
             config.okta_api_key = os.environ.get(config.okta_api_key_env_var)
+        else:
+            config.okta_api_key = None
 
         # CRXcavator config
         if config.crxcavator_api_base_uri and config.crxcavator_api_key_env_var:
-            logger.debug("Reading API key for CRXcavator from environment variable '%s'.")
+            logger.debug(f"Reading API key for CRXcavator from env variable {config.crxcavator_api_key_env_var}.")
             config.crxcavator_api_key = os.environ.get(config.crxcavator_api_key_env_var)
         else:
             config.crxcavator_api_key = None