Fix breaking issues in Auth and Browshot (#499)

* Fixed broken URL string creation in browshot feature

* Rolling back change to skip env auth if no keys set - breaks KV setup

* Allowing skip of env auth method whilst fixing breaking issue

* Return empty list to ChainedTokenCredential

* Liniting fix

Author: petebryan

msticpy/auth/azure_auth_core.py

@@ -302,8 +302,7 @@ def _build_chained_creds(
         raise MsticpyAzureConfigError(
             "At least one valid authentication method required."
         )
-
-    return ChainedTokenCredential(*[client for client in clients if client])
+    return ChainedTokenCredential([client for client in clients if client])  # type: ignore
 
 
 class _AzCachedConnect:

msticpy/context/domain_utils.py

@@ -94,7 +94,7 @@ def screenshot(url: str, api_key: str = None) -> httpx.Response:
     )
     image_string = (
         f"https://api.browshot.com/api/v1/screenshot/thumbnail?id={bs_id}"
-        "&zoom=50&key={bs_api_key}"
+        f"&zoom=50&key={bs_api_key}"
     )
     # Wait until the screenshot is ready and keep user updated with progress
     print("Getting screenshot")

requirements-all.txt

@@ -10,7 +10,7 @@ azure-mgmt-monitor>=2.0.0
 azure-mgmt-network>=2.7.0
 azure-mgmt-resource>=16.1.0
 azure-mgmt-resourcegraph>=8.0.0
-azure-mgmt-subscription>=1.0.0
+azure-mgmt-subscription>=3.0.0
 azure-storage-blob>=12.5.0
 bokeh>=1.4.0
 cryptography>=3.1