Revert "Fix access control tab fetching for remote cluster (opensearc…

…h-project#7456)" This reverts commit d152ced.
ananzh · Jul 24, 2024 · 4429721 · 4429721
1 parent 9b2f264
commit 4429721
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/.../components/direct_query_data_sources_components/connection_detail/access_control_tab.tsx b/.../components/direct_query_data_sources_components/connection_detail/access_control_tab.tsx
@@ -25,7 +25,6 @@ interface AccessControlTabProps {
   connector: string;
   properties: unknown;
   allowedRoles: string[];
-  dataSourceMDSId: string;
 }
 
 export const AccessControlTab = (props: AccessControlTabProps) => {
@@ -35,17 +34,17 @@ export const AccessControlTab = (props: AccessControlTabProps) => {
   const { http } = useOpenSearchDashboards<DataSourceManagementContext>().services;
 
   useEffect(() => {
-    http
-      .get(SECURITY_ROLES, { query: { dataSourceId: props.dataSourceMDSId } })
-      .then((data) => {
+    http!
+      .get(SECURITY_ROLES)
+      .then((data) =>
         setRoles(
           Object.keys(data.data).map((key) => {
             return { label: key };
           })
-        );
-      })
+        )
+      )
       .catch((err) => setHasSecurityAccess(false));
-  }, [http, props.dataSourceMDSId]);
+  }, [http]);
 
   const [selectedQueryPermissionRoles, setSelectedQueryPermissionRoles] = useState<Role[]>(
     props.allowedRoles.map((role) => {

diff --git a/...direct_query_data_sources_components/connection_detail/direct_query_connection_detail.tsx b/...direct_query_data_sources_components/connection_detail/direct_query_connection_detail.tsx
@@ -397,7 +397,6 @@ export const DirectQueryDataConnectionDetail: React.FC<DirectQueryDataConnection
           properties={datasourceDetails.properties}
           allowedRoles={datasourceDetails.allowedRoles}
           key={JSON.stringify(datasourceDetails.allowedRoles)}
-          dataSourceMDSId={featureFlagStatus ? dataSourceMDSId ?? '' : ''}
         />
       ),
     },