FHEM Forum

FHEM - Hausautomations-Systeme => Unterstützende Dienste => Thema gestartet von: gandy am 20 April 2015, 21:46:55

Titel: Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 20 April 2015, 21:46:55
Hi,

anbei ein Patch für 75_MSG.pm mit folgenden Änderungen:
- UTF8-kodierte Emails (hilft bei Problemen mit Umlauten)
- Perl Warnung wird behoben

Grüße,
Andy.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: rudolfkoenig am 21 April 2015, 07:32:20
Ich habe das Gefuehl, dass der Maintainer (ruebedo) sich hier seit laengerem nicht gezeigt hat.

Falls er zu diesem Thema in 2-3 Wochen nichts gesagt hat, dann sollten wir was unternehmen.
Kannst du bitte dann hier nochmal posten?
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 21 April 2015, 09:01:06
Hallo Rudi,

das Gefühl hatte ich auch schon.. und ja, mach ich gern

Grüße,
Andy.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 03 Mai 2015, 16:31:15
Hallo Rudi,

.. was ich hiermit tun will. Hab den oben angehängten Patch nochmal leicht modifiziert, bitte also ggfls neu herunterladen.

Danke und schöne Grüße,
Andy.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: rudolfkoenig am 03 Mai 2015, 18:09:55
Habs eingecheckt. Falls du das Modul uebernehmen moechtest, melde dich.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 04 Mai 2015, 12:10:08
Danke für's einchecken!

Zitat von: rudolfkoenig am 03 Mai 2015, 18:09:55
Falls du das Modul uebernehmen moechtest, melde dich.

Ich denk mal drüber nach. Mit MSGMail und MSGFile zusammen immerhin 3 Module zum Preis von einem  ;D
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 05 Mai 2015, 12:18:23
Hab mir zur Unterstützung beim Nachdenken nochmal die drei fraglichen Module 75_MSG.pm, 76_MSGMail.pm und 76_MSGFile.pm genauer durchgelesen. So wie ich das sehe, existiert MSG nur, um den Sende/Schreibvorgang für die anderen beiden Module zu übernehmen, ohne erkennbaren Mehrwert.  Wenn ich die Module übernehme, würde ich den entsprechenden Code aus MSG in die anderen beiden Module verschieben und für bestehende Installationen vielleicht noch delegates in MSG vorhalten.

Ich nehme an als Maintainer brauche ich Zugriff aufs SVN, evtl aufs Wiki, was noch?
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: rudolfkoenig am 05 Mai 2015, 12:29:18
SVN ist Pflicht (bitte deine Sourceforge-ID per email an mich), Wiki ist Kuer.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: rudolfkoenig am 05 Mai 2015, 12:50:22
Hab vergessen zu schreiben: bitte nicht vom sourceforge account schreiben, da kann ich (warum auch immer) nicht antworten. Meine email-Adresse steht auf fhem.de unten.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: dman am 06 Mai 2015, 21:28:42
Ich würde da noch anmerken, dass es bei dem Modul zu Problemen mit neueren Versionen von Net::SMTP  (ab Version 3 oder so) und Net::SMTP::SSL  (ab 1.01?) kommt. Ich habe das für mich auf die Schnelle gepatcht, aber allgemeingültig ist die Lösung natürlich nicht. Ich weiß nicht, ob viele damit ein Problem haben.

Siehe http://forum.fhem.de/index.php/topic,12821.msg249300.html#msg249300 (http://forum.fhem.de/index.php/topic,12821.msg249300.html#msg249300) und die Änderung in dem dort angehängten Modul.

Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 06 Mai 2015, 22:00:44
Danke für den Hinweis, meine Version von Net:SMTP ist die 2.31.

Ich kann versuchen, daraus eine allgemein gültige Lösung zu stricken, dazu müsste ich nur wissen,  wie ich zwischen den Versionen zuverlässig unterscheiden kann. Allerdings kann ich nur mit meiner Version testen, zum testen mit deiner Version bräuchte ich dann deine Hilfe.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 07 Mai 2015, 09:07:12
@d-man: Kannst Du bitte mal die Version im Anhang testen, ob die bei Dir funktioniert? Die Ausgabe im Log dazu wäre noch interessant, falls es nicht geht.

Wenn es keine Probleme damit gibt, check ich das so ein.

Danke,
Andy.
Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: dman am 07 Mai 2015, 23:06:08
@Andy: Danke für's schnelle Einbauen!

Ich habe es getestet, das sieht gut aus, Mail versenden funktioniert und folgender Log-Eintrag:

MSG: SSL is available, provided by Net::SMTP

Titel: Antw:Patch: UTF8 für 75_MSG.pm
Beitrag von: gandy am 08 Mai 2015, 08:22:12
Danke für's Testen und natürlich für die Anregung. Im Augenblick betrifft das vermutlich noch nicht viele Installationen aber es werden sicher irgendwann einige auf die neue Net::SMTP Version umstellen.

Die neue Modul-Version liegt jetzt im SVN bereit, ab morgen ist sie per update verfügbar.

Damit schließe ich diesen Thread, weitere Probleme/Wünsche/Anregungen zum Modul bitte ich unter Automatisierung zu diskutieren.