Fix stale Sessions: Fingerprint-Reconfig bei Profil-Aenderung, Doku

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?
@@ -205,12 +207,13 @@ final class PTYSession: ObservableObject {
     func send(_ string: String) {
         let fd = masterFD
         guard fd >= 0 else { return }
-        guard let data = string.data(using: .utf8) else { return }
+        guard let data = string.data(using: .utf8), !data.isEmpty else { return }
         ioQueue.async {
             data.withUnsafeBytes { ptr in
+                guard let base = ptr.baseAddress else { return }
                 var written = 0
                 while written < data.count {
-                    let n = write(fd, ptr.baseAddress!.advanced(by: written), data.count - written)
+                    let n = write(fd, base.advanced(by: written), data.count - written)
                     if n <= 0 { break }
                     written += n
                 }
@@ -287,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 {
@@ -301,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
@@ -310,8 +332,6 @@ final class TerminalManager: ObservableObject {
         }
         s.customEnv = env
         s.setAutoLogin(cfg.autoLogin)
-        sessions[profile.id] = s
-        return s
     }
 
     func close(_ profile: ConnectionProfile) {

Sources/RemminaMac/Views/TerminalInputField.swift

@@ -38,10 +38,19 @@ struct TerminalInputField: NSViewRepresentable {
 
     func updateNSView(_ view: NSView, context: Context) {
         context.coordinator.onSubmit = onSubmit
-        if let f = view as? NSTextField, f.stringValue != text {
+        // Delegat während Programm-Sets abhängen: sonst feuert textDidChange synchron
+        // zurück ins SwiftUI-State — mitten im Render -> Crash ("Modifying state
+        // during view update"). User-Tippen läuft weiter über den Delegaten.
+        if let f = view as? NSTextField {
+            if f.stringValue != text {
+                f.delegate = nil
                 f.stringValue = text
+                f.delegate = context.coordinator
+            }
         } else if let tv = view as? NSTextView, tv.string != text {
+            tv.delegate = nil
             tv.string = text
+            tv.delegate = context.coordinator
         }
     }
 

docs/ERRORS.md

@@ -21,6 +21,8 @@ Stand: 2026-09-21 · Version 0.1.0 · macOS arm64 · Swift 5.9+ · Xcode 15+
 | 12 | 2026-09-21 | „Immer Verbindung falsch" | User-Report ohne Details (Ziel/Credentials unbekannt) | offen | ⬜ Teils: Editor-Validierung + kopierbarer Fehlerbericht; warte auf Bericht |
 | 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) |
 
 ---
 
@@ -198,3 +200,21 @@ ssh -o IdentitiesOnly=yes administrator@sysdaemon.xyz
 ```
 Zweite Zeile erzwingt Passwort-Login ohne Agent — scheitert sie auch im Terminal, war es immer der Agent-Key.
 **Status:** ✅ App-Seite umgesetzt — warte auf Gegenprobe.
+
+## #15 — Crash bei Tippen + Senden (User-Report)
+**Stand:** „Sobald ich was eintippe und absende, crasht das Programm." Headless nicht reproduzierbar — alle Ebenen einzeln grün.
+**Ausgeschlossen per Test:**
+- SIGPIPE beim Schreiben (Harness: macOS liefert EIO statt Signal; `send` nach `stop()` überlebt) — Hypothese widerlegt
+- View-Schicht (echter Coordinator + NSTextView: Tippen mit Umlauten/Quotes/Paste, Enter, 200 Programm-Sets) — grün
+- PTY-Roundtrip, Refused-, Keychain-, TCP-Harness — grün
+**Gehärtet (v0.5.3, ohne Beweis dass es DER Crash war):**
+- `TerminalInputField.updateNSView`: Delegat bei Programm-Sets abgehängt — sonst feuert `textDidChange` synchron zurück ins State während SwiftUI rendert („Modifying state during view update" → Crash-Kandidat Nr. 1, genau beim Senden: Eingabe wird geleert)
+- `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).