Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
4209c481dc | ||
|
|
8511e4fdae | ||
|
|
7a792fe680 |
+7
-1
@@ -7,12 +7,18 @@ let package = Package(
|
||||
.macOS(.v14)
|
||||
],
|
||||
products: [
|
||||
.executable(name: "RemminaMac", targets: ["RemminaMac"])
|
||||
.executable(name: "RemminaMac", targets: ["RemminaMac"]),
|
||||
.executable(name: "ctty-run", targets: ["ctty-run"])
|
||||
],
|
||||
targets: [
|
||||
.executableTarget(
|
||||
name: "RemminaMac",
|
||||
path: "Sources/RemminaMac"
|
||||
),
|
||||
.executableTarget(
|
||||
name: "ctty-run",
|
||||
path: "Sources/ctty-run",
|
||||
sources: ["ctty-run.c"]
|
||||
)
|
||||
]
|
||||
)
|
||||
|
||||
@@ -0,0 +1,148 @@
|
||||
import Foundation
|
||||
|
||||
/// Host-Key-Prüfung OHNE interaktiven Yes/No-Prompt (der kommt ohne echtes
|
||||
/// Terminal/Ctty nicht an — ssh bricht dann mit „Host key verification failed." ab).
|
||||
/// Ablauf: ssh-keyscan → Fingerprint → Popup (akzeptieren/ablehnen) → known_hosts.
|
||||
/// Nutzt bewusst ~/.ssh/known_hosts gemeinsam mit Terminal.app.
|
||||
enum HostKeyStatus: Equatable {
|
||||
case known
|
||||
case unknown(HostKeyInfo)
|
||||
case changed(HostKeyInfo)
|
||||
case unreachable(String)
|
||||
}
|
||||
|
||||
struct HostKeyInfo: Equatable {
|
||||
var host: String
|
||||
var port: Int
|
||||
var keyType: String
|
||||
var fingerprintSHA256: String
|
||||
var fingerprintMD5: String
|
||||
var allKeyLines: [String]
|
||||
}
|
||||
|
||||
enum HostKeyService {
|
||||
static var knownHostsPath = NSHomeDirectory() + "/.ssh/known_hosts"
|
||||
static var scanTimeoutSeconds = "5"
|
||||
|
||||
static func canonicalHost(_ host: String, port: Int) -> String {
|
||||
port == 22 ? host : "[\(host)]:\(port)"
|
||||
}
|
||||
|
||||
@discardableResult
|
||||
static func run(_ exe: String, _ args: [String]) -> (Int32, String) {
|
||||
let p = Process()
|
||||
p.executableURL = URL(fileURLWithPath: exe)
|
||||
p.arguments = args
|
||||
let out = Pipe()
|
||||
p.standardOutput = out
|
||||
p.standardError = Pipe()
|
||||
do {
|
||||
try p.run()
|
||||
} catch {
|
||||
return (-1, "")
|
||||
}
|
||||
p.waitUntilExit()
|
||||
let data = out.fileHandleForReading.readDataToEndOfFile()
|
||||
return (p.terminationStatus, String(data: data, encoding: .utf8) ?? "")
|
||||
}
|
||||
|
||||
/// Rohe Key-Zeilen vom Server (ohne #-Kommentare).
|
||||
static func keyscan(host: String, port: Int) -> [String] {
|
||||
let (code, out) = run("/usr/bin/ssh-keyscan", ["-T", scanTimeoutSeconds, "-p", "\(port)", host])
|
||||
guard code == 0 else { return [] }
|
||||
return out.split(separator: "\n").map(String.init).filter { line in
|
||||
let t = line.trimmingCharacters(in: .whitespaces)
|
||||
return !t.isEmpty && !t.hasPrefix("#") && t.contains(" ")
|
||||
}
|
||||
}
|
||||
|
||||
static func knownHostsContent() -> String {
|
||||
(try? String(contentsOfFile: knownHostsPath, encoding: .utf8)) ?? ""
|
||||
}
|
||||
|
||||
static func status(host: String, port: Int) -> HostKeyStatus {
|
||||
let h = host.trimmingCharacters(in: .whitespacesAndNewlines)
|
||||
guard !h.isEmpty else { return .unreachable("Kein Host.") }
|
||||
let lines = keyscan(host: h, port: port)
|
||||
guard !lines.isEmpty else {
|
||||
return .unreachable("Host-Key nicht abfragbar (ssh-keyscan ohne Antwort).")
|
||||
}
|
||||
guard let info = info(host: h, port: port, lines: lines) else {
|
||||
return .unreachable("Fingerprint nicht berechenbar.")
|
||||
}
|
||||
let (fcode, fout) = run("/usr/bin/ssh-keygen", ["-F", canonicalHost(h, port: port), "-f", knownHostsPath])
|
||||
let hasEntry = fcode == 0 && fout.contains("found:")
|
||||
if hasEntry {
|
||||
let knownKeys = lines.compactMap { $0.split(separator: " ").last.map(String.init) }
|
||||
if knownKeys.contains(where: { knownHostsContent().contains($0) }) {
|
||||
return .known
|
||||
}
|
||||
return .changed(info)
|
||||
}
|
||||
return .unknown(info)
|
||||
}
|
||||
|
||||
/// Bevorzugter Key (ed25519 > ecdsa > Rest) + Fingerprints via ssh-keygen.
|
||||
static func info(host: String, port: Int, lines: [String]) -> HostKeyInfo? {
|
||||
func rank(_ l: String) -> Int {
|
||||
if l.contains("ssh-ed25519") { return 0 }
|
||||
if l.contains("ecdsa-") { return 1 }
|
||||
return 2
|
||||
}
|
||||
guard let line = lines.sorted(by: { rank($0) < rank($1) }).first else { return nil }
|
||||
let parts = line.split(separator: " ")
|
||||
guard parts.count >= 3 else { return nil }
|
||||
let tmp = FileManager.default.temporaryDirectory.appendingPathComponent("rmm-fp-\(UUID().uuidString)")
|
||||
do {
|
||||
try (line + "\n").write(to: tmp, atomically: true, encoding: .utf8)
|
||||
defer { try? FileManager.default.removeItem(at: tmp) }
|
||||
let (_, sha) = run("/usr/bin/ssh-keygen", ["-l", "-f", tmp.path])
|
||||
let (_, md5) = run("/usr/bin/ssh-keygen", ["-l", "-E", "md5", "-f", tmp.path])
|
||||
return HostKeyInfo(host: host, port: port, keyType: String(parts[1]),
|
||||
fingerprintSHA256: parseFP(sha), fingerprintMD5: parseFP(md5),
|
||||
allKeyLines: lines)
|
||||
} catch {
|
||||
return nil
|
||||
}
|
||||
}
|
||||
|
||||
/// Aus `256 SHA256:XYZ host (ED25519)` wird `SHA256:XYZ`.
|
||||
static func parseFP(_ out: String) -> String {
|
||||
let parts = out.trimmingCharacters(in: .whitespacesAndNewlines).split(separator: " ")
|
||||
guard parts.count >= 2 else { return out.trimmingCharacters(in: .whitespacesAndNewlines) }
|
||||
return String(parts[1])
|
||||
}
|
||||
|
||||
/// Fehlende Key-Zeilen an known_hosts anhängen (0600, ~/.ssh 0700).
|
||||
static func accept(_ info: HostKeyInfo) -> Bool {
|
||||
let dir = (knownHostsPath as NSString).deletingLastPathComponent
|
||||
do {
|
||||
try FileManager.default.createDirectory(atPath: dir, withIntermediateDirectories: true,
|
||||
attributes: [.posixPermissions: 0o700])
|
||||
let existing = knownHostsContent()
|
||||
let missing = info.allKeyLines.filter { line in
|
||||
let key = line.split(separator: " ").last.map(String.init) ?? line
|
||||
return !existing.contains(key)
|
||||
}
|
||||
guard !missing.isEmpty else { return true }
|
||||
if !FileManager.default.fileExists(atPath: knownHostsPath) {
|
||||
FileManager.default.createFile(atPath: knownHostsPath, contents: nil,
|
||||
attributes: [.posixPermissions: 0o600])
|
||||
}
|
||||
let fh = try FileHandle(forWritingTo: URL(fileURLWithPath: knownHostsPath))
|
||||
defer { try? fh.close() }
|
||||
try fh.seekToEnd()
|
||||
fh.write(((missing.joined(separator: "\n")) + "\n").data(using: .utf8)!)
|
||||
try FileManager.default.setAttributes([.posixPermissions: 0o600], ofItemAtPath: knownHostsPath)
|
||||
return true
|
||||
} catch {
|
||||
return false
|
||||
}
|
||||
}
|
||||
|
||||
/// Alle Einträge für Host entfernen (ssh-keygen -R).
|
||||
static func revoke(host: String, port: Int) -> Bool {
|
||||
let (code, _) = run("/usr/bin/ssh-keygen", ["-R", canonicalHost(host, port: port), "-f", knownHostsPath])
|
||||
return code == 0
|
||||
}
|
||||
}
|
||||
@@ -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?
|
||||
@@ -78,10 +80,22 @@ final class PTYSession: ObservableObject {
|
||||
}
|
||||
masterFD = master
|
||||
|
||||
// Ctty-Helper (gleiches Verzeichnis wie dieses Binary — .app wie .build):
|
||||
// Gibt ssh ein echtes Controlling-Terminal, damit Passwort- und
|
||||
// Yes/No-Prompts funktionieren (/dev/tty statt stiller Abbrüche).
|
||||
// Ohne Helper: Legacy-Direktstart (Prompts teils stumm).
|
||||
let pipe = Pipe()
|
||||
let p = Process()
|
||||
if let helper = Self.cttyHelperURL() {
|
||||
p.executableURL = helper
|
||||
p.arguments = [slavePath, executable] + arguments
|
||||
} else {
|
||||
p.executableURL = URL(fileURLWithPath: executable)
|
||||
p.arguments = arguments
|
||||
DispatchQueue.main.async { [weak self] in
|
||||
self?.note("Hinweis: ohne ctty-run-Helper (eingeschränkte Prompts).")
|
||||
}
|
||||
}
|
||||
var env = ProcessInfo.processInfo.environment
|
||||
env["TERM"] = "xterm-256color"
|
||||
if env["LANG"] == nil { env["LANG"] = "de_DE.UTF-8" }
|
||||
@@ -223,6 +237,17 @@ final class PTYSession: ObservableObject {
|
||||
lines = [[]]
|
||||
}
|
||||
|
||||
/// ctty-run liegt neben diesem Binary (Contents/MacOS wie .build/debug).
|
||||
static func cttyHelperURL() -> URL? {
|
||||
let sibling = URL(fileURLWithPath: CommandLine.arguments[0])
|
||||
.deletingLastPathComponent()
|
||||
.appendingPathComponent("ctty-run")
|
||||
if FileManager.default.isExecutableFile(atPath: sibling.path) {
|
||||
return sibling
|
||||
}
|
||||
return nil
|
||||
}
|
||||
|
||||
func note(_ message: String) {
|
||||
lines.append([StyledText(text: message, style: TermStyle(fg: 8))])
|
||||
}
|
||||
@@ -239,7 +264,7 @@ final class PTYSession: ObservableObject {
|
||||
|
||||
func stop() {
|
||||
let wasRunning = isRunning
|
||||
process?.terminate()
|
||||
killTree()
|
||||
cleanupSync()
|
||||
if wasRunning {
|
||||
epoch += 1 // späte terminationHandler-Ausläufer verwerfen
|
||||
@@ -250,6 +275,14 @@ final class PTYSession: ObservableObject {
|
||||
}
|
||||
}
|
||||
|
||||
/// Beendet Kind + Enkel (ctty-run-Wächter + ssh teilen die Prozessgruppe).
|
||||
private func killTree() {
|
||||
if let pid = process?.processIdentifier, pid > 0 {
|
||||
Darwin.kill(-pid, SIGTERM)
|
||||
}
|
||||
process?.terminate()
|
||||
}
|
||||
|
||||
/// Synchroner Cleanup auf Main: Source canceln, Master schließen, Pipe abklemmen.
|
||||
/// Laufende ioQueue-Reads bekommen EBADF und beenden sich selbst.
|
||||
private func cleanupSync() {
|
||||
@@ -276,6 +309,9 @@ final class PTYSession: ObservableObject {
|
||||
}
|
||||
|
||||
deinit {
|
||||
if let pid = process?.processIdentifier, pid > 0 {
|
||||
Darwin.kill(-pid, SIGTERM)
|
||||
}
|
||||
process?.terminate()
|
||||
readSource?.cancel()
|
||||
if masterFD >= 0 { close(masterFD) }
|
||||
@@ -288,8 +324,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 +357,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 +366,6 @@ final class TerminalManager: ObservableObject {
|
||||
}
|
||||
s.customEnv = env
|
||||
s.setAutoLogin(cfg.autoLogin)
|
||||
sessions[profile.id] = s
|
||||
return s
|
||||
}
|
||||
|
||||
func close(_ profile: ConnectionProfile) {
|
||||
|
||||
@@ -11,6 +11,9 @@ struct EmbeddedTerminalView: View {
|
||||
@State private var input = ""
|
||||
@State private var secureInput = false
|
||||
@State private var checking = false
|
||||
@State private var hostKeyInfo: HostKeyInfo?
|
||||
@State private var hostKeyChange = false
|
||||
@State private var showingHostKey = false
|
||||
|
||||
var body: some View {
|
||||
VStack(spacing: 0) {
|
||||
@@ -98,6 +101,18 @@ struct EmbeddedTerminalView: View {
|
||||
connect() // Auto-Connect beim Öffnen des Tabs
|
||||
}
|
||||
}
|
||||
.sheet(isPresented: $showingHostKey) {
|
||||
if let info = hostKeyInfo {
|
||||
HostKeySheet(info: info, isChange: hostKeyChange,
|
||||
onAccept: acceptHostKey,
|
||||
onCancel: {
|
||||
showingHostKey = false
|
||||
checking = false
|
||||
hostKeyInfo = nil
|
||||
term.fail("Host-Key nicht akzeptiert — Verbindung abgebrochen.")
|
||||
})
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
func connect() {
|
||||
@@ -105,16 +120,47 @@ struct EmbeddedTerminalView: View {
|
||||
checking = true
|
||||
TCPCheck.check(host: profile.host, port: profile.port) { message in
|
||||
DispatchQueue.main.async {
|
||||
checking = false
|
||||
if let message {
|
||||
checking = false
|
||||
term.fail("Vorab-Check: \(message)")
|
||||
} else {
|
||||
return
|
||||
}
|
||||
// Host-Key vorab per keyscan prüfen (Yes/No-Prompt käme ohne Ctty nicht an)
|
||||
let hk = HostKeyService.status(host: profile.host, port: profile.port)
|
||||
switch hk {
|
||||
case .known, .unreachable:
|
||||
checking = false
|
||||
term.start()
|
||||
case .unknown(let info):
|
||||
hostKeyInfo = info
|
||||
hostKeyChange = false
|
||||
showingHostKey = true
|
||||
case .changed(let info):
|
||||
hostKeyInfo = info
|
||||
hostKeyChange = true
|
||||
showingHostKey = true
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
func acceptHostKey() {
|
||||
guard let info = hostKeyInfo else { return }
|
||||
showingHostKey = false
|
||||
if hostKeyChange {
|
||||
_ = HostKeyService.revoke(host: profile.host, port: profile.port)
|
||||
}
|
||||
if HostKeyService.accept(info) {
|
||||
checking = false
|
||||
term.note("Host-Key akzeptiert (\(info.keyType), \(info.fingerprintSHA256.prefix(19))…).")
|
||||
term.start()
|
||||
} else {
|
||||
checking = false
|
||||
term.fail("Host-Key konnte nicht in known_hosts gespeichert werden.")
|
||||
}
|
||||
hostKeyInfo = nil
|
||||
}
|
||||
|
||||
func diagnose() {
|
||||
guard !checking else { return }
|
||||
checking = true
|
||||
|
||||
@@ -0,0 +1,57 @@
|
||||
import SwiftUI
|
||||
|
||||
/// Popup bei unbekanntem/geändertem Host-Key: Fingerprint prüfen, dann
|
||||
/// akzeptieren (→ known_hosts + verbinden) oder abbrechen.
|
||||
struct HostKeySheet: View {
|
||||
var info: HostKeyInfo
|
||||
var isChange: Bool
|
||||
var onAccept: () -> Void
|
||||
var onCancel: () -> Void
|
||||
|
||||
var body: some View {
|
||||
VStack(alignment: .leading, spacing: 12) {
|
||||
HStack(spacing: 10) {
|
||||
Image(systemName: isChange ? "exclamationmark.octagon.fill" : "key.fill")
|
||||
.font(.title)
|
||||
.foregroundStyle(isChange ? Dracula.red : Dracula.orange)
|
||||
VStack(alignment: .leading) {
|
||||
Text(isChange ? "Host-Key GEÄNDERT" : "Neuer Host-Key")
|
||||
.font(.headline)
|
||||
Text("\(info.host):\(info.port) · \(info.keyType)")
|
||||
.font(.subheadline).foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
|
||||
if isChange {
|
||||
Text("Warnung: Der gespeicherte Schlüssel passt NICHT zum Server. Das kann ein neu aufgesetzter Server sein — oder ein Angriff (MITM). Nur fortfahren, wenn du die Änderung erwartest!")
|
||||
.font(.callout).foregroundStyle(Dracula.red)
|
||||
} else {
|
||||
Text("Erstverbindung: Dieser Server ist noch unbekannt. Fingerprint prüfen (z. B. mit dem Server-Admin oder einer Terminal-Verbindung vergleichen), dann akzeptieren.")
|
||||
.font(.callout).foregroundStyle(.secondary)
|
||||
}
|
||||
|
||||
GroupBox("SHA256-Fingerprint") {
|
||||
Text(info.fingerprintSHA256)
|
||||
.font(.system(.body, design: .monospaced))
|
||||
.textSelection(.enabled)
|
||||
.frame(maxWidth: .infinity, alignment: .leading)
|
||||
}
|
||||
GroupBox("MD5-Fingerprint") {
|
||||
Text(info.fingerprintMD5)
|
||||
.font(.system(.callout, design: .monospaced))
|
||||
.foregroundStyle(.secondary)
|
||||
.textSelection(.enabled)
|
||||
.frame(maxWidth: .infinity, alignment: .leading)
|
||||
}
|
||||
|
||||
HStack {
|
||||
Button("Abbrechen", role: .cancel, action: onCancel)
|
||||
Spacer()
|
||||
Button(isChange ? "Trotzdem ersetzen & verbinden" : "Akzeptieren & verbinden", action: onAccept)
|
||||
.keyboardShortcut(.defaultAction)
|
||||
}
|
||||
}
|
||||
.padding(20)
|
||||
.frame(width: 480)
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,58 @@
|
||||
// ctty-run: gibt dem Kindprozess ein Controlling-Terminal auf dem PTY-Slave.
|
||||
// Usage: ctty-run <slave-pfad> <programm> [args...]
|
||||
// stdin/stdout zeigen dabei bereits auf den Slave (vom Parent gesetzt),
|
||||
// stderr bleibt unberührt (Eltern-Pipe für garantierte Fehlerausgabe).
|
||||
//
|
||||
// Warum Double-Fork: Apples Process startet Kinder in eigener Prozessgruppe,
|
||||
// daher schlägt setsid() im direkten Kind mit EPERM fehl. Der Wächter forkt
|
||||
// einmal weiter (Enkel = kein Leader mehr), wartet auf ihn und übernimmt
|
||||
// dessen Exit-Code — Lifecycle/Exit-Codes bleiben für den Parent korrekt.
|
||||
// Der Enkel: setsid() -> Slave einmal öffnen (wird dadurch Ctty) -> exec.
|
||||
// Ohne Ctty scheitern ssh-Passwort- und Yes/No-Prompts still (/dev/tty ENXIO).
|
||||
#include <stdio.h>
|
||||
#include <stdlib.h>
|
||||
#include <string.h>
|
||||
#include <unistd.h>
|
||||
#include <fcntl.h>
|
||||
#include <errno.h>
|
||||
#include <sys/wait.h>
|
||||
|
||||
int
|
||||
main(int argc, char **argv)
|
||||
{
|
||||
if (argc < 3) {
|
||||
dprintf(STDERR_FILENO, "usage: ctty-run <slave> <prog> [args...]\n");
|
||||
return 2;
|
||||
}
|
||||
pid_t pid = fork();
|
||||
if (pid < 0) {
|
||||
dprintf(STDERR_FILENO, "ctty-run: fork: %s\n", strerror(errno));
|
||||
return 1;
|
||||
}
|
||||
if (pid > 0) {
|
||||
/* Wächter: keine FDs offen halten (sonst nie EOF), aufs Kind warten. */
|
||||
close(STDIN_FILENO);
|
||||
close(STDOUT_FILENO);
|
||||
close(STDERR_FILENO);
|
||||
int st = 0;
|
||||
while (waitpid(pid, &st, 0) < 0 && errno == EINTR)
|
||||
;
|
||||
if (WIFEXITED(st))
|
||||
return WEXITSTATUS(st);
|
||||
return 128;
|
||||
}
|
||||
/* Enkel: eigene Session; danach Slave-Open = Ctty. */
|
||||
if (setsid() < 0) {
|
||||
dprintf(STDERR_FILENO, "ctty-run: setsid: %s (weiter ohne Ctty)\n",
|
||||
strerror(errno));
|
||||
}
|
||||
int fd = open(argv[1], O_RDWR);
|
||||
if (fd < 0) {
|
||||
dprintf(STDERR_FILENO, "ctty-run: open slave: %s\n", strerror(errno));
|
||||
_exit(1);
|
||||
}
|
||||
close(fd); /* Ctty bleibt der Session erhalten */
|
||||
execvp(argv[2], &argv[2]);
|
||||
dprintf(STDERR_FILENO, "ctty-run: execvp: %s\n", strerror(errno));
|
||||
_exit(127);
|
||||
}
|
||||
@@ -22,6 +22,9 @@ 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) |
|
||||
| 17 | 2026-09-22 | rzsv: Vorab-Check Timeout | Port 22 wird serverseitig gedroppt (443 ok) — kein App-Fehler | extern | ⬜ Server-Seite: fail2ban/VPN/Firewall prüfen |
|
||||
| 18 | 2026-09-22 | Host-Key-Yes/No unmöglich | ssh ohne Controlling-Terminal: /dev/tty ENXIO → stummer Abbruch | kritisch | ✅ ctty-run-Helper (Double-Fork) + Fingerprint-Popup |
|
||||
|
||||
---
|
||||
|
||||
@@ -211,3 +214,30 @@ 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).
|
||||
|
||||
## #17 — rzsv: Vorab-Check Timeout (kein App-Fehler, Screenshot-Beweis)
|
||||
**Bild:** Tab rzsv (`administrator@sysdaemon.xyz:22`), Banner „Vorab-Check: Keine Antwort in 4 s", ssh wird korrekt NICHT gestartet.
|
||||
**Messung (gleiche Maschine, 22.09.):**
|
||||
- `sysdaemon.xyz` → `185.117.250.190` (auch `git.sysdaemon.xyz`)
|
||||
- Port 443: OK in 0.02 s — Port 22: Timeout (DROP, kein Refused)
|
||||
- Fazit: Server erreichbar, aber SSH-Port wird aus diesem Netz gedroppt. Von hier kann ssh in KEINEM Client gehen (Terminal eingeschlossen) — kein Terminal-/App-Problem, der Vorab-Check hatte recht.
|
||||
**Wahrscheinlichste Ursachen (Reihenfolge):**
|
||||
1. **fail2ban-Bann** nach den vielen `Permission denied`-Versuchen (`#13`-Ära): `fail2ban-client status sshd` → `unbanip <eigene-IP>`
|
||||
2. **VPN an/aus:** „Terminal geht" evtl. mit VPN (Split-DNS/Intern), App ohne (oder umgekehrt) — Zustände vergleichen
|
||||
3. Firewall lässt 22 nur aus bestimmten Netzen zu (`ufw status`, `iptables -L`, `ss -tlnp | grep 22` auf dem Server)
|
||||
**Gegenprobe (10 s, Terminal):** `ssh -v -o ConnectTimeout=8 administrator@sysdaemon.xyz` bleibt bei `Connecting...` hängen → bewiesen netzseitig.
|
||||
**Status:** ⬜ Server-/Netz-Seite, warte auf Gegenprobe + VPN-Status.
|
||||
|
||||
## #18 — Host-Key-Yes/No unmöglich (User-Screenshot: Terminal geht, App sagt nur „Host key verification failed.")
|
||||
**Analyse:** Terminal zeigt den `Are you sure...?`-Prompt (mit Fingerprint), App nur den Abbruch ohne Prompt. Beweis per `-v`: `read_passphrase: can't open /dev/tty: Device not configured` — ssh-Prompts brauchen ein Controlling-Terminal, unser PTY-Slave war keins (GUI-App hat keins, Kind erbt keins). Betrifft Passwort- UND Yes/No-Prompts.
|
||||
**Fix:**
|
||||
- `Sources/ctty-run/ctty-run.c` (C-Helper, SPM-Target): Double-Fork (nötig, weil Apples Process Kinder in eigener Prozessgruppe startet → direktes setsid = EPERM) → setsid → Slave-Open (= Ctty) → exec ssh. stderr bleibt auf der Eltern-Pipe (verlustfrei). `stop()` killt die Prozessgruppe (keine Waisen, verifiziert).
|
||||
- `Views/HostKeySheet.swift` + Flow: Vorab per `ssh-keyscan` Fingerprint holen → Popup (SHA256/MD5, Typ) → Akzeptieren (→ `~/.ssh/known_hosts`, geteilt mit Terminal) / Abbrechen; bei GEÄNDERTEM Key rote MITM-Warnung + `ssh-keygen -R`-Pfad.
|
||||
**Verifikation (lokaler paramiko-Server):** Bug reproduziert (kein Prompt, Exit 255) → nach Fix: Ctty per `ps` bewiesen (ttys001), Yes/No sichtbar + beantwortbar, Passwort akzeptiert, Shell aktiv. HostKeyService: unknown/accept/known/changed/revoke/unreachable grün. Regression (stop ohne Waisen) grün.
|
||||
**Status:** ✅ behoben (v0.6.0) — Fingerprint im Popup mit Terminal-Ausgabe vergleichbar.
|
||||
|
||||
@@ -26,6 +26,11 @@
|
||||
- `Neuen Key generieren`: ruft `/usr/bin/ssh-keygen -t ed25519 -f ~/.ssh/remminamac_<timestamp>` auf.
|
||||
- Passphrase → Keychain (`sshkey-<UUID>`).
|
||||
|
||||
## Host-Key prüfen & akzeptieren (Popup)
|
||||
- Bei Erstverbindung zeigt die App ein Popup mit SHA256-/MD5-Fingerprint (per `ssh-keyscan` geholt) — mit Terminal-Ausgabe oder Admin-Angabe vergleichen, dann Akzeptieren & Verbinden (landet in `~/.ssh/known_hosts`, gemeinsam mit Terminal.app).
|
||||
- Bei GEÄNDERTEM Key: rote Warnung (MITM möglich!) — nur ersetzen, wenn erwartet (Server neu aufgesetzt). Zurücksetzen auch manuell: `ssh-keygen -R HOST`.
|
||||
- Technik: `Sources/ctty-run/ctty-run.c` gibt ssh ein Controlling-Terminal (sonst scheitern alle interaktiven Prompts still).
|
||||
|
||||
## 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.
|
||||
|
||||
@@ -18,6 +18,12 @@ rm -rf "$APP"
|
||||
mkdir -p "$APP/Contents/MacOS" "$APP/Contents/Resources"
|
||||
cp "$BIN" "$APP/Contents/MacOS/RemminaMac"
|
||||
chmod +x "$APP/Contents/MacOS/RemminaMac"
|
||||
# Ctty-Helper (gibt ssh ein Controlling-Terminal für Prompts)
|
||||
if [ -x "$ROOT/.build/release/ctty-run" ]; then
|
||||
cp "$ROOT/.build/release/ctty-run" "$APP/Contents/MacOS/"
|
||||
else
|
||||
echo "[build-app] WARNUNG: ctty-run fehlt (swift build lief ohne C-Target?)"
|
||||
fi
|
||||
cp "$ROOT/Resources/AppIcon.icns" "$APP/Contents/Resources/" 2>/dev/null || echo "[build-app] Hinweis: kein AppIcon.icns (./scripts/generate-icon.sh)"
|
||||
|
||||
# Info.plist mit Version versehen
|
||||
|
||||
Reference in New Issue
Block a user