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

Sources/RemminaMac/Models/ProfileValidation.swift

@@ -0,0 +1,38 @@
+import Foundation
+
+/// Eingabe-Prüfung + Bereinigung für Profile (reine Funktionen, testbar).
+/// Fängt die häufigsten „Verbindung falsch"-Ursachen ab: Leerzeichen,
+/// user@host im Host-Feld, Protokoll-Präfix, leerer Benutzername.
+enum ProfileValidation {
+    static func cleaned(_ profile: ConnectionProfile) -> ConnectionProfile {
+        var p = profile
+        p.name = p.name.trimmingCharacters(in: .whitespacesAndNewlines)
+        p.host = p.host.trimmingCharacters(in: .whitespacesAndNewlines)
+        p.username = p.username.trimmingCharacters(in: .whitespacesAndNewlines)
+        p.group = p.group.trimmingCharacters(in: .whitespacesAndNewlines)
+        p.resolution = p.resolution.trimmingCharacters(in: .whitespacesAndNewlines)
+        p.extraArgs = p.extraArgs.trimmingCharacters(in: .whitespacesAndNewlines)
+        return p
+    }
+
+    static func warnings(for profile: ConnectionProfile) -> [String] {
+        var w: [String] = []
+        let h = profile.host.trimmingCharacters(in: .whitespacesAndNewlines)
+        if h.contains("@") {
+            w.append("Host enthält „@“ — den Benutzernamen ins Feld darunter eintragen, hier nur Hostname/IP.")
+        }
+        if h.contains("://") {
+            w.append("Host enthält „://“ — nur Hostname/IP, kein Protokoll (das steht oben).")
+        }
+        if h.contains(where: { $0.isWhitespace }) {
+            w.append("Host enthält Leerzeichen.")
+        }
+        if profile.username.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
+            w.append("Benutzername fehlt.")
+        }
+        if !(1...65535).contains(profile.port) {
+            w.append("Port ungültig (1–65535).")
+        }
+        return w
+    }
+}

Sources/RemminaMac/Services/ConnectionServices.swift

@@ -39,6 +39,67 @@ enum SSHService {
         NSPasteboard.general.clearContents()
         NSPasteboard.general.setString(cmd, forType: .string)
     }
