Pushover - Line Feeds funktionieren nicht mehr

[Wolfpunk]

Hi all,

irgendwie funktionieren Line Feeds im Pushover-Modul seit einiger Zeit nicht mehr.

Wenn ich z.B. das Beispiel aus der Doku abschicke, wird \n nicht mehr als Line Feed interpretiert, sondern als "Backslash-n" in der Pushover Message angezeigt:

Code Auswählen Erweitern

set Pushover1 msg My second Pushover message.\nThis time with two lines.

Ich bin mir ziemlich sicher, dass das schon mal funktioniert hat. Steht ja schließlich auch so in der FHEM Hilfe. Weiß vielleicht jemand Rat?

[Wolfpunk]

Hab's rausgefunden. Es ist dieser Commit vom 7.4.2023 in der HttpUtils.pm von Rudi, der den oben genannten Fehler verursacht:

Code Auswählen Erweitern

fhem/FHEM/HttpUtils.pm
@@ -49,7 +49,7 @@ filename2MIMEType($) {
 sub
 urlEncode($) {
   $_= $_[0];
-  s/([\x00-\x2F \x3A-\x40 \x5B-\x60 \x7B-\xFF])/sprintf("%%%02x",ord($1))/eg;
+  s/([\x00-\x2F \x3A-\x40 \x5B-\x60 \x7B-\xFF])/sprintf("%%%02X",ord($1))/eg;
   return $_;
 }
Sobald ich diese Änderung rückgängig mache, funktionieren Line Feeds im Pushover Modul wieder.

[Jamo]

Danke, das hatte mich auch schon gewundert das die Zeilenumbrüche nicht mehr funktionieren. Vielleich kann Rudi das wieder korrigieren!

[rudolfkoenig]

Das Pushover Modul hat sich darauf verlassen, dass die urlEncode Funktion \ als %5c zurueckliefert, das habe ich aber in HttpUtils.pm nach %5C geaendert, damit ich die Ausgabe einfacher mit anderen URL-Encoder vergleichen kann.

Ich habe jetzt Pushover.pm direkt modifiziert (und eingecheckt), da der Maintainer sich seit einem Jahr nicht mehr im Forum angemeldet hat.

[Wolfpunk]

Macht Sinn. Besten Dank für den schnellen Fix!

[CoolTux]

Ich danke Dir Rudi.