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

This commit is contained in:
x3
2026-09-22 08:46:45 +02:00
parent 69f9e9952c
commit 7a792fe680
2 changed files with 30 additions and 4 deletions
+23 -4
View File
@@ -25,6 +25,8 @@ final class PTYSession: ObservableObject {
var executable = "" var executable = ""
var arguments: [String] = [] var arguments: [String] = []
var customEnv: [String: String] = [:] var customEnv: [String: String] = [:]
/// Fingerprint der Konfiguration (TerminalManager) — bei Änderung wird neu konfiguriert.
var configTag = ""
private var masterFD: Int32 = -1 private var masterFD: Int32 = -1
private var process: Process? private var process: Process?
@@ -288,8 +290,27 @@ final class TerminalManager: ObservableObject {
private var sessions: [UUID: PTYSession] = [:] private var sessions: [UUID: PTYSession] = [:]
func session(for profile: ConnectionProfile, key: StoredSSHKey?) -> 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() 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 baseExe: String
let baseArgs: [String] let baseArgs: [String]
if profile.proto == .sftp { if profile.proto == .sftp {
@@ -302,7 +323,7 @@ final class TerminalManager: ObservableObject {
// Auto-Login nur mit Keychain-Secret + sshpass, sonst interaktiver Prompt im Terminal. // Auto-Login nur mit Keychain-Secret + sshpass, sonst interaktiver Prompt im Terminal.
// SSH-Agent durchreichen (falls vorhanden): gleiche Keys wie in Terminal.app. // SSH-Agent durchreichen (falls vorhanden): gleiche Keys wie in Terminal.app.
let cfg = SSHService.sessionConfig(executable: baseExe, args: baseArgs, let cfg = SSHService.sessionConfig(executable: baseExe, args: baseArgs,
password: SSHService.loginSecret(for: profile, key: key), password: secret,
sshpassPath: SSHService.findSshpass()) sshpassPath: SSHService.findSshpass())
s.configure(executable: cfg.executable, arguments: cfg.arguments) s.configure(executable: cfg.executable, arguments: cfg.arguments)
var env = cfg.extraEnv var env = cfg.extraEnv
@@ -311,8 +332,6 @@ final class TerminalManager: ObservableObject {
} }
s.customEnv = env s.customEnv = env
s.setAutoLogin(cfg.autoLogin) s.setAutoLogin(cfg.autoLogin)
sessions[profile.id] = s
return s
} }
func close(_ profile: ConnectionProfile) { func close(_ profile: ConnectionProfile) {
+7
View File
@@ -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) | | 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 | | 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 | | 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 - `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. **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. **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).