chore: using URL object to set the api key

usebruno · Jan 17, 2025 · 525394b · 525394b
1 parent d16a679
commit 525394b
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/packages/bruno-cli/src/runner/prepare-request.js b/packages/bruno-cli/src/runner/prepare-request.js
@@ -54,9 +54,14 @@ const prepareRequest = (item = {}, collection = {}) => {
       }
 
       if (collectionAuth.apikey?.placement === 'queryparams') {
-        if (axiosRequest.url) {
-          const prefix = (!axiosRequest.url.includes('?')) ? '?' : '&';
-          axiosRequest.url += `${prefix}${collectionAuth.apikey.key}=${collectionAuth.apikey.value}`;
+        if (axiosRequest.url && collectionAuth.apikey?.key) {
+          try {
+            const urlObj = new URL(request.url);
+            urlObj.searchParams.set(collectionAuth.apikey?.key, collectionAuth.apikey?.value);
+            axiosRequest.url = urlObj.toString();
+          } catch (error) {
+            console.error('Invalid URL:', request.url, error);
+          }
         }
       }
     }