Adaverse commented on issue #31811:
URL: https://github.com/apache/airflow/issues/31811#issuecomment-1600337711

   > Then what is the purpose of the extra field called Blob Storage Shared 
Access Key(Optional)?
   
   To answer above question, we neeed to look into the signature of 
`BlobServiceClient` - 
   `BlobServiceClient(account_url: str, credential: str | Dict[str, str] | 
AzureNamedKeyCredential | AzureSasCredential | TokenCredential | None = None, 
**kwargs: Any)`
   
   We can see account_url is required but credential is optional. The reason 
why credentail is optional because the account_url can itself contain the 
credential (example SAS). Since you are filling the `shared access key` field, 
it expects the `account_url` field to be filled as well of the form 
`https://{account_name}.blob.core.windows.net` and since this field is blank in 
your case hence the error. The `Blob Storage Login` is completly ignored in 
this case.
   
   The first case works just fine because URL is being built 
[here](https://github.com/apache/airflow/blob/fe7a1aa0962538a99f5e73dbd3890a6f210d22ed/airflow/providers/microsoft/azure/hooks/wasb.py#L217)
 using `Blob Storage Login`
   
   Will create a PR to handle this case. But I hope it clears the doubt here.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to