[Storage] Fix #16872: az storage blob now (2.19) requires login even if connection-string is provided

[Juliehzl]

Description

Fix #16872

Testing Guide

az account clear
az storage blob sync with account key or connection string

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

---

This checklist is used to make sure that common guidelines for a pull request are followed.

• The PR title and description has followed the guideline in Submitting Pull Requests.

• I adhere to the Command Guidelines.

• I adhere to the Error Handling Guidelines.

[yonzhan]

Storage

[evelyn-ys]

Do azcopy.remove have the same issue? I remember in #16519 both remove and sync are modified due to permissions.

azure-cli/src/azure-cli/azure/cli/command_modules/storage/operations/azcopy.py

Lines 57 to 60 in bc4d70c

	sas_token = _generate_sas_token(cmd, client.account_name, client.account_key, service=service,
	resource_types='co',
	permissions='rdl')
	azcopy.remove(_add_url_sas(target, sas_token), flags=flags)

[Juliehzl]

Do azcopy.remove have the same issue? I remember in #16519 both remove and sync are modified due to permissions.

azure-cli/src/azure-cli/azure/cli/command_modules/storage/operations/azcopy.py

Lines 57 to 60 in bc4d70c

	sas_token = _generate_sas_token(cmd, client.account_name, client.account_key, service=service,
	resource_types='co',
	permissions='rdl')
	azcopy.remove(_add_url_sas(target, sas_token), flags=flags)

Good point! updated and added test.