[Storage] [Named Keywords] Blob Package

sdk/storage/azure-storage-blob/azure/storage/blob/__init__.py

@@ -126,7 +126,7 @@ def upload_blob_to_url(
     :rtype: dict(str, Any)
     """
     with BlobClient.from_blob_url(blob_url, credential=credential) as client:
-        return cast(BlobClient, client).upload_blob(data=data, blob_type=BlobType.BLOCKBLOB, **kwargs)
+        return client.upload_blob(data=data, blob_type=BlobType.BLOCKBLOB, **kwargs)
 
 
 def _download_to_stream(client: BlobClient, handle: IO[bytes], **kwargs: Any) -> None:

sdk/storage/azure-storage-blob/azure/storage/blob/_blob_client.py

@@ -6,13 +6,13 @@
 # pylint: disable=too-many-lines, docstring-keyword-should-match-keyword-only
 
 import warnings
+from contextlib import AbstractContextManager
 from datetime import datetime
 from functools import partial
 from typing import (
-    Any, AnyStr, cast, Dict, IO, Iterable, List, Optional, overload, Tuple, Union,
+    Any, AnyStr, cast, Dict, IO, Iterable, List, Optional, Tuple, Union,
     TYPE_CHECKING
 )
-from typing_extensions import Self
 
 from azure.core.exceptions import HttpResponseError, ResourceExistsError, ResourceNotFoundError
 from azure.core.paging import ItemPaged
@@ -89,7 +89,7 @@
     )
 
 
-class BlobClient(StorageAccountHostsMixin, StorageEncryptionMixin):  # pylint: disable=too-many-public-methods
+class BlobClient(AbstractContextManager, StorageAccountHostsMixin, StorageEncryptionMixin):  # pylint: disable=too-many-public-methods
     """A client to interact with a specific blob, although that blob may not yet exist.
 
     For more optional configuration, please click
@@ -190,7 +190,7 @@ def __init__(
         self._client._config.version = get_api_version(kwargs)  # type: ignore [assignment]
         self._configure_encryption(kwargs)
 
-    def __enter__(self) -> Self:
+    def __enter__(self) -> "BlobClient":
         self._client.__enter__()
         return self
 
@@ -221,7 +221,7 @@ def from_blob_url(
         credential: Optional[Union[str, Dict[str, str], "AzureNamedKeyCredential", "AzureSasCredential", "TokenCredential"]] = None,  # pylint: disable=line-too-long
         snapshot: Optional[Union[str, Dict[str, Any]]] = None,
         **kwargs: Any
-    ) -> Self:
+    ) -> "BlobClient":
         """Create BlobClient from a blob url. This doesn't support customized blob url with '/' in blob name.
 
         :param str blob_url:
@@ -269,7 +269,7 @@ def from_connection_string(
         snapshot: Optional[Union[str, Dict[str, Any]]] = None,
         credential: Optional[Union[str, Dict[str, str], "AzureNamedKeyCredential", "AzureSasCredential", "TokenCredential"]] = None,  # pylint: disable=line-too-long
         **kwargs: Any
-    ) -> Self:
+    ) -> "BlobClient":
         """Create BlobClient from a Connection String.
 
         :param str conn_str:
@@ -633,26 +633,6 @@ def upload_blob(
             return upload_page_blob(**options)
         return upload_append_blob(**options)
 
-    @overload
-    def download_blob(
-        self, offset: Optional[int] = None,
-        length: Optional[int] = None,
-        *,
-        encoding: str,
-        **kwargs: Any
-    ) -> StorageStreamDownloader[str]:
-        ...
-
-    @overload
-    def download_blob(
-        self, offset: Optional[int] = None,
-        length: Optional[int] = None,
-        *,
-        encoding: None = None,
-        **kwargs: Any
-    ) -> StorageStreamDownloader[bytes]:
-        ...
-
     @distributed_trace
     def download_blob(
         self, offset: Optional[int] = None,