Autor Thema: 57_Calendar - Unterstützung für gzip  (Gelesen 2300 mal)

Offline Johannes_B

  • Developer
  • Full Member
  • ****
  • Beiträge: 116
57_Calendar - Unterstützung für gzip
« am: 10 Januar 2015, 12:42:07 »
Hi,

bei mir hat das Kalender Modul in Verbindung mit iCloud nicht funktioniert und bin daher auf die Spurensuche gegangen.
Es scheint so, dass die Daten von iCloud komprimiert ankommen, und damit kommt das Modul nicht klar.
Da man von HttpUtils_BlockingGet leider keinen Header bekommt, kann man den Content-Type nicht direkt überprüfen, deswegen wäre meine Lösung "Try-and-Error".

Sicherlich kann man den Code noch schöner bauen, aber ich wollte jetzt eine pragmatische Lösung:

  if($#entries<0) {
    eval { require Compress::Zlib; };
    if($@) {
      Log3 $hash, 1, "Calendar " . $hash->{NAME} . ": Maybe gzip data, but cannot load Compress::Zlib";
    }
    else {
      $ics = Compress::Zlib::memGunzip($ics);
      $ical->parse(split("\n",$ics));
      @entries= @{$ical->{entries}};
    }
  };

Das Ganze kommt in Zeile 945.

Wäre schön, wenn das (oder was Vergleichbares) ins Repo kommt.


Gruß,

Johannes
FHEM Control - an iOS app - available on the App Store:
https://itunes.apple.com/app/id936674170

Offline Dr. Boris Neubert

  • Global Moderator
  • Hero Member
  • ****
  • Beiträge: 4480
Antw:57_Calendar - Unterstützung für gzip
« Antwort #1 am: 10 Januar 2015, 14:28:40 »
Hallo,

bitte hänge einen Patch an.

svn diff 57_Calendar.pm > 57_Calendar.pm.patch
Dann teste ich das nochmal und checke es ein.

Danke
Boris
Globaler Moderator, Developer, aktives Mitglied des FHEM e.V. (Marketing, Verwaltung)
Bitte keine unaufgeforderten privaten Nachrichten!

Offline Johannes_B

  • Developer
  • Full Member
  • ****
  • Beiträge: 116
Antw:57_Calendar - Unterstützung für gzip
« Antwort #2 am: 10 Januar 2015, 14:35:10 »
Klar, gerne, anbei der Patch. :)


Gruß,

Johannes
FHEM Control - an iOS app - available on the App Store:
https://itunes.apple.com/app/id936674170

Offline Dr. Boris Neubert

  • Global Moderator
  • Hero Member
  • ****
  • Beiträge: 4480
Antw:57_Calendar - Unterstützung für gzip
« Antwort #3 am: 10 Januar 2015, 21:20:47 »
eingecheckt
bn
Globaler Moderator, Developer, aktives Mitglied des FHEM e.V. (Marketing, Verwaltung)
Bitte keine unaufgeforderten privaten Nachrichten!

Offline Johannes_B

  • Developer
  • Full Member
  • ****
  • Beiträge: 116
Antw:57_Calendar - Unterstützung für gzip
« Antwort #4 am: 11 Januar 2015, 10:19:53 »
Super, danke! :)


Gruß,

Johannes
FHEM Control - an iOS app - available on the App Store:
https://itunes.apple.com/app/id936674170

 

decade-submarginal