Fix bug: snapshot SAS shouldn't contain snapshot time query parameter

Author: Jinming-Hu

Microsoft.WindowsAzure.Storage/src/shared_access_signature.cpp

@@ -178,7 +178,6 @@ namespace azure { namespace storage { namespace protocol {
         auto builder = get_sas_token_builder(identifier, policy, signature);
 
         add_query_if_not_empty(builder, uri_query_sas_resource, resource_type, /* do_encoding */ true);
-        add_query_if_not_empty(builder, uri_query_snapshot, snapshot_time, /* do_encoding */ true);
         add_query_if_not_empty(builder, uri_query_sas_cache_control, headers.cache_control(), /* do_encoding */ true);
         add_query_if_not_empty(builder, uri_query_sas_content_type, headers.content_type(), /* do_encoding */ true);
         add_query_if_not_empty(builder, uri_query_sas_content_encoding, headers.content_encoding(), /* do_encoding */ true);

Microsoft.WindowsAzure.Storage/tests/cloud_blob_test.cpp

@@ -132,7 +132,7 @@ void blob_test_base::check_access(const utility::string_t& sas_token, uint8_t pe
     }
 
     azure::storage::cloud_blob_container container(m_container.uri(), credentials);
-    azure::storage::cloud_blob blob = container.get_blob_reference(original_blob.name());
+    azure::storage::cloud_blob blob = container.get_blob_reference(original_blob.name(), original_blob.snapshot_time());
 
     if (permissions & azure::storage::blob_shared_access_policy::permissions::list)
     {
@@ -197,13 +197,16 @@ void blob_test_base::check_access(const utility::string_t& sas_token, uint8_t pe
         CHECK_THROW(blob.download_attributes(azure::storage::access_condition(), azure::storage::blob_request_options(), m_context), azure::storage::storage_exception);
     }
 
-    if (permissions & azure::storage::blob_shared_access_policy::permissions::write)
+    if (!blob.is_snapshot())
     {
-        blob.upload_metadata(azure::storage::access_condition(), azure::storage::blob_request_options(), m_context);
-    }
-    else
-    {
-        CHECK_THROW(blob.upload_metadata(azure::storage::access_condition(), azure::storage::blob_request_options(), m_context), azure::storage::storage_exception);
+        if (permissions & azure::storage::blob_shared_access_policy::permissions::write)
+        {
+            blob.upload_metadata(azure::storage::access_condition(), azure::storage::blob_request_options(), m_context);
+        }
+        else
+        {
+            CHECK_THROW(blob.upload_metadata(azure::storage::access_condition(), azure::storage::blob_request_options(), m_context), azure::storage::storage_exception);
+        }
     }
 
     if (permissions & azure::storage::blob_shared_access_policy::permissions::del)