Merge pull request #53680 from nextcloud/backport/53677/stable30

[stable30] fix(dav): catch exception on non local account
This commit is contained in:
Andy Scherzinger 2025-06-25 18:03:41 +02:00 committed by GitHub
commit 7c3d2d39f4
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -9,6 +9,7 @@ namespace OCA\DAV\Connector\Sabre;
use OC\AppFramework\Http\Request;
use OC\FilesMetadata\Model\FilesMetadata;
use OC\User\NoUserException;
use OCA\DAV\Connector\Sabre\Exception\InvalidPath;
use OCP\Accounts\IAccountManager;
use OCP\Constants;
@ -370,7 +371,13 @@ class FilesPlugin extends ServerPlugin {
}
// Check if the user published their display name
$ownerAccount = $this->accountManager->getAccount($owner);
try {
$ownerAccount = $this->accountManager->getAccount($owner);
} catch (NoUserException) {
// do not lock process if owner is not local
return null;
}
$ownerNameProperty = $ownerAccount->getProperty(IAccountManager::PROPERTY_DISPLAYNAME);
// Since we are not logged in, we need to have at least the published scope