[App Configuration] - Update query param policy (#36344)

### Packages impacted by this PR

@azure/app-configuration

### Issues associated with this PR

See
[comment](#35962 (comment))

### Describe the problem that is addressed by this PR


### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?


### Are there test cases added in this PR? _(If not, why?)_


### Provide a list of related PRs _(if any)_


### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_

### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)

---------

Co-authored-by: Minh-Anh Phan <[email protected]>

Author: zhiyuanliang-ms

sdk/appconfiguration/app-configuration/src/internal/queryParamPolicy.ts

@@ -7,6 +7,7 @@ import type {
   PipelineResponse,
   SendRequest,
 } from "@azure/core-rest-pipeline";
+import { logger } from "../logger.js";
 
 /**
  * Creates a PipelinePolicy that normalizes query parameters:
@@ -18,47 +19,51 @@ export function queryParamPolicy(): PipelinePolicy {
   return {
     name: "queryParamPolicy",
     async sendRequest(request: PipelineRequest, next: SendRequest): Promise<PipelineResponse> {
+      const originalUrl: string = request.url;
+      let url: URL;
       try {
-        const originalUrl: string = request.url;
-        const url = new URL(originalUrl);
-
-        if (url.search === "") {
+        url = new URL(originalUrl);
+      } catch (error) {
+        if (error instanceof TypeError) {
+          logger.warning(`"[queryParamPolicy] Could not parse URL: ${request.url}"`);
           return next(request);
         }
+        throw error;
+      }
+
+      if (url.search === "") {
+        return next(request);
+      }
 
-        const params: ParamEntry[] = [];
-        for (const entry of url.search.substring(1).split("&")) {
-          if (entry === "") {
-            continue;
-          }
-          const equalIndex = entry.indexOf("=");
-          const name = equalIndex === -1 ? entry : entry.substring(0, equalIndex);
-          const value = equalIndex === -1 ? "" : entry.substring(equalIndex + 1);
-          params.push({ lowercaseName: name.toLowerCase(), value });
+      const params: ParamEntry[] = [];
+      for (const entry of url.search.substring(1).split("&")) {
+        if (entry === "") {
+          continue;
         }
+        const equalIndex = entry.indexOf("=");
+        const name = equalIndex === -1 ? entry : entry.substring(0, equalIndex);
+        const value = equalIndex === -1 ? "" : entry.substring(equalIndex + 1);
+        params.push({ lowercaseName: name.toLowerCase(), value });
+      }
 
-        // Modern JavaScript Array.prototype.sort is stable
-        // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#sort_stability
-        params.sort((a, b) => {
-          if (a.lowercaseName < b.lowercaseName) {
-            return -1;
-          } else if (a.lowercaseName > b.lowercaseName) {
-            return 1;
-          }
-          return 0;
-        });
+      // Modern JavaScript Array.prototype.sort is stable
+      // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#sort_stability
+      params.sort((a, b) => {
+        if (a.lowercaseName < b.lowercaseName) {
+          return -1;
+        } else if (a.lowercaseName > b.lowercaseName) {
+          return 1;
+        }
+        return 0;
+      });
 
-        const newSearchParams: string = params
-          .map(({ lowercaseName, value }) => `${lowercaseName}=${value}`)
-          .join("&");
+      const newSearchParams: string = params
+        .map(({ lowercaseName, value }) => `${lowercaseName}=${value}`)
+        .join("&");
 
-        const newUrl = url.origin + url.pathname + "?" + newSearchParams + url.hash;
-        if (newUrl !== originalUrl) {
-          request.url = newUrl;
-        }
-      } catch {
-        // If anything goes wrong, fall back to sending the original request.
-        console.log("Failed to normalize query parameters.");
+      const newUrl = url.origin + url.pathname + "?" + newSearchParams + url.hash;
+      if (newUrl !== originalUrl) {
+        request.url = newUrl;
       }
 
       return next(request);

sdk/appconfiguration/app-configuration/src/internal/syncTokenPolicy.ts

@@ -29,7 +29,7 @@ export function syncTokenPolicy(syncTokens: SyncTokens): PipelinePolicy {
 
       if (syncTokenHeaderValue) {
         logger.info(
-          "[syncTokenPolicy] Setting headers with ${SyncTokenHeaderName} and ${syncTokenHeaderValue}",
+          `[syncTokenPolicy] Setting headers with ${SyncTokenHeaderName} and ${syncTokenHeaderValue}`,
         );
         request.headers.set(SyncTokenHeaderName, syncTokenHeaderValue);
       }