feat: Verschluesselung waehlbar, dazu zwei Layout-Fehler
Verschluesselung: Bisher gab es nur den Schalter use_ssl – also implizites TLS auf 993 oder gar nichts. STARTTLS auf 143 fehlte ganz, und das bieten viele Anbieter an. Jetzt ein Feld mit drei Werten, der Port zieht bei der Auswahl mit, und "keine" warnt, dass Zugangsdaten im Klartext gehen. Layout: Die Postfach-Karte steht in einer schmalen Spalte; der waagerechte Aufbau brach dort auf ein Wort je Zeile um und die Knoepfe lagen ueber dem Titel. Jetzt gestapelt. Die Einstellungsseite nutzte ein Raster, das die Karten zeilenweise ausrichtet – die zweite Reihe begann erst unter der hoechsten Karte der ersten, weshalb "Backup zurueckladen" allein weit unten stand. Jetzt fliessen die Karten spaltenweise, und die Spalten sind breiter: vier schmale Karten nebeneinander waren auf dem Notebook nicht lesbar. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -698,7 +698,10 @@ class MailSource(Base):
|
||||
name: Mapped[str] = mapped_column(String(80), nullable=False, default="")
|
||||
host: Mapped[str] = mapped_column(String(255), nullable=False)
|
||||
port: Mapped[int] = mapped_column(Integer, nullable=False, default=993)
|
||||
use_ssl: Mapped[bool] = mapped_column(Boolean, nullable=False, default=True)
|
||||
# "ssl" = TLS ab der ersten Zeile (Port 993, der Normalfall),
|
||||
# "starttls" = im Klartext verbinden und dann verschlüsseln (Port 143),
|
||||
# "none" = unverschlüsselt (nur sinnvoll innerhalb desselben Rechners).
|
||||
security: Mapped[str] = mapped_column(String(16), nullable=False, default="ssl")
|
||||
username: Mapped[str] = mapped_column(String(255), nullable=False)
|
||||
# Verschlüsselt, nicht gehasht: Certain muss es dem Mailserver vorlegen.
|
||||
password_enc: Mapped[str] = mapped_column(Text, nullable=False)
|
||||
|
||||
@@ -549,7 +549,7 @@ class MailSourceIn(BaseModel):
|
||||
name: str = Field(default="", max_length=80)
|
||||
host: str = Field(min_length=1, max_length=255)
|
||||
port: int = 993
|
||||
use_ssl: bool = True
|
||||
security: Literal["ssl", "starttls", "none"] = "ssl"
|
||||
username: str = Field(min_length=1, max_length=255)
|
||||
# Beim Ändern weglassen, um das gespeicherte Passwort zu behalten.
|
||||
password: str | None = None
|
||||
@@ -565,7 +565,7 @@ class MailSourceOut(BaseModel):
|
||||
name: str
|
||||
host: str
|
||||
port: int
|
||||
use_ssl: bool
|
||||
security: str
|
||||
username: str
|
||||
folder: str
|
||||
done_folder: str | None
|
||||
|
||||
@@ -45,14 +45,22 @@ def _verbinden(quelle: MailSource) -> imaplib.IMAP4:
|
||||
"Das gespeicherte Passwort lässt sich nicht mehr entschlüsseln. "
|
||||
"Wurde das Server-Geheimnis getauscht? Dann bitte neu eintragen."
|
||||
)
|
||||
art = (quelle.security or "ssl").lower()
|
||||
try:
|
||||
verbindung = (
|
||||
imaplib.IMAP4_SSL(quelle.host, quelle.port)
|
||||
if quelle.use_ssl
|
||||
else imaplib.IMAP4(quelle.host, quelle.port)
|
||||
)
|
||||
if art == "ssl":
|
||||
# TLS ab der ersten Zeile – der Normalfall auf Port 993.
|
||||
verbindung = imaplib.IMAP4_SSL(quelle.host, quelle.port)
|
||||
else:
|
||||
verbindung = imaplib.IMAP4(quelle.host, quelle.port)
|
||||
if art == "starttls":
|
||||
# Erst im Klartext verbinden, dann verschlüsseln (Port 143).
|
||||
verbindung.starttls()
|
||||
except OSError as e:
|
||||
raise MailFehler(f"Keine Verbindung zu {quelle.host}:{quelle.port} – {e}") from e
|
||||
except imaplib.IMAP4.error as e:
|
||||
raise MailFehler(
|
||||
f"Verschlüsselung fehlgeschlagen ({art}). Passt die Angabe zum Port {quelle.port}? – {e}"
|
||||
) from e
|
||||
try:
|
||||
verbindung.login(quelle.username, passwort)
|
||||
except imaplib.IMAP4.error as e:
|
||||
|
||||
@@ -0,0 +1,35 @@
|
||||
"""Verschluesselungsart des Postfachs waehlbar machen.
|
||||
|
||||
Bisher gab es nur den Schalter ``use_ssl`` – also implizites TLS auf Port 993
|
||||
oder gar nichts. STARTTLS auf Port 143 fehlte, und genau das bieten viele
|
||||
Anbieter an. Statt eines zweiten Schalters gibt es jetzt ein Feld mit drei
|
||||
klaren Werten.
|
||||
|
||||
Revision ID: 0017
|
||||
Revises: 0016
|
||||
"""
|
||||
|
||||
import sqlalchemy as sa
|
||||
from alembic import op
|
||||
|
||||
revision = "0017"
|
||||
down_revision = "0016"
|
||||
branch_labels = None
|
||||
depends_on = None
|
||||
|
||||
|
||||
def upgrade() -> None:
|
||||
op.add_column("mail_sources",
|
||||
sa.Column("security", sa.String(length=16), nullable=False, server_default="ssl"))
|
||||
# Bestehende Postfaecher behalten ihre Einstellung.
|
||||
op.execute("UPDATE mail_sources SET security = CASE WHEN use_ssl THEN 'ssl' ELSE 'none' END")
|
||||
with op.batch_alter_table("mail_sources") as batch:
|
||||
batch.drop_column("use_ssl")
|
||||
|
||||
|
||||
def downgrade() -> None:
|
||||
op.add_column("mail_sources",
|
||||
sa.Column("use_ssl", sa.Boolean(), nullable=False, server_default=sa.true()))
|
||||
op.execute("UPDATE mail_sources SET use_ssl = (security = 'ssl')")
|
||||
with op.batch_alter_table("mail_sources") as batch:
|
||||
batch.drop_column("security")
|
||||
Reference in New Issue
Block a user