Emailserver (POP3, SMTP und IMAP)

[kieselsteini]

Hallo liebes Forum und liebe Python Gemeinde.

Ich habe vor einer Woche mit Python angefangen zu programmieren, da ich mich mal von den "Zwängen" der Java EE Programmierung befreien wollte.

Als kleine Einstiegsprojekt habe ich mich an einen Emailserver gewagt, der sowohl als POP3 als auch als SMTP Server funktioniert. IMAP ist leider noch sehr in den Kinderschuhen da ich mich a) mit Python noch nicht recht auskenne und b) IMAP ein Drecks-Protokoll ist (sorry *g).

Es würde mich freuen, wenn ein paar Python Profis sich das mal ansehen würden und ihren Senf dazu abgeben würden. Is das "pythonic", verstoße ich gegen ein paar "goldene Regeln" der Python Programmierung? Was würden die Pros besser machen?

Natürlich freue mich über jede Zeile Code die den Server verbessert. Zumindest auf meinem Test-Rechner lief dieser Code ~48h ohne Probleme durch wobei er Mails von 3 gut besuchten Mailinglisten bekam.

Hier ist der Code zu finden: http://pastie.org/604334

Vielen Dank und ich freue mich schon auf die Antworten

[EyDu]

Hallo.

Ich habe es nur ganz grob überflogen, vom Code her sieht es ganz vernünftig aus. Du solltest allerdings die ganzen if/elif-Kaskaden durch Dictionaries ersetzen. Deine Funktionen sind so lang, die kann doch kein Mensch mehr überblicken

Für die ganzen "self.outlines.append"-Aufrufe solltest du dir vielleicht mal eine Methode basteln

Auch das Starten der verschiedenen Serverarten kannst du mit einem Dictionary erschlagen. Du brauchst auch keine drei "start_..."-Funktionen, das ist mein einer einzigen erledigt.

Das du auch Adressen wie ".....@............-......de" für gültig befindest weißt du wahrscheinlich selber.

Sebastian

[Hyperion]

Desweiteren wäre ein Pastebin mit Syntax-Highlighting zu emfpehlen - wie z.B. paste.pocoo.org

[frabron]

Coole Idee

Allerdings finde ich diese riesigen if Verschachtelungen echt grausam zu lesen. Evt. wäre ja da eine Lösung basierend auf Dictionaries besser. Grade diese command == Geschichte bietet sich ja dazu an.

Außerdem finde ich es nicht gut, die Statuscodes (250 hello %s, nice to meet you, etc.) fest im Quelltext zu schreiben, da würde ich etwas ausserhalb bevorzugen. Keine Ahnung, wie da der Standard ist, aber dann könnte man auch Meldungen auf Deutsch oder so versenden.

Aber das ist nur meine Meinung ...

[Leonidas]

Keine Ahnung, wie da der Standard ist, aber dann könnte man auch Meldungen auf Deutsch oder so versenden.

Was bringen eingedeutschte Meldungen die eh von keinem gelesen werden?

[frabron]

Wenn man ständig Kunden aus drei oder mehr Ländern mit unterschiedlichen Sprachen hat, ist das irgendwann ein Reflex, alle Strings so vorzubereiten, dass sie übersetzt werden können Immer schön die Strings aus der Programmlogik extrahieren, so dass sie bei Bedarf geändert werden können.

Ob das jetzt hier Sinn macht, kann ich natürlich nicht sagen, dazu kenne ich mich nicht genug mit Mailprotokollen aus. Gerade bei der SMTP Klasse fiel mir das aber ins Auge ...

[Leonidas]

Zu bedenken ist, dass die Texte zum Wire-Protokoll gehören und, obzwar da Variationen AFAIR erlaubt sind würde ich davon abraten etwas anderes als ASCII auszugeben, da SMTP nicht garantiert 8-Bit safe ist - hängt immer ab was man da für eine Gegenstelle hat.

[kieselsteini]

Hallo Zusammen.

Erstmal vielen Dank für eure Ratschläge und Hilfestellungen. Ich habe gleich mal gesucht was Dicts sein sollen und dann gleich mal den Mailserver dahingehend aufgeräumt. Keine langen if/else Kaskaden mehr, sondern schön alles in commands Dictionaries untergebracht.

Zur Mehrsprachigkeit der Server-Antworten: Das ist eigentlich nicht notwendig, weil der Benutzer im Regelfall diese Meldungen nicht zu Gesicht bekommt und sie ganz eigentlich auch komplett weggelassen werden könnten. Die Mailclients werten bei SMTP eh nur die Statuscodes aus und bei POP3 die +OK / -ERR Antworten.

Das Problem mit der Emailadresse sehe ich nicht (bin auch kein Regexp Profi). Für mein Empfinden werden da nur offizielle Mailadressen akzeptiert. Für bessere Regexp Vorschläge bin ich offen

So jetzt noch eine Frage an die Python-Profis. Wenn ich jetzt eine Relay-Funktionalität einbauen möchte (SMTP-AUTH is ja schon dafür implementiert), dann benutze ich die smtplib die in Python schon mit an Bord ist oder? Da ja der Relay einer Nachricht auch noch dauern kann, sollte ich das in Threads auslagern ?! Ist ein Threading Modell oder ein Forking Modell besser? Habe gelesen, dass Python bei Threading Probleme mit Mehrkernprozessoren hat bzw. diese nicht gut ausnutzt.

So. Zu guter letzt noch der aktuelle Code: http://paste.pocoo.org/show/137942/
Sorry für das alte pastebin, aber Opera hat mit pocoo Probleme. Konnte nur mim Feuerfuchs den Code absetzen.

Freu mich auf weitere Kommentare

[tiax]

Zum Thema Mailadressen-prüfendes Regexp: Probier's gar nicht erst. Da kann man so viele Fälle übersehen, die wider erwarten doch gültig sind. Etwa Sonderzeichen, Kommentare, exotische TLDs wie .aero, etc

[saqdefaq]

Zum Thema Email-Adresse kann ich das hier empfehlen: Klick mich!