search

Befox / cdav

Dolibarr module to access calendars and address book from CalDAV/CardDAV tools
GNU General Public License v3.0
47 stars 28 forks source link

2.20 NullLogger issue #54

Open cfoellmann opened 1 year ago

cfoellmann commented 1 year ago

I tested the 2.10 which is working fine as far as I can see.

2.20 and 2.21 only give a 500 error when accessing DAV

[Tue Dec 06 12:36:30.452678 2022] [proxy_fcgi:error] [pid 2229022:tid 140390678103808] [client 10.1.15.1:0] AH01071: Got error 'PHP message: PHP Fatal error:  Declaration of Psr\\Log\\NullLogger::log($level, Stringable|string $message, array $context = []): void must be compatible with Psr\\Log\\LoggerInterface::log($level, $message, array $context = []) in /home/runcloud/webapps/erp-staging/htdocs/cdav/lib/SabreDAV/vendor/psr/log/src/NullLogger.php on line 26'

gets logged to the server log

jpfox commented 1 year ago

When you upgrade to 2.20 or 2.21, have you replace cdav folder with the new one ? If you copy cdav content into an existing folder, it can fail

jpfox commented 1 year ago

Are you using PHP 8.x ? it is required for cdav v2.2x

cfoellmann commented 1 year ago

First I got the composer error because of php 7.4.x. I switched to 8.0 (tested also with 8.1)

I did replace the whole folder. Did a mirror with WinSCP have a 1:1 copy of the repo to my webserver folder

jpfox commented 1 year ago

Strange, I have not this error. Have to test more, but no available space in my end of year planning :-(

cfoellmann commented 1 year ago

I did another test. Downloaded the zip of the master. Same issue.

It is something with the logging somehow but it give me no more info on the issue. The extended class in SabreDav from PSR seems correct to me. 👿

Something curious:

  1. run 2.10 and log in to server.php
  2. replace cdav with 2.21 -> I am still logged in and I can browse all endpoints in the "webclient"

It seems to have something to do with AUTH?