diff --git a/Sources/RemminaMac/Services/PTYSession.swift b/Sources/RemminaMac/Services/PTYSession.swift index 681f6c2..a155f47 100644 --- a/Sources/RemminaMac/Services/PTYSession.swift +++ b/Sources/RemminaMac/Services/PTYSession.swift @@ -25,6 +25,8 @@ final class PTYSession: ObservableObject { var executable = "" var arguments: [String] = [] var customEnv: [String: String] = [:] + /// Fingerprint der Konfiguration (TerminalManager) — bei Änderung wird neu konfiguriert. + var configTag = "" private var masterFD: Int32 = -1 private var process: Process? @@ -288,8 +290,27 @@ final class TerminalManager: ObservableObject { private var sessions: [UUID: PTYSession] = [:] func session(for profile: ConnectionProfile, key: StoredSSHKey?) -> PTYSession { - if let existing = sessions[profile.id] { return existing } + // Fingerprint aus allem, was die Verbindung bestimmt (inkl. Secret-Wert!): + // Ändert der User Host/Passwort/Key, wird neu konfiguriert statt stale weiterzuwählen. + let secret = SSHService.loginSecret(for: profile, key: key) + let tag = [profile.proto.rawValue, profile.host, "\(profile.port)", profile.username, + profile.extraArgs, key?.privateKeyPath ?? "", secret ?? ""].joined(separator: "|") + if let existing = sessions[profile.id] { + // Laufende Session niemals unter den Füßen wegkonfigurieren — erst Trennen. + if existing.isRunning || existing.configTag == tag { return existing } + existing.stop() + configure(existing, profile: profile, key: key, secret: secret) + existing.configTag = tag + return existing + } let s = PTYSession() + configure(s, profile: profile, key: key, secret: secret) + s.configTag = tag + sessions[profile.id] = s + return s + } + + private func configure(_ s: PTYSession, profile: ConnectionProfile, key: StoredSSHKey?, secret: String?) { let baseExe: String let baseArgs: [String] if profile.proto == .sftp { @@ -302,7 +323,7 @@ final class TerminalManager: ObservableObject { // Auto-Login nur mit Keychain-Secret + sshpass, sonst interaktiver Prompt im Terminal. // SSH-Agent durchreichen (falls vorhanden): gleiche Keys wie in Terminal.app. let cfg = SSHService.sessionConfig(executable: baseExe, args: baseArgs, - password: SSHService.loginSecret(for: profile, key: key), + password: secret, sshpassPath: SSHService.findSshpass()) s.configure(executable: cfg.executable, arguments: cfg.arguments) var env = cfg.extraEnv @@ -311,8 +332,6 @@ final class TerminalManager: ObservableObject { } s.customEnv = env s.setAutoLogin(cfg.autoLogin) - sessions[profile.id] = s - return s } func close(_ profile: ConnectionProfile) { diff --git a/docs/ERRORS.md b/docs/ERRORS.md index 79bc3d2..d8cfdcb 100644 --- a/docs/ERRORS.md +++ b/docs/ERRORS.md @@ -22,6 +22,7 @@ Stand: 2026-09-21 · Version 0.1.0 · macOS arm64 · Swift 5.9+ · Xcode 15+ | 13 | 2026-09-21 | Permission denied trotz richtigem Passwort | Text-Ersetzung im Eingabefeld verfälscht Sonderzeichen | hoch | ✅ behoben — gehärtetes Eingabefeld (alle Ersetzungen aus, 9/9 Tests) | | 14 | 2026-09-21 | Terminal geht, App nicht (gleiches Passwort) | SSH-Agent-Socket fehlt GUI-Prozessen → evtl. Key-Login in Terminal | hoch | ✅ App reicht Agent jetzt durch; Gegenprobe beim User offen | | 15 | 2026-09-21 | Crash bei Tippen + Senden | Ursache unklar (headless nicht reproduzierbar) | kritisch | ⬜ Gehärtet (v0.5.3); Crash-Log angefragt | +| 16 | 2026-09-22 | Profil-Änderungen wirkungslos | Session-Cache ignorierte Host-/Passwort-/Key-Korrekturen | hoch | ✅ behoben — Fingerprint-Reconfig (9/9 Tests) | --- @@ -211,3 +212,9 @@ Zweite Zeile erzwingt Passwort-Login ohne Agent — scheitert sie auch im Termin - `PTYSession.send`: Force-Unwrap `baseAddress!` → Guard, Leer-Check **Weiter nötig bei erneutem Crash:** Console.app → Absturzberichte → neuesten `RemminaMac`-Eintrag öffnen → `Exception Type`, `Termination Reason` und crashed Thread hierher schicken. Damit steht die Stelle fest. **Status:** ⬜ wartet auf Rückmeldung / Crash-Log. + +## #16 — Profil-Änderungen wirkungslos („geht immer noch nicht") +**Stand:** Nach mehreren Fixes meldet der User weiterhin pauschal Misserfolg. Code-Review (`TerminalManager.session`, Zeile 291): Eine einmal erstellte Session wurde für immer wiederverwendet — Host-/Passwort-/Key-Korrekturen im Editor erreichten die laufende/stale Session nie. Jede Korrektur lief ins Leere. +**Fix:** Konfigurations-Fingerprint (Proto/Host/Port/User/ExtraArgs/Key-Pfad/Secret-Wert) pro Session. Bei Abweichung und nicht-laufender Session: stoppen + neu konfigurieren (gleiches Objekt, Tabs bleiben). Laufende Sessions werden nie unter den Füßen wegkonfiguriert ( erst Trennen). +**Verifikation:** Harness 9/9 (Host-Wechsel reconfiguriert, Secret-Wechsel erkannt, laufende ssh-Session an Banner-lesendem Socket unangetastet). +**Status:** ✅ behoben — bitte mit v0.5.4 erneut testen (Profil ggf. einmal neu speichern).