fix: guacd hosts not migarting password

src/backend/database/db/index.ts

@@ -1092,9 +1092,9 @@ const migrateSchema = () => {
     }
   }
 
-  // One-time migration: copy old generic credentials into protocol-specific fields
+  // Copy unencrypted username/domain into protocol-specific columns for old guac hosts.
-  // for hosts that were created before the multi-protocol schema was introduced.
+  // Passwords are handled via the legacy field name fallback in lazy-field-encryption.ts.
-  const credentialBackfills = [
+  const usernameDomainBackfills = [
     {
       protocol: "rdp",
       sql: "UPDATE ssh_data SET rdp_user = username, rdp_password = password, rdp_domain = domain WHERE connection_type = 'rdp' AND rdp_user IS NULL AND rdp_password IS NULL",
@@ -1108,21 +1108,20 @@ const migrateSchema = () => {
       sql: "UPDATE ssh_data SET telnet_user = username, telnet_password = password WHERE connection_type = 'telnet' AND telnet_user IS NULL AND telnet_password IS NULL",
     },
   ];
-
+  for (const backfill of usernameDomainBackfills) {
-  for (const backfill of credentialBackfills) {
     try {
       const result = sqlite.prepare(backfill.sql).run();
       if (result.changes > 0) {
         databaseLogger.info(
-          `Backfilled credentials for ${result.changes} ${backfill.protocol} host(s)`,
+          `Backfilled ${result.changes} ${backfill.protocol} host credential(s)`,
-          { operation: "credential_backfill" },
+          { operation: "guac_credential_backfill" },
         );
       }
-    } catch (backfillError) {
+    } catch (e) {
-      databaseLogger.warn(
+      databaseLogger.warn(`Failed to backfill ${backfill.protocol} host credentials`, {
-        `Failed to backfill ${backfill.protocol} credentials`,
+        operation: "guac_credential_backfill",
-        { operation: "credential_backfill", error: backfillError },
+        error: e,
-      );
+      });
     }
   }
 

src/backend/utils/lazy-field-encryption.ts

@@ -103,6 +103,27 @@ export class LazyFieldEncryption {
           }
         }
 
+        // Guac hosts migrated from single-protocol: rdpPassword/vncPassword/telnetPassword
+        // columns were populated by copying the encrypted `password` blob. Try decrypting
+        // under the original field name before giving up.
+        if (
+          fieldName === "rdpPassword" ||
+          fieldName === "vncPassword" ||
+          fieldName === "telnetPassword"
+        ) {
+          try {
+            const decrypted = FieldCrypto.decryptField(
+              fieldValue,
+              userKEK,
+              recordId,
+              "password",
+            );
+            return decrypted;
+          } catch {
+            // not encrypted as "password" either
+          }
+        }
+
         const sensitiveFields = [
           "totpSecret",
           "totpBackupCodes",