[v1.0-rc/bug] Bug on the abs source type. Error converting ClientSecretCredential into a string.

[fmoreind]

Describe the bug
In the Azure Blob Storage source type, when using the Azure connection configuration with a Service entity (tenant_id, client_id and client_secret) instead of SAS_token, an error is generated when converting the ClientSecretCredential class to string type when it should be an object. See also BlobServiceClient class

Since the ClientSecretCredential is class type object, imported from Azure SDK, this cannot be converted into a string.

See get_credentials and get_blob_service_client functions in azure_common.

To Reproduce
Steps to reproduce the behavior:

1. Generate a .yaml recipe type abs with azure_config.client_secret. Azure App Registry should be configured as documentation.
2. Execute the recipe.yml file in VSCode or cloud.
3. See error

Expected behavior
The expected behaviour is the same succeded message as the azure_config.token_sas.

In fact, if I change the azure_common.py replacing:

    def get_blob_service_client(self):
        return BlobServiceClient(
            account_url=f"https://{self.account_name}.blob.core.windows.net",
            credential=f"{self.get_credentials()}",
        )

For this:

    def get_blob_service_client(self):
        return BlobServiceClient(
            account_url=f"https://{self.account_name}.blob.core.windows.net",
            credential=self.get_credentials(),
        )

The behaviour is shown in the below screenshots.

Screenshots and/or Screen Recordings
Fail message:

Success meassage after change get_blob_service_client function:

System details (please complete the following information):

• DataHub Version Tag: v1.0-rc1
• OS: iOS
• Browser: chrome

Additional context
I've been comparing with oldest versions and I founf out e.g.v0.9.6.1 and it's not been converted into a string.

[tonysanchez64]

I have the same issue, thank you for bringing it up.