Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.20 NullLogger issue #54

Open
cfoellmann opened this issue Dec 6, 2022 · 5 comments
Open

2.20 NullLogger issue #54

cfoellmann opened this issue Dec 6, 2022 · 5 comments

Comments

@cfoellmann
Copy link

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
Copy link
Member

jpfox commented Dec 6, 2022

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
Copy link
Member

jpfox commented Dec 6, 2022

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

@cfoellmann
Copy link
Author

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
Copy link
Member

jpfox commented Dec 6, 2022

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

@cfoellmann
Copy link
Author

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants