enhance: improve AdminWeb UI/UX and features

powerkimhub · powerkimhub · commit e6f1601cc0d0 · 2025-12-30T22:11:20.000+09:00
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Cluster.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Cluster.go
@@ -90,5 +90,9 @@ func ClusterManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Connection.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Connection.go
@@ -688,5 +688,9 @@ func ConnectionManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Credential.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Credential.go
@@ -98,5 +98,9 @@ func CredentialManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Dashboard.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Dashboard.go
@@ -219,5 +219,9 @@ func Dashboard(c echo.Context) error {
 		ShowEmpty:      showEmpty,
 	}
 
-	return tmplCache.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmplCache.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Disk.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Disk.go
@@ -118,5 +118,9 @@ func DiskManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Driver.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Driver.go
@@ -210,5 +210,9 @@ func DriverManagement(c echo.Context) error {
 	}
 
 	// Execute template
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-KeyPair.go b/api-runtime/rest-runtime/admin-web/AdminWeb-KeyPair.go
@@ -102,5 +102,9 @@ func KeyPairManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-MyImage.go b/api-runtime/rest-runtime/admin-web/AdminWeb-MyImage.go
@@ -102,5 +102,9 @@ func MyImageManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-NLB.go b/api-runtime/rest-runtime/admin-web/AdminWeb-NLB.go
@@ -118,5 +118,9 @@ func NLBManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-Region.go b/api-runtime/rest-runtime/admin-web/AdminWeb-Region.go
@@ -85,5 +85,9 @@ func RegionManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-SecurityGroup.go b/api-runtime/rest-runtime/admin-web/AdminWeb-SecurityGroup.go
@@ -121,5 +121,9 @@ func SecurityGroupManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-SystemStats.go b/api-runtime/rest-runtime/admin-web/AdminWeb-SystemStats.go
@@ -170,5 +170,9 @@ func SystemStatsInfoPage(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-VM.go b/api-runtime/rest-runtime/admin-web/AdminWeb-VM.go
@@ -151,5 +151,9 @@ func VMManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/AdminWeb-VPC-Subnet.go b/api-runtime/rest-runtime/admin-web/AdminWeb-VPC-Subnet.go
@@ -115,5 +115,9 @@ func VPCSubnetManagement(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})
 	}
 
-	return tmpl.Execute(c.Response().Writer, data)
+	c.Response().WriteHeader(http.StatusOK)
+	if err := tmpl.Execute(c.Response().Writer, data); err != nil {
+		return fmt.Errorf("failed to execute template: %w", err)
+	}
+	return nil
 }
diff --git a/api-runtime/rest-runtime/admin-web/html/body_frame.html b/api-runtime/rest-runtime/admin-web/html/body_frame.html
@@ -75,7 +75,7 @@ <h2>How To Use</h2>
             <h3 style="display: flex; align-items: center; justify-content: space-between;">
                 1. Set Up Connections
                 <span style="font-size: 12px; font-weight: normal; color: #007bff;">
-                    💡 <a href="https://github.com/cloud-barista/cb-spider/tree/master/utils/default-connection-helper#readme" target="_blank" style="color: #007bff; text-decoration: none;">Default Connection Helper</a> for quick setup
+                    💡 <a href="https://github.com/cloud-barista/cb-spider/tree/master/utils/default-connection-helper#readme" target="_blank" style="color: #007bff; text-decoration: none;">CLI Auto-Registration Tool </a>
                 </span>
             </h3>
             <ol>
