Fixes adding additional QueryParameters to http request (#838)

* Fixes adding aditional QueryParameters to http request * Update tests * Fix formatting * Undo change * Remove unused code
minio · Aug 23, 2023 · af5e161 · af5e161
1 parent 59865de
commit af5e161
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/Minio/HttpRequestMessageBuilder.cs b/Minio/HttpRequestMessageBuilder.cs
@@ -18,7 +18,6 @@
 using System.Globalization;
 using System.Net.Http.Headers;
 using System.Text;
-using System.Web;
 using Minio.Exceptions;
 
 namespace Minio;
@@ -61,13 +60,6 @@ public HttpRequestMessage Request
         get
         {
             var requestUriBuilder = new UriBuilder(RequestUri);
-
-            foreach (var queryParameter in QueryParameters)
-            {
-                var query = HttpUtility.ParseQueryString(requestUriBuilder.Query);
-                requestUriBuilder.Query = query.ToString();
-            }
-
             var requestUri = requestUriBuilder.Uri;
             var request = new HttpRequestMessage(Method, requestUri);
 
@@ -95,12 +87,15 @@ public HttpRequestMessage Request
                             }
 
                             break;
+
                         case "content-length":
                             request.Content.Headers.ContentLength = Convert.ToInt32(val, CultureInfo.InvariantCulture);
                             break;
+
                         case "content-md5":
                             request.Content.Headers.ContentMD5 = Convert.FromBase64String(val);
                             break;
+
                         default:
                             var errMessage = "Unsupported signed header: (" + key + ": " + val;
                             throw new UnexpectedMinioException(errMessage);