Fix Mail versenden bei Exception bei Versand

[JohannMaierhofer]

Ich habe try/catch in die Schleife eingebaut so wie es auch in der Prenotification implementiert ist.
Damit wird bei einem fehlgeschlagenen Senden weiter gemacht.

[mbmueller]

Sieht auf den ersten Blick gut aus, habe gerade keine Möglichkeit das ausführlich zu testen

[JohannMaierhofer]

Ich habe es getestet indem ich bei mehreren Mitgliedern meine Mail Adresse eingetragen habe und bei einem eine ungültige. Dann versenden. Da wird dann keiner mit den Test Mails belästigt.

[NicoB77]

In dem Issue wird erwähnt, dass der Versand an die fehlerhafte Adresse ewig dauert. War das bei Dir auch so? Ich vermute, dass Deine Änderungen nur verhindern, dass der Versand an die folgenden Adressen nicht mehr gemacht wird, aber nichts an der Dauer. Die könnte an der E-Mail-Konfiguration liegen. Wenn ich es richtig sehe, wurden die E-Mails auch vorher schon einzeln versandt.

[JohannMaierhofer]

Bei mir hat es nicht lange gedauert beim Senden an eine ungültige Mail Adresse.
Ja, vorher wurde auch einzeln versendet, aber der try/catch war so, dass im Fehlerfall ganz abgebrochen wurde und die folgenden Mails nicht mehr verschickt wurden.
Mein neuer try/catch macht Dann bei den folgenden Mails weiter.

[NicoB77]

Das hatte ich gesehen. Im Issue werden zwei Probleme genannt:

• dauert ewig bei Fehler
• bricht bei Fehler ab

Der try/catch-Block löst nur das zweite Problem, die Performanceprobleme sind vermutlich immer noch da. Darauf wollte ich hinweisen, war wohl nicht klar genug formuliert.

[mbmueller]

Das hatte ich gesehen. Im Issue werden zwei Probleme genannt:

• dauert ewig bei Fehler
• bricht bei Fehler ab

Der try/catch-Block löst nur das zweite Problem, die Performanceprobleme sind vermutlich immer noch da. Darauf wollte ich hinweisen, war wohl nicht klar genug formuliert.

Du hast das Issue wohl falsch verstanden. Nichts von beidem passiert. Es dauert nicht "ewig" sondern endlos. Daher wird eben NICHT abgebrochen bei einem Fehler und eine ungültige Adresse blockiert den weiteren Versand.
Der Issue zielt darauf ab, dass nach einer bestimmten Anzahl an Fehlversuchen der Versand an die fehlerhafte Adresse übersprungen wird.

[JohannMaierhofer]

Ja, dieser Pull Request löst nur, dass bei einem Fehler eines Senden nicht abgebrochen wird. Man könnte also diesen Request schon einmal übernehmen ohne den Issue zu schliesen.

Das zweite Problem mit dem endlosen Warten konnte ich nicht reproduzieren. Wie lautet denn die EMail Adresse bei der es hängen bleibt?
Ich hatte den Test mit einem Unsinn nach dem @ gemacht. Evtl ist das Problem wenn vor dem @ etwas falsch ist und nach dem @ korrekt.

[NicoB77]

@mbmueller Du hast einen Ausschnitt aus dem Log gepostet, der nach einem Abbruch aussieht. Wenn es keine Exception gibt, nützt der neue try/catch-Block auch nichts. Der verbessert nur das Verhalten bei Fehlern, damit die weiteren E-Mails noch versandt werden.

[mbmueller]

Ja, dieser Pull Request löst nur, dass bei einem Fehler eines Senden nicht abgebrochen wird. Man könnte also diesen Request schon einmal übernehmen ohne den Issue zu schliesen.

Das zweite Problem mit dem endlosen Warten konnte ich nicht reproduzieren. Wie lautet denn die EMail Adresse bei der es hängen bleibt? Ich hatte den Test mit einem Unsinn nach dem @ gemacht. Evtl ist das Problem wenn vor dem @ etwas falsch ist und nach dem @ korrekt.

martin@stammbussard-germering.de

[JohannMaierhofer]

Ich habe jetzt eine Mail an die oben genannte Adresse versendet. Das Versenden hat funktioniert ohne ewiges Warten.

Die Adresse ist ungültig und ich bekam in meinem EMail Programm eine entsprechende Mail zurück.

[JohannMaierhofer]

@mbmueller, du hast den Request schon gereviewt. Könntest du dann den Request genehmigen. Dann kann er übernommen werden.
Danke schon mal.