From 059b7435abee8269fd5ac2948bb5fde138d7e24d Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma <roeland@famdouma.nl> Date: Thu, 14 Jul 2016 22:53:12 +0200 Subject: [PATCH] PasswordLoginForbidden is not a FATAL exception It is just a 'Sabre\DAV\Exception\NotAuthenticated' exception with some special meaning. So just log it as DEBUG and not as FATAL. --- apps/dav/lib/Connector/Sabre/ExceptionLoggerPlugin.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/apps/dav/lib/Connector/Sabre/ExceptionLoggerPlugin.php b/apps/dav/lib/Connector/Sabre/ExceptionLoggerPlugin.php index 38bddef8769..addacec9b03 100644 --- a/apps/dav/lib/Connector/Sabre/ExceptionLoggerPlugin.php +++ b/apps/dav/lib/Connector/Sabre/ExceptionLoggerPlugin.php @@ -32,6 +32,9 @@ use Sabre\HTTP\Response; class ExceptionLoggerPlugin extends \Sabre\DAV\ServerPlugin { protected $nonFatalExceptions = array( 'Sabre\DAV\Exception\NotAuthenticated' => true, + // If tokenauth can throw this exception (which is basically as + // NotAuthenticated. So not fatal. + 'OCA\DAV\Connector\Sabre\Exception\PasswordLoginForbidden' => true, // the sync client uses this to find out whether files exist, // so it is not always an error, log it as debug 'Sabre\DAV\Exception\NotFound' => true, -- GitLab