search

andig / carddav2fb

Download CardDAV VCards and upload as phonebook to AVM FRITZ!Box
63 stars 19 forks source link

Workaround Sabre HTTP 500 error on REPORT requests #218

Open andig opened 3 years ago

andig commented 3 years ago

Fix https://github.com/andig/carddav2fb/issues/196

blacksenator commented 3 years ago

Sorry for delay - took a few days before I found time for testing. Findings:

  1. PR has no side effects to iCloud requested via REPORT
  2. PR has no side effects to T-Online requested via PROPFIND (see #205)
  3. PR throws an error at T-Online requested via REPORT: In Backend.php line 185: String could not be parsed as XML

Case 2:

'server' => [
    [
        'url'      => 'https://spica.t-online.de/spica-contacts/carddav/principals/XXXXXXXX@t-online.de/addressBooks/default/',
        'user'     => 'XXXXXXXX@t-online.de',
        'password' => 'XXXXXXXX',
        'method'   => 'PROPFIND',  // uncomment if 'REPORT' (default) causes an error (e.g. t-online)
        ],
],
blacksenator commented 3 years ago

According to this feedback I would suggest to close this PR

andig commented 3 years ago

Mit diesem PR sollte es auch mit REPORT laufen- dann aber VIEL schneller als PROPFIND. Ich würde den also nicht schliessen.

blacksenator commented 3 years ago

Ja, bin ja bei Dir: "sollte". Nur hat ja der "implizite" Test von altheeagle mit Baikal* und mein Test mit T-online gezeigt, dass die Änderung nicht den gewünschten Effekt hatte.

andig commented 3 years ago

Du könntest bei T-Online mal den nicht parsebaren String ausgeben damit wir sehen was da los ist. Die Zeile steht ja ind er Fehlermeldung.