fix: Hostnamen normalisieren (Schema/Slash strippen) + freundlichere Nextcloud-Fehler

Bug: nextcloudhost mit https://-Präfix in CSV/GUI führte zu Doppel-Schema
(https://https://cloud.foo.de) und damit zu DNS-Fehler "host=https".

- models.clean_host() entfernt http(s):// und Trailing-Slash; Account
  ruft das im __post_init__ für plesk-/kerio-/nextcloudhost auf.
- NextcloudClient wickelt requests-Connection-Fehler in NextcloudError
  ein, damit die Log-Ausgabe lesbar bleibt statt HTTPSConnectionPool-Stacktrace.
- README: Hostname-Eingabe als tolerant dokumentiert; pleskhost-Pflicht
  im manual-Modus klarer (POP3-Sammler braucht es trotzdem).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-05-12 13:05:51 +02:00
parent dda5c746ce
commit b88c766dd6
3 changed files with 39 additions and 10 deletions
+6 -1
View File
@@ -97,7 +97,7 @@ case-insensitive. Beispiel: [`example.csv`](./example.csv).
| `Vorname` | ja | Vorname | | `Vorname` | ja | Vorname |
| `Name` | ja | Nachname | | `Name` | ja | Nachname |
| `emailadresse` | ja | volle Mailadresse, ist Login bei Plesk + Kerio | | `emailadresse` | ja | volle Mailadresse, ist Login bei Plesk + Kerio |
| `pleskhost` | ja | Hostname Plesk-Mailserver (auch POP3-Server für Kerio-Sammler) | | `pleskhost` | ja | Hostname Plesk-Mailserver **auch im `manual`-Modus Pflicht**, weil Kerio davon per POP3 abholt |
| `keriohost` | ja | Hostname Kerio Connect (Admin-API auf Port 4040, Webmail auf 443) | | `keriohost` | ja | Hostname Kerio Connect (Admin-API auf Port 4040, Webmail auf 443) |
| `nextcloudhost` | ja | Nextcloud-Hostname | | `nextcloudhost` | ja | Nextcloud-Hostname |
| `kerioemailkennwort` | ja | Passwort für den Kerio-User | | `kerioemailkennwort` | ja | Passwort für den Kerio-User |
@@ -110,6 +110,11 @@ case-insensitive. Beispiel: [`example.csv`](./example.csv).
> (`vorname.nachname`, lowercase, ä→ae, ö→oe, ü→ue, ß→ss). Die Emailadresse > (`vorname.nachname`, lowercase, ä→ae, ö→oe, ü→ue, ß→ss). Die Emailadresse
> aus der CSV wird im Nextcloud-Profil als E-Mail eingetragen. > aus der CSV wird im Nextcloud-Profil als E-Mail eingetragen.
> **Hostnamen-Eingabe ist tolerant**: `cloud.foo.de`, `https://cloud.foo.de`
> und `https://cloud.foo.de/` werden alle akzeptiert Schema (`http(s)://`)
> und Trailing-Slash werden automatisch gestrippt, sodass beim URL-Bau
> kein doppeltes `https://https://…` entsteht.
--- ---
## Aufruf ## Aufruf
+14 -9
View File
@@ -31,8 +31,17 @@ class NextcloudClient:
raise NextcloudError(f"Nextcloud {op}: ungültige Antwort: {r.text[:300]}") raise NextcloudError(f"Nextcloud {op}: ungültige Antwort: {r.text[:300]}")
return data.get("ocs", {}) return data.get("ocs", {})
def _request(self, method: str, url: str, op: str, **kw):
try:
return self.session.request(method, url, timeout=30, **kw)
except requests.RequestException as e:
raise NextcloudError(
f"Nextcloud {op} Verbindung zu {url} fehlgeschlagen: {e}"
) from e
def user_exists(self, userid: str) -> bool: def user_exists(self, userid: str) -> bool:
r = self.session.get(f"{self.base}/ocs/v2.php/cloud/users/{userid}") r = self._request("GET", f"{self.base}/ocs/v2.php/cloud/users/{userid}",
op="user-lookup")
ocs = self._meta(r, "user-lookup") ocs = self._meta(r, "user-lookup")
sc = ocs.get("meta", {}).get("statuscode") sc = ocs.get("meta", {}).get("statuscode")
return sc in (100, 200) return sc in (100, 200)
@@ -40,10 +49,8 @@ class NextcloudClient:
def ensure_group(self, group: str) -> None: def ensure_group(self, group: str) -> None:
if not group: if not group:
return return
r = self.session.post( r = self._request("POST", f"{self.base}/ocs/v2.php/cloud/groups",
f"{self.base}/ocs/v2.php/cloud/groups", op="group-create", data={"groupid": group})
data={"groupid": group},
)
ocs = self._meta(r, "group-create") ocs = self._meta(r, "group-create")
sc = ocs.get("meta", {}).get("statuscode") sc = ocs.get("meta", {}).get("statuscode")
# 100/200 = ok, 102 = exists already # 100/200 = ok, 102 = exists already
@@ -66,10 +73,8 @@ class NextcloudClient:
body.append(("groups[]", group)) body.append(("groups[]", group))
body.append(("quota", f"{quota_gb} GB" if quota_gb else "none")) body.append(("quota", f"{quota_gb} GB" if quota_gb else "none"))
r = self.session.post( r = self._request("POST", f"{self.base}/ocs/v2.php/cloud/users",
f"{self.base}/ocs/v2.php/cloud/users", op="user-create", data=body)
data=body,
)
ocs = self._meta(r, "user-create") ocs = self._meta(r, "user-create")
sc = ocs.get("meta", {}).get("statuscode") sc = ocs.get("meta", {}).get("statuscode")
if sc not in (100, 200): if sc not in (100, 200):
+19
View File
@@ -2,6 +2,19 @@ from dataclasses import dataclass, field
from typing import List, Optional from typing import List, Optional
def clean_host(s: str) -> str:
"""`https://cloud.foo.de/` → `cloud.foo.de`.
Akzeptiert die Eingaben unabhängig davon, ob der User Schema/Slash
mit eingetippt hat. Verhindert Doppel-Schemas wie
`https://https://cloud.foo.de` beim Bauen von URLs.
"""
s = (s or "").strip()
if "://" in s:
s = s.split("://", 1)[1]
return s.rstrip("/")
@dataclass @dataclass
class Account: class Account:
name: str name: str
@@ -16,6 +29,12 @@ class Account:
nextcloudspeicher: Optional[int] = None # GB; None = unlimitiert nextcloudspeicher: Optional[int] = None # GB; None = unlimitiert
nextcloudkennwort: str = "" nextcloudkennwort: str = ""
def __post_init__(self):
self.pleskhost = clean_host(self.pleskhost)
self.keriohost = clean_host(self.keriohost)
self.nextcloudhost = clean_host(self.nextcloudhost)
self.emailadresse = (self.emailadresse or "").strip()
@property @property
def vollname(self) -> str: def vollname(self) -> str:
return f"{self.vorname} {self.name}".strip() return f"{self.vorname} {self.name}".strip()