Properly handle array query arguments in Admin Client (#24483)

Closes #20135

js/libs/keycloak-admin-client/src/utils/stringifyQueryParams.ts

@@ -1,21 +1,29 @@
 export function stringifyQueryParams(params: Record<string, unknown>) {
-  return new URLSearchParams(
+  const searchParams = new URLSearchParams();
-    Object.entries(params).filter((param): param is [string, string] => {
-      const [, value] = param;
 
-      if (typeof value === "undefined" || value === null) {
+  for (const [key, value] of Object.entries(params)) {
-        return false;
+    // Ignore undefined and null values.
-      }
+    if (value === undefined || value === null) {
+      continue;
+    }
 
-      if (typeof value === "string" && value.length === 0) {
+    // Ignore empty strings.
-        return false;
+    if (typeof value === "string" && value.length === 0) {
-      }
+      continue;
+    }
 
-      if (Array.isArray(value) && value.length === 0) {
+    // Ignore empty arrays.
-        return false;
+    if (Array.isArray(value) && value.length === 0) {
-      }
+      continue;
+    }
 
-      return true;
+    // Append each entry of an array as a separate parameter, or the value itself otherwise.
-    }),
+    if (Array.isArray(value)) {
-  ).toString();
+      value.forEach((item) => searchParams.append(key, item.toString()));
+    } else {
+      searchParams.append(key, value.toString());
+    }
+  }
+
+  return searchParams.toString();
 }

js/libs/keycloak-admin-client/test/auth.spec.ts

@@ -42,6 +42,6 @@ describe("Authorization", () => {
       "idToken",
     );
 
-    expect(data.scope).to.equal("openid profile email");
+    expect(data.scope).to.equal("openid email profile");
   });
 });

js/libs/keycloak-admin-client/test/authenticationManagement.spec.ts

@@ -176,7 +176,6 @@ describe("Authentication management", () => {
         "clients",
         "first broker login",
         "docker auth",
-        "http challenge",
       ]);
     });
 

js/libs/keycloak-admin-client/test/stringifyQueryParams.spec.ts

@@ -23,9 +23,10 @@ describe("stringifyQueryParams", () => {
         numZero: 0,
         numNegative: -1,
         str: "Hello World!",
+        arr: ["foo", "bar"],
       }),
     ).to.equal(
-      "boolTrue=true&boolFalse=false&numPositive=1&numZero=0&numNegative=-1&str=Hello+World%21",
+      "boolTrue=true&boolFalse=false&numPositive=1&numZero=0&numNegative=-1&str=Hello+World%21&arr=foo&arr=bar",
     );
   });
 });