(cloudstack) branch main updated: Make authentication request parameter order to be deterministic (#8185)

[dahn]

This is an automated email from the ASF dual-hosted git repository.

dahn pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/main by this push:
     new e9b24b6c326 Make authentication request parameter order to be 
deterministic (#8185)
e9b24b6c326 is described below

commit e9b24b6c3266f14a85f2db86fb5d824539206097
Author: rRajivramachandran <rrajiv9...@gmail.com>
AuthorDate: Mon Nov 6 02:53:49 2023 -0600

    Make authentication request parameter order to be deterministic (#8185)
---
 .../src/main/java/com/cloud/network/nicira/NiciraRestClient.java      | 4 ++--
 .../src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java  | 4 ++--
 utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java   | 3 ++-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git 
a/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
 
b/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
index a98480b11b3..aa428b08dc3 100644
--- 
a/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
+++ 
b/plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/nicira/NiciraRestClient.java
@@ -20,7 +20,7 @@
 package com.cloud.network.nicira;
 
 import java.io.IOException;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 import org.apache.http.HttpEntity;
@@ -120,7 +120,7 @@ public class NiciraRestClient extends BasicRestClient {
     }
 
     private HttpUriRequest createAuthenticationRequest() {
-        final Map<String, String> parameters = new HashMap<>();
+        final Map<String, String> parameters = new LinkedHashMap<>();
         parameters.put("username", username);
         parameters.put("password", password);
         return HttpUriRequestBuilder.create()
diff --git 
a/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
 
b/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
index 75b751c8ae1..bae11c7bfa2 100644
--- 
a/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
+++ 
b/plugins/network-elements/nicira-nvp/src/test/java/com/cloud/network/nicira/NiciraRestClientTest.java
@@ -32,7 +32,7 @@ import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 import org.apache.http.HttpHost;
@@ -70,7 +70,7 @@ public class NiciraRestClientTest {
     private static final StatusLine HTTP_200_STATUSLINE = new 
BasicStatusLine(new ProtocolVersion(HTTPS, 1, 1), 200, "OK");
     private static final StatusLine HTTP_401_STATUSLINE = new 
BasicStatusLine(new ProtocolVersion(HTTPS, 1, 1), 401, "Unauthorized");
 
-    private static final Map<String, String> loginParameters = new 
HashMap<String, String>();
+    private static final Map<String, String> loginParameters = new 
LinkedHashMap<String, String>();
     private static HttpUriRequest request;
     private static HttpUriRequest loginRequest;
     private final CloseableHttpClient httpClient = 
mock(CloseableHttpClient.class);
diff --git 
a/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java 
b/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
index 47ac06af111..52039a92ede 100644
--- a/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
+++ b/utils/src/main/java/com/cloud/utils/rest/HttpUriRequestBuilder.java
@@ -22,6 +22,7 @@ package com.cloud.utils.rest;
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Map.Entry;
 
@@ -47,7 +48,7 @@ public class HttpUriRequestBuilder {
     private String path;
     private Optional<String> jsonPayload = ABSENT;
     private final Map<String, String> parameters = new HashMap<String, 
String>();
-    private final Map<String, String> methodParameters = new HashMap<String, 
String>();
+    private final Map<String, String> methodParameters = new 
LinkedHashMap<String, String>();
 
     private HttpUriRequestBuilder() {