Fix command with query parameter rejected in DevTools

Signed-off-by: Hailong Cui <[email protected]>

CHANGELOG.md

@@ -224,6 +224,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - [BUG] Fix suggestion list cutoff issue ([#2607](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/2607))
 - [TSVB] Fixes undefined serial diff aggregation documentation link ([#3503](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3503))
 - [Console] Fix dev tool console autocomplete not loading issue ([#3775](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3775))
+- [Console] Fix dev tool console run command with query parameter error ([#3813](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3813))
 
 ### 🚞 Infrastructure
 

src/plugins/console/server/routes/api/console/proxy/create_handler.ts

@@ -75,7 +75,11 @@ function toUrlPath(path: string) {
   // in JS can lead to data loss (7.0 will get munged into 7, thus losing indication of
   // measurement precision)
   if (!urlPath.includes('?pretty')) {
-    urlPath += '?pretty=true';
+    if (urlPath.includes('?')) {
+      urlPath += '&pretty=true';
+    } else {
+      urlPath += '?pretty=true';
+    }
   }
   return urlPath;
 }

src/plugins/console/server/routes/api/console/proxy/tests/query_string.test.ts

@@ -82,6 +82,14 @@ describe('Console Proxy Route', () => {
           expect(args.path).toBe('/index/id?pretty=true');
         });
       });
+
+      describe(`contains query parameter`, () => {
+        it('adds slash to path before sending request', async () => {
+          await request('GET', '_cat/tasks?v');
+          const [[args]] = opensearchClient.asCurrentUser.transport.request.mock.calls;
+          expect(args.path).toBe('/_cat/tasks?v&pretty=true');
+        });
+      });
     });
   });
 });