[Storage][Bug]request fail when metadata has trailing space (#11677)

Azure · Jun 2, 2020 · 379ae6e · 379ae6e
1 parent 63497ae
commit 379ae6e
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/sdk/storage/azure-storage-blob/azure/storage/blob/_shared/request_handlers.py b/sdk/storage/azure-storage-blob/azure/storage/blob/_shared/request_handlers.py
@@ -143,5 +143,5 @@ def add_metadata_headers(metadata=None):
     headers = {}
     if metadata:
         for key, value in metadata.items():
-            headers['x-ms-meta-{}'.format(key)] = value
+            headers['x-ms-meta-{}'.format(key.strip())] = value.strip() if value else value
     return headers
diff --git a/sdk/storage/azure-storage-blob/tests/test_common_blob.py b/sdk/storage/azure-storage-blob/tests/test_common_blob.py
@@ -587,7 +587,7 @@ def test_set_blob_metadata_with_upper_case(self, resource_group, location, stora
         # bug in devtools...converts upper case header to lowercase
         # passes live.
         self._setup(storage_account, storage_account_key)
-        metadata = {'hello': 'world', 'number': '42', 'UP': 'UPval'}
+        metadata = {'hello': ' world ', ' number ': '42', 'UP': 'UPval'}
         blob_name = self._create_block_blob()
 
         # Act

diff --git a/sdk/storage/azure-storage-blob/tests/test_common_blob_async.py b/sdk/storage/azure-storage-blob/tests/test_common_blob_async.py
@@ -692,7 +692,7 @@ async def test_set_blob_metadata_with_upper_case(self, resource_group, location,
         # passes live.
         # Arrange
         await self._setup(storage_account, storage_account_key)
-        metadata = {'hello': 'world', 'number': '42', 'UP': 'UPval'}
+        metadata = {'hello': ' world ', ' number ': '42', 'UP': 'UPval'}
         blob_name = await self._create_block_blob()
 
         # Act

diff --git a/...torage/azure-storage-file-datalake/azure/storage/filedatalake/_shared/request_handlers.py b/...torage/azure-storage-file-datalake/azure/storage/filedatalake/_shared/request_handlers.py
@@ -143,5 +143,5 @@ def add_metadata_headers(metadata=None):
     headers = {}
     if metadata:
         for key, value in metadata.items():
-            headers['x-ms-meta-{}'.format(key)] = value
+            headers['x-ms-meta-{}'.format(key.strip())] = value.strip() if value else value
     return headers
diff --git a/sdk/storage/azure-storage-file-share/azure/storage/fileshare/_shared/request_handlers.py b/sdk/storage/azure-storage-file-share/azure/storage/fileshare/_shared/request_handlers.py
@@ -143,5 +143,5 @@ def add_metadata_headers(metadata=None):
     headers = {}
     if metadata:
         for key, value in metadata.items():
-            headers['x-ms-meta-{}'.format(key)] = value
+            headers['x-ms-meta-{}'.format(key.strip())] = value.strip() if value else value
     return headers
diff --git a/sdk/storage/azure-storage-queue/azure/storage/queue/_shared/request_handlers.py b/sdk/storage/azure-storage-queue/azure/storage/queue/_shared/request_handlers.py
@@ -143,5 +143,5 @@ def add_metadata_headers(metadata=None):
     headers = {}
     if metadata:
         for key, value in metadata.items():
-            headers['x-ms-meta-{}'.format(key)] = value
+            headers['x-ms-meta-{}'.format(key.strip())] = value.strip() if value else value
     return headers