Add comments to clarify usage of DefaultAzureCredential in credential… (#261)

* Add comments to clarify usage of DefaultAzureCredential in credential.py and __init__.py

* Formattting

* Updating line length of flake for modern displays

* Formatting

Author: srijan55

.flake8

@@ -1,5 +1,5 @@
 [flake8]
-max-line-length = 88
+max-line-length = 120
 extend-ignore = E203, W503
 exclude =
     .git

pccommon/pccommon/credential.py

@@ -24,5 +24,7 @@ def get_token(cls, *scopes: str, **kwargs: Any) -> AccessToken:
     def get_credential(cls) -> DefaultAzureCredential:
         if cls._instance is None:
             with cls._lock:
-                cls._instance = DefaultAzureCredential()
+                cls._instance = (
+                    DefaultAzureCredential()  # CodeQL [SM05139] Automatically switches to managed identity in prod
+                )
         return cls._instance

pcfuncs/ipban/__init__.py

@@ -17,7 +17,9 @@ def main(mytimer: func.TimerRequest) -> None:
         datetime.datetime.utcnow().replace(tzinfo=datetime.timezone.utc).isoformat()
     )
     logger.info("Updating the ip ban list at %s", utc_timestamp)
-    credential: DefaultAzureCredential = DefaultAzureCredential()
+    credential: DefaultAzureCredential = (
+        DefaultAzureCredential()  # CodeQL [SM05139] Used only as servicability module not deployed in production
+    )
     with LogsQueryClient(credential) as logs_query_client:
         with TableServiceClient(
             endpoint=settings.storage_account_url, credential=credential