diff --git a/api-runtime/rest-runtime/admin-web/html/left_menu.html b/api-runtime/rest-runtime/admin-web/html/left_menu.html
@@ -4,8 +4,8 @@
     <meta charset="UTF-8">
     <style>
         .menu-item {
-            margin: 3px 0; /* Further reduced margin */
-            padding: 10px;
+            margin: 0; /* Remove margin between menu items */
+            padding: 6px 10px; /* Reduced vertical padding */
             font-size: 12px;
             font-weight: bold;
             color: #0645AD;
@@ -72,7 +72,8 @@
                 myimage: "/spider/adminweb/myimage",
                 nlb: "/spider/adminweb/nlb",
                 cluster: "/spider/adminweb/cluster",
-                s3: "/spider/adminweb/s3"
+                s3: "/spider/adminweb/s3",
+                filesystem: "/spider/adminweb/filesystem"
             };
 
             const targetUrl = `${menuUrlMap[menu]}/${encodeURIComponent(connectionName)}`;
@@ -132,7 +133,7 @@
         }
 
         function changeSelectedImage(selectedMenu) {
-            const menus = ['dashboard', 'connection', 'vpc', 'securitygroup', 'keypair', 'vm', 'disk', 'myimage', 'nlb', 'cluster', 's3'];
+            const menus = ['dashboard', 'connection', 'vpc', 'securitygroup', 'keypair', 'vm', 'disk', 'myimage', 'nlb', 'cluster', 's3', 'filesystem'];
             menus.forEach(menu => {
                 const imgElement = document.getElementById(menu + '-img');
                 if (imgElement) {
@@ -147,7 +148,7 @@
 
         window.addEventListener('message', function(event) {
             if (event.data.type === 'deselectLeftMenu') {
-                const menus = ['dashboard', 'connection', 'vpc', 'securitygroup', 'keypair', 'vm', 'disk', 'myimage', 'nlb', 'cluster', 's3'];
+                const menus = ['dashboard', 'connection', 'vpc', 'securitygroup', 'keypair', 'vm', 'disk', 'myimage', 'nlb', 'cluster', 's3', 'filesystem'];
                 menus.forEach(menu => {
                     const imgElement = document.getElementById(menu + '-img');
                     if (imgElement) {
@@ -159,7 +160,7 @@
     </script>
 </head>
 <body>
-    <a href="/spider/adminweb" target="_top" class="menu-item logo">
+    <a href="/spider/adminweb" target="_top" class="menu-item logo" onclick="sessionStorage.setItem('skipAbout', 'true');">
         <img src="./images/logo.png" border="0">
     </a>
     <div class="divider"></div> <!-- Divider after logo -->
@@ -211,6 +212,10 @@
         <img id="s3-img" src="./images/left-menu/s3.png" alt="S3 Icon">
         <div class="menu-text">S3</div>
     </div>
-    <div class="divider"></div> <!-- Divider after S3 -->
+    <div class="menu-item" onclick="checkConnectionAndRedirect('filesystem', event)">
+        <img id="filesystem-img" src="./images/left-menu/filesystem.png" alt="FileSystem Icon" onerror="this.src='./images/left-menu/disk.png'">
+        <div class="menu-text">FS (WIP)</div>
+    </div>
+    <div class="divider"></div> <!-- Divider after FileSystem -->
 </body>
 </html>
diff --git a/api-runtime/rest-runtime/admin-web/html/main.html b/api-runtime/rest-runtime/admin-web/html/main.html

Original file line number	Diff line number	Diff line change
`@@ -90,5 +90,9 @@ func ClusterManagement(c echo.Context) error {`
`90`	`90`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`91`	`91`	`}`
`92`	`92`
`93`		`- return tmpl.Execute(c.Response().Writer, data)`
	`93`	`+ c.Response().WriteHeader(http.StatusOK)`
	`94`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`95`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`96`	`+ }`
	`97`	`+ return nil`
`94`	`98`	`}`
Original file line number	Diff line number	Diff line change
`@@ -688,5 +688,9 @@ func ConnectionManagement(c echo.Context) error {`
`688`	`688`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`689`	`689`	`}`
`690`	`690`
`691`		`- return tmpl.Execute(c.Response().Writer, data)`
	`691`	`+ c.Response().WriteHeader(http.StatusOK)`
	`692`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`693`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`694`	`+ }`
	`695`	`+ return nil`
`692`	`696`	`}`
Original file line number	Diff line number	Diff line change
`@@ -98,5 +98,9 @@ func CredentialManagement(c echo.Context) error {`
`98`	`98`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`99`	`99`	`}`
`100`	`100`
`101`		`- return tmpl.Execute(c.Response().Writer, data)`
	`101`	`+ c.Response().WriteHeader(http.StatusOK)`
	`102`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`103`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`104`	`+ }`
	`105`	`+ return nil`
`102`	`106`	`}`
Original file line number	Diff line number	Diff line change
`@@ -219,5 +219,9 @@ func Dashboard(c echo.Context) error {`
`219`	`219`	`ShowEmpty: showEmpty,`
`220`	`220`	`}`
`221`	`221`
`222`		`- return tmplCache.Execute(c.Response().Writer, data)`
	`222`	`+ c.Response().WriteHeader(http.StatusOK)`
	`223`	`+ if err := tmplCache.Execute(c.Response().Writer, data); err != nil {`
	`224`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`225`	`+ }`
	`226`	`+ return nil`
`223`	`227`	`}`
Original file line number	Diff line number	Diff line change
`@@ -118,5 +118,9 @@ func DiskManagement(c echo.Context) error {`
`118`	`118`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`119`	`119`	`}`
`120`	`120`
`121`		`- return tmpl.Execute(c.Response().Writer, data)`
	`121`	`+ c.Response().WriteHeader(http.StatusOK)`
	`122`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`123`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`124`	`+ }`
	`125`	`+ return nil`
`122`	`126`	`}`
Original file line number	Diff line number	Diff line change
`@@ -210,5 +210,9 @@ func DriverManagement(c echo.Context) error {`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`// Execute template`
`213`		`- return tmpl.Execute(c.Response().Writer, data)`
	`213`	`+ c.Response().WriteHeader(http.StatusOK)`
	`214`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`215`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`216`	`+ }`
	`217`	`+ return nil`
`214`	`218`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,5 +102,9 @@ func KeyPairManagement(c echo.Context) error {`
`102`	`102`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`103`	`103`	`}`
`104`	`104`
`105`		`- return tmpl.Execute(c.Response().Writer, data)`
	`105`	`+ c.Response().WriteHeader(http.StatusOK)`
	`106`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`107`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`108`	`+ }`
	`109`	`+ return nil`
`106`	`110`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,5 +102,9 @@ func MyImageManagement(c echo.Context) error {`
`102`	`102`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`103`	`103`	`}`
`104`	`104`
`105`		`- return tmpl.Execute(c.Response().Writer, data)`
	`105`	`+ c.Response().WriteHeader(http.StatusOK)`
	`106`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`107`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`108`	`+ }`
	`109`	`+ return nil`
`106`	`110`	`}`
Original file line number	Diff line number	Diff line change
`@@ -118,5 +118,9 @@ func NLBManagement(c echo.Context) error {`
`118`	`118`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`119`	`119`	`}`
`120`	`120`
`121`		`- return tmpl.Execute(c.Response().Writer, data)`
	`121`	`+ c.Response().WriteHeader(http.StatusOK)`
	`122`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`123`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`124`	`+ }`
	`125`	`+ return nil`
`122`	`126`	`}`
Original file line number	Diff line number	Diff line change
`@@ -85,5 +85,9 @@ func RegionManagement(c echo.Context) error {`
`85`	`85`	`return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Error loading template: " + err.Error()})`
`86`	`86`	`}`
`87`	`87`
`88`		`- return tmpl.Execute(c.Response().Writer, data)`
	`88`	`+ c.Response().WriteHeader(http.StatusOK)`
	`89`	`+ if err := tmpl.Execute(c.Response().Writer, data); err != nil {`
	`90`	`+ return fmt.Errorf("failed to execute template: %w", err)`
	`91`	`+ }`
	`92`	`+ return nil`
`89`	`93`	`}`