reset feature switch

Signed-off-by: Su <szhongna@amazon.com>

config/opensearch_dashboards.yml

@@ -231,7 +231,7 @@
 
 # Set the value of this setting to true to enable the experimental multiple data source
 # support feature. Use with caution.
-data_source.enabled: true
+#data_source.enabled: false
 # Set the value of these settings to customize crypto materials to encryption saved credentials
 # in data sources.
 #data_source.encryption.wrappingKeyName: 'changeme'

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

@@ -67,12 +67,12 @@ function getProxyHeaders(req: OpenSearchDashboardsRequest) {
 }
 
 function toUrlPath(path: string) {
-  const urlPath = `/${trimStart(path, '/')}`;
+  let urlPath = `/${trimStart(path, '/')}`;
   // Appending pretty here to have OpenSearch do the JSON formatting, as doing
   // 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.concat('?pretty=true');
+    urlPath += '?pretty=true';
   }
   return urlPath;
 }

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

@@ -64,21 +64,21 @@ describe('Console Proxy Route', () => {
         it('treats the url as a path', async () => {
           await request('GET', 'http://evil.com/test');
           const [[args]] = opensearchClient.asCurrentUser.transport.request.mock.calls;
-          expect(args.path).toBe('http://evil.com/test?pretty=true');
+          expect(args.path).toBe('/http://evil.com/test?pretty=true');
         });
       });
       describe('starts with a slash', () => {
-        it('combines well with the base url', async () => {
+        it('keeps as it is', async () => {
           await request('GET', '/index/id');
           const [[args]] = opensearchClient.asCurrentUser.transport.request.mock.calls;
           expect(args.path).toBe('/index/id?pretty=true');
         });
       });
       describe(`doesn't start with a slash`, () => {
-        it('combines well with the base url', async () => {
+        it('adds slash to path before sending request', async () => {
           await request('GET', 'index/id');
           const [[args]] = opensearchClient.asCurrentUser.transport.request.mock.calls;
-          expect(args.path).toBe('index/id?pretty=true');
+          expect(args.path).toBe('/index/id?pretty=true');
         });
       });
     });