removed subscription_id from azure_keyvault_secret lookup when using az cli auth

[msl0]

Added support for Azure CLI authentication without subscription ID in azure_keyvault_secret lookup plugin, fixes #1174

[Fred-sun]

@msl0 I recommend adding support for "az login" credentials, as there are users using this plugin even through az login! Thanks!

[msl0]

@Fred-sun support for "az login" is included. DefaultAzureCredential() supports multiple authentication scenarios. It can use credentials from environment variables and works with managed identity, identity currently logged in to the Azure CLI and PowerShell

[Fred-sun]

@msl0 Ok, I'll test it out as soon as possible! Thanks!

[Fred-sun]

@msl0 There is a conflict in the file you submitted. Can you fix it?

[msl0]

@Fred-sun fixed and then tested once again

[msl0]

@Fred-sun did you have a chance to take a look at my PR?

[Fred-sun]

@msl0 Being tested. Thank you!

[Fred-sun]

@msl0 Everything else is fine, But there's one sanity test that always has a problem, you can't find the cause. Can you adjust the location of 'import logging'? Avoid this mistake. Please change line 121 below line 118. Thank you!

ERROR: plugins/lookup/azure_keyvault_secret.py:134:0: traceback: NameError: name 'logging' is not defined

[msl0]

@Fred-sun Thank you! I changed the import order if that's what you meant. But I don't fully understand the problem. I didn't notice anything when testing the change locally so maybe there is some problem with the test