fixed struts message and save persist store

akto-api-security · Dec 13, 2024 · fc400cf · fc400cf
1 parent f2a3cfc
commit fc400cf
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 2 deletions.
diff --git a/apps/dashboard/src/main/java/com/akto/action/ApiCollectionsAction.java b/apps/dashboard/src/main/java/com/akto/action/ApiCollectionsAction.java
@@ -741,7 +741,7 @@ public String editCollectionName() {
         }
 
         if (apiCollection.getHostName() != null) {
-            String errorMessage = "Traffic API collection can't edited";
+            String errorMessage = "Unable to modify the Traffic API collection";
             addActionError(errorMessage);
             return Action.ERROR.toUpperCase();
         }

diff --git a/apps/dashboard/src/main/resources/struts.xml b/apps/dashboard/src/main/resources/struts.xml
@@ -7389,7 +7389,7 @@
             <interceptor-ref name="roleAccessInterceptor">
                 <param name="featureLabel">API_COLLECTIONS</param>
                 <param name="accessType">READ_WRITE</param>
-                <param name="actionDescription">User created an empty collection</param>
+                <param name="actionDescription">User edited collection name</param>
             </interceptor-ref>
             <result name="FORBIDDEN" type="json">
                 <param name="statusCode">403</param>

diff --git a/...ard/web/polaris_web/web/src/apps/dashboard/pages/observe/api_collections/ApiEndpoints.jsx b/...ard/web/polaris_web/web/src/apps/dashboard/pages/observe/api_collections/ApiEndpoints.jsx
@@ -154,6 +154,8 @@ function ApiEndpoints(props) {
     const setShowDetails = ObserveStore(state => state.setInventoryFlyout)
     const collectionsMap = PersistStore(state => state.collectionsMap)
     const allCollections = PersistStore(state => state.allCollections);
+    const setCollectionsMap = PersistStore(state => state.setCollectionsMap)
+    const setAllCollections = PersistStore(state => state.setAllCollections)
 
     const [ pageTitle, setPageTitle] = useState(collectionsMap[apiCollectionId])
     const [apiEndpoints, setApiEndpoints] = useState([])
@@ -907,12 +909,24 @@ function ApiEndpoints(props) {
         )
       ]
 
+    function updateCollectionName(list, apiCollectionId, newName) {
+        list.forEach(item => {
+            if (item.id === apiCollectionId) {
+                item.displayName = newName;
+                item.name = newName;
+            }
+        });
+    }
+
 
       const handleSaveClick = async () => {
         api.editCollectionName(apiCollectionId, editableTitle).then((resp) => {
             func.setToast(true, false, 'Collection name updated successfully!')
             setPageTitle(editableTitle)
             collectionsMap[apiCollectionId] = editableTitle
+            setCollectionsMap(collectionsMap)
+            updateCollectionName(allCollections, parseInt(apiCollectionId, 10), editableTitle)
+            setAllCollections(allCollections)
             setIsEditing(false);
         }).catch((err) => {
             setEditableTitle(pageTitle)