Fix escaping issue

Author: JoeDupuis

lib/azure_blob/client.rb

@@ -194,7 +194,8 @@ def blob_exist?(key, options = {})
     #
     # Returns a hash of the blob's tags.
     def get_blob_tags(key)
-      uri = generate_uri("#{container}/#{key}?comp=tags")
+      uri = generate_uri("#{container}/#{key}")
+      uri.query = URI.encode_www_form(comp: "tags")
       response = Http.new(uri, signer:).get
 
       Tags.from_response(response).to_h
@@ -246,7 +247,8 @@ def delete_container(options = {})
     #
     # Example: +generate_uri("#{container}/#{key}")+
     def generate_uri(path)
-      URI.parse(URI::RFC2396_PARSER.escape(File.join(host, path)))
+      encoded = path.split("/").map { |segment| CGI.escape(segment) }.join("/")
+      URI.parse([host.chomp("/"), encoded].join("/"))
     end
 
     # Returns an SAS signed URI

test/client/test_client.rb

@@ -23,7 +23,7 @@ def setup
       host: @host,
     )
     @uid = SecureRandom.uuid
-    @key = "test-client-#{name}-#{@uid}"
+    @key = "test-client-?-#{name}-#{@uid}" # ? in key to test proper escaping
     @content = "Some random content #{Random.rand(200)}"
   end