THANK YOU
I want to start by thanking you for being here, helping me and for being such great community.
NEXTCLOUD:
I am using 29.0.7 (got limited possibilities to upgrade to new major version as i have very big instance here)
DESCRIPTION
We using external file storage for storing files.
User has shared directory with permissions 11101 which means he can do literally everything except update existing files.
However when he send any PUT webdav request: /remote.php/dav/files/userid/shareddirectory he causes errors on server side.
Forbidden You cannot update the version's metadata because you do not have update permissions on the source file. Exception thrown: OCA\DAV\Connector\Sabre\Exception\Forbidden
I picked random error initiator and figured out nothing strange. Simple windows 10 user that used Edge browser. I looked into direct source of error and noticed hardcoded update permission check for \OCP\Constants::PERMISSION_UPDATE which does not have sense on file upload. Why would he need that if he want simply to upload new file to shared directory?
I understand that size will be recalculated on cron.php execution so it has low priority for you but why is that ? Is there any particular reason you doing it that way for LegacyVersionsBackend.php?