fix: reject unknown ssh host keys

Author: Lukas Geiger

CHANGELOG.md

@@ -7,6 +7,8 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
 
 ### Behoben
 
+- `features/remote_editor.py`: SSH/SFTP-Verbindungen laden bekannte Hostkeys und
+  lehnen unbekannte Hostkeys jetzt ab, statt sie automatisch zu akzeptieren.
 - `ui/main_window.py` (B-011): ProjectView blieb beim Öffnen einer Datei aus
   einem anderen Ordner auf dem ersten Root hängen. Der Projektbaum folgt jetzt
   auch bei späteren Dateiwechseln dem aktuellen Dateiverzeichnis; neuer

features/remote_editor.py

@@ -52,7 +52,15 @@ def connect(self) -> bool:
 
         try:
             self._ssh = paramiko.SSHClient()
-            self._ssh.set_missing_host_key_policy(paramiko.AutoAddPolicy())
+            try:
+                self._ssh.load_system_host_keys()
+            except OSError as exc:
+                logger.debug("System-Hostkeys konnten nicht geladen werden: %s", exc)
+
+            user_known_hosts = Path.home() / ".ssh" / "known_hosts"
+            if user_known_hosts.exists():
+                self._ssh.load_host_keys(str(user_known_hosts))
+            self._ssh.set_missing_host_key_policy(paramiko.RejectPolicy())
 
             connect_kwargs = {
                 "hostname": self.host.hostname,

tests/test_remote_editor.py

@@ -63,3 +63,25 @@ def fake_get(remote, local):
     # Aufräumen
     session.disconnect()
     session._sftp = None
+
+
+def test_connect_rejects_unknown_host_keys(monkeypatch):
+    """SSH-Verbindungen dürfen unbekannte Hostkeys nicht automatisch akzeptieren."""
+    from features import remote_editor
+
+    fake_paramiko = MagicMock()
+    ssh_mock = MagicMock()
+    reject_policy = object()
+    fake_paramiko.SSHClient.return_value = ssh_mock
+    fake_paramiko.RejectPolicy.return_value = reject_policy
+    monkeypatch.setattr(remote_editor, "PARAMIKO_AVAILABLE", True)
+    monkeypatch.setattr(remote_editor, "paramiko", fake_paramiko, raising=False)
+
+    host = remote_editor.RemoteHost(name="test", hostname="localhost", username="user")
+    session = remote_editor.SFTPSession(host)
+
+    assert session.connect() is True
+
+    ssh_mock.load_system_host_keys.assert_called_once_with()
+    ssh_mock.set_missing_host_key_policy.assert_called_once_with(reject_policy)
+    ssh_mock.connect.assert_called_once_with(hostname="localhost", port=22, username="user")