Patch: UTF8 für 75_MSG.pm

Begonnen von gandy, 20 April 2015, 21:46:55

Vorheriges Thema - Nächstes Thema

gandy

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.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

rudolfkoenig

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?

gandy

Hallo Rudi,

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

Grüße,
Andy.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

gandy

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.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

rudolfkoenig

Habs eingecheckt. Falls du das Modul uebernehmen moechtest, melde dich.

gandy

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
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

gandy

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?
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

rudolfkoenig

SVN ist Pflicht (bitte deine Sourceforge-ID per email an mich), Wiki ist Kuer.

rudolfkoenig

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.

dman

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 und die Änderung in dem dort angehängten Modul.


gandy

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.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

gandy

@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.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1

dman

@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


gandy

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.
fhem (svn) auf i5-4210U NUC
2x HMLAN, 19x HM-SEC-RHS, 15x HM-LC-Bl1PBU-FM, etc.
ODYS Neron Tablet / Android 4.2
Samsung Galaxy Tab 2 10.1N / Android 4.1.2
Samsung Galaxy Note / Android 6.0.1