]> source.dussan.org Git - nextcloud-server.git/commit
Fix WebDAV auth for session authentication only
authorLukas Reschke <lukas@owncloud.com>
Tue, 20 Jan 2015 08:53:03 +0000 (09:53 +0100)
committerLukas Reschke <lukas@owncloud.com>
Tue, 20 Jan 2015 09:03:14 +0000 (10:03 +0100)
commit476579b9c64753ec4e614b64f5a931b39eb2ddaa
treefec90ee6895687e0cb59b62e2991dc62d7b4004e
parent2ac015256fda60f684952131f220cd3db5a616ba
Fix WebDAV auth for session authentication only

\Sabre\DAV\Auth\Backend\AbstractBasic::authenticate was only calling \OC_Connector_Sabre_Auth::validateUserPass when the response of \Sabre\HTTP\BasicAuth::getUserPass was not null.

However, there is a case where the value can be null and the user could be authenticated anyways: The authentication via ownCloud web-interface and then accessing WebDAV resources. This was not possible anymore with this patch because it never reached the code path in this scenario.

This patchs allows authenticating with a session without isDavAuthenticated value stored (this is for ugly WebDAV clients that send the cookie in any case) and thus the functionality should work again.

To test this go to the admin settings and test if the WebDAV check works fine. Furthermore all the usual stuff (WebDAV / Shibboleth / etc...) needs testing as well.
lib/private/connector/sabre/auth.php
lib/private/user.php