Revert "Fix error handling for user w/o proper permissions (opensearc…

…h-project#195) (opensearch-project#197)" This reverts commit 26ed0dc.
mengweieric · Nov 8, 2023 · 5919996 · 5919996
1 parent 4e26e8d
commit 5919996
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/public/components/Main/__snapshots__/main.test.tsx.snap b/public/components/Main/__snapshots__/main.test.tsx.snap
@@ -2381,7 +2381,7 @@ exports[`<Main /> spec click run button, and response causes an error 1`] = `
                       class="euiText euiText--medium"
                     >
                       <p>
-                        Error in loading OpenSearch indices
+                        err
                       </p>
                     </div>
                   </span>

diff --git a/public/components/SQLPage/table_view.tsx b/public/components/SQLPage/table_view.tsx
@@ -112,7 +112,7 @@ export const TableView = ({ http, selectedItems, updateSQLQueries, refreshTree }
           body: JSON.stringify(query),
         })
         .then((res) => {
-          const responseObj = JSON.parse(res.data.resp);
+          const responseObj = res.data.resp ? JSON.parse(res.data.resp) : {};
           const dataRows: any[][] = _.get(responseObj, 'datarows');
           if (dataRows.length > 0) {
             const fields = dataRows.map((data) => {
@@ -131,9 +131,9 @@ export const TableView = ({ http, selectedItems, updateSQLQueries, refreshTree }
           console.error(err);
           setIsLoading({
             flag: false,
-            status: 'Error in loading OpenSearch indices',
+            status: err,
           });
-          setToast(`Error in loading OpenSearch indices, please check user permissions`, 'danger');
+          setToast(`ERROR ${err}`, 'danger');
         });
     } else {
       setTableNames([]);
@@ -536,7 +536,7 @@ export const TableView = ({ http, selectedItems, updateSQLQueries, refreshTree }
                     {node.type === TREE_ITEM_TABLE_NAME_DEFAULT_NAME && !node.isLoading && (
                       <EuiIcon
                         type="editorCodeBlock"
-                        onClick={(e) => handleQuery(e, parentName, node.name)}
+                        onClick={(e) => handleQuery(e,parentName,node.name)}
                       ></EuiIcon>
                     )}
                   </EuiText>
-Original file line number
+Diff line change
@@ Expand Up @@
                           class="euiText euiText--medium"
                         >
                           <p>
-                            Error in loading OpenSearch indices
+                            err
                           </p>
                         </div>
                       </span>
@@ Expand Down @@