fix delete connection POST request

deployment/docker-compose.yml

@@ -38,7 +38,7 @@ services:
     # Optionally expose code to other services (read-only)
     # - ./:/srv/webssh:ro
     healthcheck:
-      test: ["CMD", "curl", "-f", "http://localhost:8080/"]
+      test: ["CMD", "python", "-c", "import sys, urllib.request; sys.exit(0 if urllib.request.urlopen('http://127.0.0.1:8080/', timeout=5).getcode() == 200 else 1)"]
       interval: 30s
       timeout: 10s
       retries: 3

src/static/webssh.js

@@ -776,16 +776,17 @@ function disconnectConnection(id) {
   }
 }
 
-function deleteConnection(id) {
+async function deleteConnection(id) {
   if (!confirm('Delete this connection?')) return;
   
   const conn = getConnection(id);
-  if (conn && conn.socket) {
-    conn.socket.close();
+  if (!await deleteConnectionFromServer(id)) {
+    alert('Failed to delete connection from server. Please try again.');
+    return;
   }
-  
+
+  if (conn && conn.socket) conn.socket.close();
   connections = connections.filter(c => c.id !== id);
-  deleteConnectionFromServer(id); // Delete from server storage
   autoSaveConnections();
   
   if (currentConnectionId === id) {

src/webssh_backend/config.py

@@ -75,18 +75,27 @@ def parse_size(size_str):
     size_str = size_str.upper().strip()
     logger.info(f"parse_size called with: '{size_str}'")
 
-    if size_str.endswith('M') or size_str.endswith('MB'):
-        # Remove 'M' or 'MB' and convert to bytes
+    if size_str.endswith('MB'):
+        result = int(float(size_str[:-2]) * 1024 * 1024)
+        logger.info(f"parse_size result: {result} bytes (from {size_str})")
+        return result
+    elif size_str.endswith('M'):
         result = int(float(size_str[:-1]) * 1024 * 1024)
         logger.info(f"parse_size result: {result} bytes (from {size_str})")
         return result
-    elif size_str.endswith('K') or size_str.endswith('KB'):
-        # Remove 'K' or 'KB' and convert to bytes
+    elif size_str.endswith('KB'):
+        result = int(float(size_str[:-2]) * 1024)
+        logger.info(f"parse_size result: {result} bytes (from {size_str})")
+        return result
+    elif size_str.endswith('K'):
         result = int(float(size_str[:-1]) * 1024)
         logger.info(f"parse_size result: {result} bytes (from {size_str})")
         return result
-    elif size_str.endswith('G') or size_str.endswith('GB'):
-        # Remove 'G' or 'GB' and convert to bytes
+    elif size_str.endswith('GB'):
+        result = int(float(size_str[:-2]) * 1024 * 1024 * 1024)
+        logger.info(f"parse_size result: {result} bytes (from {size_str})")
+        return result
+    elif size_str.endswith('G'):
         result = int(float(size_str[:-1]) * 1024 * 1024 * 1024)
         logger.info(f"parse_size result: {result} bytes (from {size_str})")
         return result

src/webssh_backend/database.py

@@ -208,8 +208,12 @@ class CredentialsManager:
                 "DELETE FROM connections WHERE id = ? AND username = ?",
                 (conn_id, username)
             )
+            deleted = cursor.rowcount
             conn.commit()
             conn.close()
+            if deleted != 1:
+                logger.warning(f"Connection {conn_id} not found for user {username}")
+                return False
             logger.info(f"Deleted connection {conn_id} for user {username}")
             return True
         except Exception as e:

src/webssh_backend/http_api.py

@@ -46,6 +46,8 @@ class LoginHandler(BaseHTTPRequestHandler):
             self.handle_verify()
         elif parsed.path == '/api/connections':
             self.handle_save_connection()
+        elif parsed.path == '/api/connections/delete':
+            self.handle_delete_connection()
         elif parsed.path == '/api/connections/status':
             self.handle_connection_status()
         elif parsed.path == '/api/config':