+
+    // MARK: - Passwort / Auto-Login
+
+    /// SSH-Agent-Socket für GUI-Prozesse auflösen (Finder-Apps erben SSH_AUTH_SOCK
+    /// nicht, Terminal schon — ohne diesen Unterschied nutzt Terminal evtl. einen
+    /// Agent-Key, während die App nach dem Passwort fragt und scheitert).
+    static func agentSocket() -> String? {
+        let p = Process()
+        p.executableURL = URL(fileURLWithPath: "/bin/launchctl")
+        p.arguments = ["getenv", "SSH_AUTH_SOCK"]
+        let out = Pipe()
+        let err = Pipe()
+        p.standardOutput = out
+        p.standardError = err
+        do {
+            try p.run()
+            p.waitUntilExit()
+        } catch {
+            return nil
+        }
+        let path = String(data: out.fileHandleForReading.readDataToEndOfFile(), encoding: .utf8)?
+            .trimmingCharacters(in: .whitespacesAndNewlines) ?? ""
+        guard !path.isEmpty,
+              FileManager.default.fileExists(atPath: path) else { return nil }
+        return path
+    }
+
+    /// Gespeichertes Login-Secret: erst Profil-Passwort, dann Key-Passphrase (Keychain, nie JSON).
+    static func loginSecret(for profile: ConnectionProfile, key: StoredSSHKey?) -> String? {
+        if let acct = profile.passwordAccount,
+           let pw = KeychainHelper.load(account: acct), !pw.isEmpty {
+            return pw
+        }
+        if let key, let acct = key.passphraseAccount,
+           let pp = KeychainHelper.load(account: acct), !pp.isEmpty {
+            return pp
+        }
+        return nil
+    }
+
+    static func findSshpass() -> String? {
+        for p in ["/opt/homebrew/bin/sshpass", "/usr/local/bin/sshpass", "/usr/bin/sshpass"] {
+            if FileManager.default.isExecutableFile(atPath: p) { return p }
+        }
+        return nil
+    }
+
+    /// true = beim Verbinden wird ohne Tippen eingeloggt (sshpass + Keychain-Secret).
+    static func willAutoLogin(profile: ConnectionProfile, key: StoredSSHKey?) -> Bool {
+        loginSecret(for: profile, key: key) != nil && findSshpass() != nil
+    }
+
+    /// Reine Funktion (testbar): mit Secret+sshpass -> sshpass -e … + SSHPASS-Env,
+    /// sonst direkter Aufruf. Passwort steht NIE in der Befehlszeile (nur im Env).
+    static func sessionConfig(executable: String, args: [String], password: String?, sshpassPath: String?)
+        -> (executable: String, arguments: [String], extraEnv: [String: String], autoLogin: Bool) {
+        if let password, !password.isEmpty, let sshpass = sshpassPath, !sshpass.isEmpty {
+            return (sshpass, ["-e", executable] + args, ["SSHPASS": password], true)
+        }
+        return (executable, args, [:], false)
+    }
 }
 
 enum RDPService {

Sources/RemminaMac/Services/PTYSession.swift

@@ -20,9 +20,13 @@ enum PTYState: Equatable {
 final class PTYSession: ObservableObject {
     @Published private(set) var lines: [[StyledText]] = [[]]
     @Published private(set) var state: PTYState = .never
+    @Published private(set) var autoLogin = false
 
     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?
@@ -48,6 +52,10 @@ final class PTYSession: ObservableObject {
         self.arguments = arguments
     }
 
+    func setAutoLogin(_ value: Bool) {
+        autoLogin = value
+    }
+
     func start() {
         cleanupSync()
         epoch += 1
@@ -79,6 +87,7 @@ final class PTYSession: ObservableObject {
         var env = ProcessInfo.processInfo.environment
         env["TERM"] = "xterm-256color"
         if env["LANG"] == nil { env["LANG"] = "de_DE.UTF-8" }
+        for (k, v) in customEnv { env[k] = v }
         p.environment = env
         // closeOnDealloc=false: Parent schließt slave explizit nach run().
         p.standardInput = FileHandle(fileDescriptor: slave)
@@ -198,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
                 }
@@ -280,17 +290,50 @@ 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 }
-        let s = PTYSession()
-        if profile.proto == .sftp {
-            s.configure(executable: "/usr/bin/sftp", arguments: ["-P", "\(profile.port)"] + (key.map { ["-i", $0.privateKeyPath] } ?? []) + ["\(profile.username)@\(profile.host)"])
-        } else {
-            s.configure(executable: "/usr/bin/ssh", arguments: SSHService.sshArguments(for: profile, key: key))
+        // 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 {
+            baseExe = "/usr/bin/sftp"
+            baseArgs = ["-P", "\(profile.port)"] + (key.map { ["-i", $0.privateKeyPath] } ?? []) + ["\(profile.username)@\(profile.host)"]
+        } else {
+            baseExe = "/usr/bin/ssh"
+            baseArgs = SSHService.sshArguments(for: profile, key: key)
+        }
+        // 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: secret,
+                                           sshpassPath: SSHService.findSshpass())
+        s.configure(executable: cfg.executable, arguments: cfg.arguments)
+        var env = cfg.extraEnv
+        if let sock = SSHService.agentSocket() {
+            env["SSH_AUTH_SOCK"] = sock
+        }
+        s.customEnv = env
+        s.setAutoLogin(cfg.autoLogin)
+    }
+
     func close(_ profile: ConnectionProfile) {
         sessions[profile.id]?.stop()
         sessions.removeValue(forKey: profile.id)

Sources/RemminaMac/Views/ContentView.swift

@@ -104,7 +104,7 @@ struct ContentView: View {
             }
         }
         .sheet(item: $editingProfile) { profile in
-            ProfileEditorView(profile: profile) { saved in
+            ProfileEditorView(profile: profile, isNew: !store.profiles.contains(where: { $0.id == profile.id })) { saved in
                 if store.profiles.contains(where: { $0.id == saved.id }) {
                     store.update(saved)
                 } else {

Sources/RemminaMac/Views/EmbeddedTerminalView.swift

@@ -41,8 +41,15 @@ struct EmbeddedTerminalView: View {
                         Text(lastMessage)
                             .font(.caption).foregroundStyle(Dracula.foreground)
                             .lineLimit(3)
+                        if lastMessage.localizedCaseInsensitiveContains("permission denied") {
+                            Text("Tipp: Passwort im Auge-Modus eingeben, Tastaturlayout prüfen. Falls Terminal.app ohne Passwort einloggt (Agent-Key: ssh-add -l), Key in der App hinterlegen oder Agent nutzen.")
+                                .font(.caption).foregroundStyle(Dracula.orange)
+                        }
                     }
                     Spacer()
+                    Button("Bericht kopieren") { copyReport() }
+                        .buttonStyle(.link).font(.caption)
+                        .help("Kopiert Profil (ohne Passwort), Status und letzte Ausgabe — hierher schicken zur Fehlersuche")
                     Button("Diagnose") { diagnose() }.buttonStyle(.link).font(.caption)
                     Button("Erneut") { connect() }.buttonStyle(.link).font(.caption)
                 }
@@ -58,16 +65,17 @@ struct EmbeddedTerminalView: View {
             // Eingabezeile
             HStack(spacing: 6) {
                 Text("❯").font(.system(.body, design: .monospaced)).foregroundStyle(Dracula.green)
-                if secureInput {
-                    SecureField("Eingabe (versteckt, z. B. Passwort) …", text: $input, onCommit: submit)
-                        .textFieldStyle(.plain)
+                // Gehärtetes Feld: keine macOS-Text-Ersetzungen (Passwort-Sonderzeichen!)
+                ZStack(alignment: .leading) {
+                    if input.isEmpty {
+                        Text(secureInput ? "Eingabe (versteckt, z. B. Passwort) …" : "Eingabe …")
                             .font(.system(.body, design: .monospaced))
-                        .foregroundStyle(Dracula.foreground)
-                } else {
-                    TextField("Eingabe …", text: $input, onCommit: submit)
-                        .textFieldStyle(.plain)
-                        .font(.system(.body, design: .monospaced))
-                        .foregroundStyle(Dracula.foreground)
+                            .foregroundStyle(Dracula.comment.opacity(0.7))
+                            .allowsHitTesting(false)
+                    }
+                    TerminalInputField(text: $input, isSecure: secureInput, onSubmit: submit)
+                        .id(secureInput)
+                        .frame(height: 22)
                 }
                 Button(action: { secureInput.toggle() }) {
                     Image(systemName: secureInput ? "eye.slash" : "eye")
@@ -129,6 +137,20 @@ struct EmbeddedTerminalView: View {
         input = ""
     }
 
+    /// Fehlerbericht ohne Secrets (Passwort steht weder im Profil noch im Befehl).
+    /// Hinweis: Wer das Passwort sichtbar eingetippt hat (nicht Auge-Modus),
+    /// löscht die Ausgabe vorher per „Leeren".
+    func copyReport() {
+        var r = "RemminaMac Fehlerbericht\n"
+        r += "Profil: \(profile.name) | \(profile.proto.rawValue) | \(profile.username)@\(profile.host):\(profile.port)\n"
+        r += "Auto-Login: \(term.autoLogin ? "ja" : "nein") | sshpass: \(SSHService.findSshpass() ?? "fehlt")\n"
+        r += "Status: \(statusText)\n"
+        let tail = term.lines.suffix(20).map { $0.map(\.text).joined() }.joined(separator: "\n")
+        r += "--- letzte Ausgabe ---\n\(tail)\n"
+        NSPasteboard.general.clearContents()
+        NSPasteboard.general.setString(r, forType: .string)
+    }
+
     var failed: Bool {
         if case .done(let code, _) = term.state {
             return (code ?? 1) != 0
@@ -164,7 +186,7 @@ struct EmbeddedTerminalView: View {
         if checking { return "Prüfe \(profile.host):\(profile.port) …" }
         switch term.state {
         case .never: return canConnect ? "Bereit — verbindet beim Öffnen." : "Host/Benutzer fehlt."
-        case .running: return "Verbunden (PTY, TERM=xterm-256color)."
+        case .running: return term.autoLogin ? "Verbunden (Auto-Login via Keychain)." : "Verbunden (PTY, TERM=xterm-256color)."
         case .done(_, let msg): return msg
         }
     }

Sources/RemminaMac/Views/ProfileEditorView.swift

@@ -4,18 +4,23 @@ struct ProfileEditorView: View {
     @Environment(\.dismiss) var dismiss
     @EnvironmentObject var store: ProfileStore
     @State var profile: ConnectionProfile
+    var isNew: Bool
     var onSave: (ConnectionProfile) -> Void
     @State private var password: String = ""
     @State private var savePassword: Bool = false
+    @State private var storedPasswordPresent: Bool = false
 
-    init(profile: ConnectionProfile, onSave: @escaping (ConnectionProfile) -> Void) {
+    init(profile: ConnectionProfile, isNew: Bool, onSave: @escaping (ConnectionProfile) -> Void) {
         _profile = State(initialValue: profile)
+        self.isNew = isNew
         self.onSave = onSave
     }
 
+    var liveWarnings: [String] { ProfileValidation.warnings(for: profile) }
+
     var body: some View {
         VStack(spacing: 0) {
-            Text(profile.id.uuidString.prefix(8) == ConnectionProfile().id.uuidString.prefix(8) ? "Profil bearbeiten" : "Profil").font(.headline).padding()
+            Text(isNew ? "Neues Profil" : "Profil bearbeiten").font(.headline).padding()
             Form {
                 Section("Allgemein") {
                     TextField("Name", text: $profile.name)
@@ -34,18 +39,38 @@ struct ProfileEditorView: View {
                 }
                 Section("Verbindung") {
                     TextField("Host", text: $profile.host)
+                        .autocorrectionDisabled()
                     TextField("Port", value: $profile.port, format: .number)
                     TextField("Benutzername", text: $profile.username)
+                        .autocorrectionDisabled()
+                    // Passwort direkt unter dem Benutzernamen (Keychain-gespeichert)
+                    if storedPasswordPresent && password.isEmpty {
+                        HStack {
+                            Image(systemName: "checkmark.circle.fill").foregroundStyle(.green)
+                            Text("Passwort gespeichert")
+                            Spacer()
+                            Button("Löschen", role: .destructive) { deleteStoredPassword() }
+                        }
+                    }
+                    SecureField(storedPasswordPresent ? "Neues Passwort …" : "Passwort", text: $password)
+                    Toggle("Passwort in Keychain speichern", isOn: $savePassword)
                     TextField("Auflösung (RDP, z. B. 1920x1080 oder Automatisch)", text: $profile.resolution)
                     TextField("Extra-Argumente", text: $profile.extraArgs)
+                    if !liveWarnings.isEmpty {
+                        ForEach(liveWarnings, id: \.self) { w in
+                            HStack(spacing: 6) {
+                                Image(systemName: "exclamationmark.triangle").foregroundStyle(.orange)
+                                Text(w).font(.caption).foregroundStyle(.orange)
                             }
-                Section("Authentifizierung") {
+                        }
+                    }
+                }
+                Section("SSH-Key (optional)") {
                     Picker("SSH-Key", selection: $profile.sshKeyID) {
-                        Text("Kein Key").tag(UUID?.none)
+                        Text("Kein Key (Passwort-Login)").tag(UUID?.none)
                         ForEach(store.sshKeys) { k in Text(k.name).tag(UUID?.some(k.id)) }
                     }
-                    SecureField("Passwort (optional, Keychain)", text: $password)
-                    Toggle("Passwort in Keychain speichern", isOn: $savePassword)
+                    Text(sshpassHint).font(.caption).foregroundStyle(.secondary)
                 }
             }
             .formStyle(.grouped)
@@ -54,7 +79,7 @@ struct ProfileEditorView: View {
                 Button("Abbrechen", role: .cancel) { dismiss() }
                 Spacer()
                 Button("Speichern") {
-                    var toSave = profile
+                    var toSave = ProfileValidation.cleaned(profile)
                     if toSave.port == 0 { toSave.port = toSave.proto.defaultPort }
                     if toSave.passwordAccount == nil { toSave.passwordAccount = "profile-\(toSave.id.uuidString)" }
                     if savePassword, !password.isEmpty, let acct = toSave.passwordAccount {
@@ -64,16 +89,35 @@ struct ProfileEditorView: View {
                     dismiss()
                 }
                 .keyboardShortcut(.defaultAction)
-                .disabled(profile.host.isEmpty || profile.name.isEmpty)
+                .disabled(profile.name.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty
+                          || profile.host.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty)
             }
             .padding()
         }
-        .frame(width: 520, height: 620)
+        .frame(width: 520, height: 680)
         .onAppear {
+            if let acct = profile.passwordAccount,
+               let stored = KeychainHelper.load(account: acct), !stored.isEmpty {
+                storedPasswordPresent = true
+                savePassword = true
+            }
+        }
+    }
+
+    var sshpassHint: String {
+        if SSHService.findSshpass() != nil {
+            return "sshpass gefunden → Auto-Login mit Keychain-Passwort aktiv, kein Tippen nötig."
+        } else {
+            return "Ohne sshpass: Passwort wird beim Verbinden im Terminal abgefragt. Für Auto-Login: brew install sshpass."
+        }
+    }
+
+    func deleteStoredPassword() {
         if let acct = profile.passwordAccount {
-                password = KeychainHelper.load(account: acct) ?? ""
-                savePassword = !password.isEmpty
-            }
+            KeychainHelper.delete(account: acct)
         }
+        password = ""
+        savePassword = false
+        storedPasswordPresent = false
     }
 }

Sources/RemminaMac/Views/TerminalInputField.swift

@@ -0,0 +1,119 @@
+import SwiftUI
+import AppKit
+
+/// Eingabefeld ohne macOS-Text-Ersetzungen (Anführungszeichen, Gedankenstriche,
+/// Auto-Korrektur, Punkt-Verdopplung, Vervollständigung, Links/Daten-Erkennung).
+/// Hintergrund: Normale Textfelder ersetzen still z. B. `"` durch „“ und `--`
+/// durch — (je nach Tastatur-Einstellung). Im Terminal fällt das nicht auf, aber
+/// ein Passwort mit Sonderzeichen kommt falsch bei ssh an → „Permission denied",
+/// obwohl das Passwort „stimmt" (docs/ERRORS.md #13).
+/// NSTextField kann das nicht alles abschalten, daher NSTextView-Basis.
+/// Secure-Modus nutzt NSSecureTextField (per Design ohne Ersetzungen).
+struct TerminalInputField: NSViewRepresentable {
+    @Binding var text: String
+    var isSecure: Bool
+    var onSubmit: () -> Void
+
+    func makeCoordinator() -> Coordinator {
+        Coordinator(text: $text, onSubmit: onSubmit)
+    }
+
+    func makeNSView(context: Context) -> NSView {
+        if isSecure {
+            let f = NSSecureTextField()
+            f.font = .monospacedSystemFont(ofSize: 13, weight: .regular)
+            f.textColor = Dracula.nsColor(for: nil)
+            f.backgroundColor = .clear
+            f.isBordered = false
+            f.focusRingType = .none
+            f.delegate = context.coordinator
+            return f
+        } else {
+            let tv = NSTextView()
+            Self.configure(tv)
+            tv.delegate = context.coordinator
+            return tv
+        }
+    }
+
+    func updateNSView(_ view: NSView, context: Context) {
+        context.coordinator.onSubmit = onSubmit
+        // 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
+        }
+    }
+
+    /// Alle Ersetzungen aus. Auch als Harness-Test nutzbar (reine Flags, headless).
+    static func configure(_ tv: NSTextView) {
+        tv.isRichText = false
+        tv.importsGraphics = false
+        tv.isAutomaticQuoteSubstitutionEnabled = false
+        tv.isAutomaticDashSubstitutionEnabled = false
+        tv.isAutomaticTextCompletionEnabled = false
+        tv.isAutomaticSpellingCorrectionEnabled = false
+        tv.isContinuousSpellCheckingEnabled = false
+        tv.isAutomaticDataDetectionEnabled = false
+        tv.isAutomaticLinkDetectionEnabled = false
+        tv.isAutomaticTextReplacementEnabled = false
+        tv.font = .monospacedSystemFont(ofSize: 13, weight: .regular)
+        tv.textColor = Dracula.nsColor(for: nil)
+        tv.backgroundColor = .clear
+        tv.drawsBackground = false
+        tv.isVerticallyResizable = false
+        tv.isHorizontallyResizable = true
+        tv.textContainer?.widthTracksTextView = false
+        tv.textContainer?.containerSize = NSSize(width: 100000, height: 24)
+    }
+
+    final class Coordinator: NSObject, NSTextFieldDelegate, NSTextViewDelegate {
+        var text: Binding<String>
+        var onSubmit: () -> Void
+
+        init(text: Binding<String>, onSubmit: @escaping () -> Void) {
+            self.text = text
+            self.onSubmit = onSubmit
+        }
+
+        func controlTextDidChange(_ obj: Notification) {
+            if let f = obj.object as? NSTextField {
+                text.wrappedValue = f.stringValue
+            }
+        }
+
+        func control(_ control: NSControl, textView: NSTextView, doCommandBy commandSelector: Selector) -> Bool {
+            if commandSelector == #selector(NSResponder.insertNewline(_:)) {
+                onSubmit()
+                return true
+            }
+            return false
+        }
+
+        func textDidChange(_ notification: Notification) {
+            if let tv = notification.object as? NSTextView {
+                // Einzeiler: harte Umbrüche sofort entfernen (Paste mit \n)
+                let single = tv.string.replacingOccurrences(of: "\n", with: "")
+                if single != tv.string { tv.string = single }
+                text.wrappedValue = single
+            }
+        }
+
+        func textView(_ textView: NSTextView, doCommandBy commandSelector: Selector) -> Bool {
+            if commandSelector == #selector(NSResponder.insertNewline(_:)) {
+                onSubmit()
+                return true
+            }
+            return false
+        }
+    }
+}

docs/ERRORS.md

@@ -17,6 +17,12 @@ Stand: 2026-09-21 · Version 0.1.0 · macOS arm64 · Swift 5.9+ · Xcode 15+
 | 8 | 2026-09-21 | SSH-Session stirbt instant ohne Meldung | stderr hing mit am PTY: macOS verwirft ungelesene Master-Daten bei Slave-Close → „Code 255" ohne Text | blockierend | ✅ behoben — stderr→Pipe, TCP-Vorab-Check, Fehler-Banner |
 | 9 | 2026-09-21 | `swift build` | `Result<Void, String>`: String conformt nicht zu Error (Folgefehler: DispatchQueue-Overload) | blockierend | ✅ behoben — Completion als `(String?) -> Void` |
 | 10 | 2026-09-21 | Icon-Generator (`RenderIcon.swift`) | 3 Swift-Fehler: `guard let x = f() ?? g()` (nicht optional), `Int * CGFloat`, falscher Preview-Pfad | klein | ✅ behoben |
+| 11 | 2026-09-21 | Passwort-Feature | Wunsch: Passwort + Keys speichern UND nutzen (bisher nur gespeichert, nie benutzt) | Feature | ✅ umgesetzt — Auto-Login via sshpass, sonst interaktiv |
+| 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) |
 
 ---
 
@@ -116,6 +122,16 @@ TEXT: "\n\n— Sitzung beendet (Code 255). —\n"   # <- Fehlermeldung fehlt!
 **Fix:** `TCPCheck.check` nutzt `(String?) -> Void` (nil = ok, sonst deutsche Meldung).
 **Status:** ✅ behoben.
 
+## #11 — Passwort speichern UND benutzen (User-Wunsch)
+**Stand:** Passwort/Key-Passphrase wurden in der Keychain gespeichert, aber nie zum Einloggen benutzt (ssh fragte immer interaktiv).
+**Umsetzung:**
+- `SSHService.loginSecret()` (Profil-Passwort → Key-Passphrase), `sessionConfig()` (rein, testbar): mit Secret + sshpass → `sshpass -e ssh/sftp …` + `SSHPASS`-Env (nie Befehlszeile/Logs); sonst direkter Aufruf + interaktiver Prompt.
+- `PTYSession.customEnv` + `autoLogin`-Flag, `TerminalManager` verdrahtet alles; Status zeigt „Auto-Login via Keychain".
+- Editor: Gespeichert-Anzeige (✓), Löschen-Button, sshpass-Hinweis; Titel-Bug (`UUID`-Vergleich mit neuem Random-Objekt) via `isNew` ersetzt.
+**Verifikation:** Config-Harness 9/9 (inkl. „Passwort nie in Befehlszeile"); echter Keychain-Roundtrip 7/7 (save/load/delete + Priorität + Fallback).
+**Hinweis:** Ohne `sshpass` (`brew install sshpass`) bleibt es beim interaktiven Prompt — hier auf dem Build-Mac ebenfalls nicht installiert, getestet wurde der interaktive Pfad + Logik.
+**Status:** ✅ umgesetzt.
+
 ## #10 — Icon-Generator Buildfehler
 **Log:** `guard let` auf nicht-optionalem Ausdruck (`??` unwrapt schon); `Int * CGFloat`; Preview lag in `.iconset/` statt `/tmp/`.
 **Fix:** `??`-Fallback direkt zuweisen; `CGFloat(...)`; Pfad korrigiert.
@@ -153,3 +169,52 @@ Oder mit Token als Passwort: `git push https://<owner>:<token>@git.sysdaemon.xyz
 **Lokaler Stand:** Commit `cd834dc` + `dist/RemminaMac-0.1.0-macos-arm64.zip` sind fertig und warteten nur auf den Push.
 **Nachtrag 2026-09-21:** ✅ behoben — mit korrigierten Credentials: `GET /api/v1/user → 200 (login x3)`, Repo `x3/remmina-mac` erstellt (201), `git push -u origin main` ok, Release `v0.1.0` (id 1) + 2 Assets (ZIP + SHA256) hochgeladen.
 **Status:** ✅ behoben.
+
+## #12 — „Immer Verbindung falsch" (User-Report, Details offen)
+**Stand:** Gemeldet ohne Ziel-Host, Banner-Text oder Credentials — aus der Ferne nicht reproduzierbar. Systematische App-Fehler als Ursache weitgehend ausgeschlossen (PTY/ssh-Pfad, Fehlertext-Anzeige, TCP-Check und Keychain-Fluss sind per Harness verifiziert).
+**Häufigste reale Ursachen (Checkliste):**
+1. `user@host` im Host-Feld oder `ssh://`-Präfix → jetzt Live-Warnung im Editor
+2. Leerzeichen/Vertipper in Host/User → wird jetzt getrimmt + gewarnt
+3. Falsches Passwort / Key ohne Passphrase-Eintrag → Banner + Diagnose nutzen
+4. Server blockt (Fail2ban, nur Key-Login erlaubt) → Server-Logs prüfen
+**Umgesetzt:** `Models/ProfileValidation.swift` (trimmen + Warnungen, Harness 7/7), Passwort direkt unter Benutzername, „Bericht kopieren" im Fehler-Banner (ohne Secrets).
+**Offen:** Bitte Banner-Text oder kopierten Bericht hierher schicken — dann steht die Ursache in einer Zeile.
+**Status:** ⬜ wartet auf Fehlerbericht.
+
+## #13 — Permission denied trotz richtigem Passwort (User-Fehlerbericht)
+**Bericht:** `administrator@sysdaemon.xyz:22`, ssh fragt 3x Passwort → `Permission denied (publickey,password)`. Gleiches Passwort im Terminal.app funktioniert.
+**Ursache:** Das Terminal-Eingabefeld war ein Standard-TextField. macOS ersetzt dort je nach Einstellung still Zeichen (`"` → „“, `--` → —, Auto-Korrektur). Ein Passwort mit Sonderzeichen kommt dadurch falsch bei ssh an — im Terminal fällt nichts auf, ssh sagt nur „denied". Verifiziert: Standard-`NSTextField` hat diese Schalter an bzw. `NSTextField` kann Dash-Ersetzung gar nicht abschalten (Compiler-Beweis).
+**Fix (`Views/TerminalInputField.swift`):** Eigenes Eingabefeld auf `NSTextView`-Basis mit allen Ersetzungen explizit aus (Quotes, Dashes, Completion, Spelling, Data/Link-Erkennung, Text-Replacement, Plain-Text); Secure-Variante via `NSSecureTextField` (per Design ersetzungsfrei); Enter → Senden, Paste-Normalisierung (keine Umbrüche).
+**Verifikation:** Harness 9/9 (alle Flags aus). Hinweis: Restrisiko Tastaturlayout (Z/Y) bleibt — Banner-Tipp ergänzt.
+**Status:** ✅ behoben — bitte mit v0.5.1 erneut testen.
+
+## #14 — Terminal.app loggt ein, RemminaMac nicht (gleiches Passwort)
+**Stand (User-Antworten):** v0.5.1 installiert, Passwort-Login, Terminal.app funktioniert mit denselben Daten.
+**Analyse:** App-seitig ist alles verifiziert (PTY, Fehleranzeige, Passwort-Loop mit Sonderzeichen, Eingabe-Ersetzungen aus). Gleiches Binary + gleiche Config → Unterschied muss aus der Umgebung kommen. Hauptkandidat: **SSH-Agent** — Finder-gestartete GUI-Prozesse erben `SSH_AUTH_SOCK` nicht. Nutzt Terminal unbemerkt einen Agent-Key (statt Passwort), scheitert die App mit Passwort. Verifiziert: `launchctl getenv SSH_AUTH_SOCK` liefert den Socket, Shell-Env wäre ohne Abfrage leer für GUI-Apps.
+**Fix:** `SSHService.agentSocket()` (launchctl + Existenzprüfung) → `TerminalManager` injiziert `SSH_AUTH_SOCK` ins ssh-Env (koexistiert mit SSHPASS). Banner-Tipp ergänzt.
+**Verifikation:** Harness 4/4 (Auflösung 0.07 s, Pfad existiert, `ssh -G` ok mit echtem + kaputtem Socket).
+**Gegenprobe beim User (bitte ausführen):**
+```bash
+ssh-add -l
+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).

docs/USAGE.md

@@ -7,10 +7,11 @@
 - **Toolbar:** `+` Neu, Stift Bearbeiten, Kopie-Symbol Duplizieren, Schlüssel Keys, `▶` Verbinden, `⚡` Quick-Connect.
 
 ## Profil anlegen
-1. `+` → Name, Gruppe, Protokoll (SSH/SFTP/RDP/VNC), Host, Port, User ausfüllen.
-2. SSH-Key wählen (vorher unter Schlüssel-Icon importieren).
-3. Optional Passwort → Haken „in Keychain speichern“.
+1. `+` → Name, Gruppe, Protokoll (SSH/SFTP/RDP/VNC), Host, Port, Benutzername, **Passwort direkt darunter** ausfüllen (Keychain-Haken setzen zum Speichern).
+2. Der Editor warnt live bei typischen Fehlern: `user@host` im Host-Feld, `ssh://`-Präfix, Leerzeichen, leerer Benutzername, ungültiger Port. Beim Speichern wird getrimmt.
+3. SSH-Key wählen (optional, Bereich darunter — vorher unter Schlüssel-Icon importieren).
 4. Speichern → liegt sofort in `profiles.json`.
+5. Schlägt eine Verbindung fehl: rotes Banner → **Bericht kopieren** und zur Fehlersuche mitschicken (enthält kein Passwort).
 
 ## Verbinden
 - Profil wählen → `▶ Verbinden` oder Doppelklick.
@@ -25,6 +26,12 @@
 - `Neuen Key generieren`: ruft `/usr/bin/ssh-keygen -t ed25519 -f ~/.ssh/remminamac_<timestamp>` auf.
 - Passphrase → Keychain (`sshkey-<UUID>`).
 
+## Passwort speichern & Auto-Login
+- Im Profil-Editor: Passwort eingeben + „in Keychain speichern“. Gespeichertes Passwort wird angezeigt (✓) und kann gelöscht werden. In `profiles.json` steht nur die Referenz, nie das Passwort.
+- Reihenfolge beim Verbinden: Profil-Passwort zuerst, sonst Key-Passphrase.
+- **Auto-Login ohne Tippen:** braucht `sshpass` (`brew install sshpass`). Dann loggt ssh via Keychain-Secret ein (Passwort nur im Prozess-Env `SSHPASS`, nie in Befehlszeile/Logs). Status zeigt „Auto-Login via Keychain".
+- **Ohne sshpass:** Passwort/Key-Passphrase wird interaktiv im Terminal abgefragt (Passwort-Modus: Auge-Icon). Bei Sonderzeichen im Passwort immer Auge-Modus nutzen (keine Text-Ersetzung).
+
 ## Quick-Connect
 - Mit Auswahl: verbindet direkt. Ohne Auswahl: öffnet Editor mit „Quick-Connect“-Vorlage.