241 Commits (cb73fe26b818cfdb8fbf07116cd8e970a9cdc708)

Author	SHA1	Message	Date
Côme Chilliet	2b07031d3e	composer run cs:fix ... Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Côme Chilliet	23f7a37c03	Remove ILogger usages in lib/private/Files/Storage ... Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Robin Appelman	468d6a9cf1	ignore forbidden files while scanning local storage ... Signed-off-by: Robin Appelman <robin@icewind.nl>	4 years ago
Robin Appelman	f9a5fb99fd	use the same cached propfind for free space for dav storage ... Signed-off-by: Robin Appelman <robin@icewind.nl>	4 years ago
Max Kunzelmann	2952c7d01f	Use the null coalescing operator to set the value of `size`. ... Signed-off-by: Max Kunzelmann <maxdev@posteo.de>	4 years ago
Côme Chilliet	e8c1f75064	Fixing psalm errors ... Encryption constructor is problematic and should still be fixed later. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Côme Chilliet	6be7aa112f	Migrate from ILogger to LoggerInterface in lib/private ... Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Max Kunzelmann	3a135f0c5f	Set `mtime` to null instead of false if `getlastmodified` does not exist. ... Signed-off-by: Max Kunzelmann <maxdev@posteo.de>	4 years ago
Max Kunzelmann	e0a5fe649a	Fix typo. ... Signed-off-by: Max Kunzelmann <maxdev@posteo.de>	4 years ago
Max Kunzelmann	ee888875bb	Fixes the undefined index error with the DAV property getlastmodified ... Signed-off-by: Max Kunzelmann <maxdev@posteo.de>	4 years ago
Robin Appelman	84f464550a	some file scanner performance improvements ... Signed-off-by: Robin Appelman <robin@icewind.nl>	4 years ago
Côme Chilliet	5cca86ce19	Remove redundant if clause ... Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Côme Chilliet	35dcda36a3	Fix OC\Files\Storage\DAV::hasUpdated when cache is not set ... Fixes the test OCA\Files_External\Tests\Storage\WebdavTest::testStat on PHP>=7.4 Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>	4 years ago
Carl Schwan	cbf9064b8e	Fix psalm issues ... Signed-off-by: Carl Schwan <carl@carlschwan.eu>	4 years ago
Carl Schwan	695165260f	Add helper method in Wrapper ... Signed-off-by: Carl Schwan <carl@carlschwan.eu>	4 years ago
Jonas Meurer	a2f4c62c19	updateEncryptedVersion: cleanup on target if cache already got renamed ... When moving a file to trash with encryption enabled, the cache gets moved before the actual file. According to @icewind1991 this is in order to not break object storage. When moving a file from an unencrypted storage (e.g. a collectives storage) to the encrypted trashbin storage, this causes errors, see This commit fixes it by doing `updateEncryptedVersion()` on the target cache entry *if* the source cache entry doesn't exist anymore, but the corresponding target cache entry does exist already. Fixes: #26544 Signed-off-by: Jonas Meurer <jonas@freesources.org>	4 years ago
Vincent Petry	be590df588	Fix fail when keys/files folder already exists ... Fixes an issue with transfer ownership in move mode where the folder "files_encryption/keys/files" already exists. Instead of failing, its existence is checked before calling mkdir. Signed-off-by: Vincent Petry <vincent@nextcloud.com>	4 years ago
Vincent Petry	6f81e60bce	Check resource before closing in encryption wrapper ... In case of error there is no guarantee that $source or $target is set or is a resource when handling an error. Without this fix, there's a risk that fclose will fail and the actual exception will not be thrown, making it impossible to find out about the root cause. Signed-off-by: Vincent Petry <vincent@nextcloud.com>	4 years ago
Joas Schilling	65d79bb592	Check for invalid characters before trimming ... Signed-off-by: Joas Schilling <coding@schilljs.com>	4 years ago
acsfer	152e2bfdaa	Check for `disk_free_space` ... Check for `disk_free_space` before calling it. Fix https://github.com/nextcloud/server/issues/29748	4 years ago
Vincent Petry	c92a0e4152	Normalize directory entries in Encoding wrapper ... Directory entry file names are now normalized in getMetaData(), getDirectoryContents() and opendir(). This makes the scanner work properly as it assumes pre-normalized names. In case the names were not normalized, the scanner will now skip the entries and display a warning when applicable. Signed-off-by: Vincent Petry <vincent@nextcloud.com>	4 years ago
Vincent Petry	93fb33d863	Update lib/private/Files/Storage/Wrapper/Availability.php add void ... Signed-off-by: Vincent Petry <vincent@nextcloud.com> Co-authored-by: Carl Schwan <carl@carlschwan.eu>	4 years ago
Vincent Petry	db29fd29ee	Return false in hasUpdated when storage is not available ... Technically, saying that a storage has no updates when it's not available is correct. This makes it possible to retrieve the cache entry for the mount point and also to list and remove unavailable federated shares. Signed-off-by: Vincent Petry <vincent@nextcloud.com>	4 years ago
alanmeeson	16f70e8647	Add type hint to fread_block $blockSize param & apply cs:fix. ... Signed-off-by: alanmeeson <alan@carefullycalculated.co.uk>	4 years ago
Alan Meeson	44c332a46e	Fix truncation of files upon read when using object store and encryption. ... When using and object store as primary storage and using the default encryption module at the same time, any encrypted file would be truncated when read, and a text error message added to the end. This was caused by a combination of the reliance of the read functions on on knowing the unencrypted file size, and a bug in the function which calculated the unencrypted file size for a given file. In order to calculate the unencrypted file size, the function would first skip the header block, then use fseek to skip to the last encrypted block in the file. Because there was a corresponence between the encrypted and unencrypted blocks, this would also be the last encrypted block. It would then read the final block and decrypt it to get the unencrypted length of the last block. With that, the number of blocks, and the unencrypted block size, it could calculate the unencrypted file size. The trouble was that when using an object store, an fread call doesn't always get you the number of bytes you asked for, even if they are available. To resolve this I adapted the stream_read_block function from lib/private/Files/Streams/Encryption.php to work here. This function wraps the fread call in a loop and repeats until it has the entire set of bytes that were requested, or there are no more to get. This fixes the imediate bug, and should (with luck) allow people to get their encrypted files out of Nextcloud now. (The problem was purely on the decryption side). In the future it would be nice to do some refactoring here. I have tested this with image files ranging from 1kb to 10mb using Nextcloud version 22.1.0 (the nextcloud:22.1-apache docker image), with sqlite and a Linode object store as the primary storage. Signed-off-by: Alan Meeson <alan@carefullycalculated.co.uk>	4 years ago
Carl Schwan	9408f8ae69	Fix security issues when copying groupfolder with advanced ACL ... Using advanced ACL, it is possible that an user has access to a directory but not to a subdirectory, so the copying use Common::copyFromStorage instead of Local::copyFromStorage. Fix https://github.com/nextcloud/groupfolders/issues/1692 Signed-off-by: Carl Schwan <carl@carlschwan.eu>	4 years ago
Robin Appelman	b3766fc99b	make param names consistent with interface for copy directory polyfill ... Signed-off-by: Robin Appelman <robin@icewind.nl>	4 years ago
acsfer	98eac0fc05	Simplify :)	4 years ago
acsfer	8a8df49bc0	Replace `file_exists()` method by `is_file()`	4 years ago
acsfer	3a25183b66	Get `filesize()` if `file_exists()` ... Should make sense.	4 years ago
Daniel Kesselberg	be3f4edf1f	explicitly close source stream on encryption storage ... Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>	4 years ago
Daniel Kesselberg	e60a829b42	explicitly close source stream on local storage ... Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>	4 years ago
acsfer	9324a74bba	Handle files with is_file instead of file_exists ... Should fix things like `fread(): read of 8192 bytes failed with errno=21 Is a directory`	5 years ago
Julius Härtl	bc3c46362a	Make sure that a empty directory can still be deleted when copied from another storage ... Signed-off-by: Julius Härtl <jus@bitgrid.net>	5 years ago
Carl Schwan	28970563a2	Remove some mentions of ownCloud from our api documentation ... Signed-off-by: Carl Schwan <carl@carlschwan.eu>	5 years ago
Lukas Reschke	28359571de	Remove Flysystem storage class ... This seems unused as per https://github.com/nextcloud/server/issues/27768 and may allow us to get rid of one more dependency. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>	5 years ago
Rid	0469acfb20	Fix scanner mistaking socket files for directories ... Signed-off-by: Rid <rid@cylo.io>	5 years ago
John Molakvoæ (skjnldsv)	215aef3cbd	Update php licenses ... Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>	5 years ago
Robin Appelman	ed2d02d5f1	better cleanup of user files on user deletion ... Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Robin Appelman	aee4caed07	show better error messages when a file with a forbidden path is encountered ... Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Robin Appelman	e5dc1a8085	Set umask before operations that create local files ... this solves issues where "other php stuff" is messing with the umask Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Robin Appelman	7995c6e1dc	dont return private storage interface from public mount interface ... Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Robin Appelman	c87b1a50d9	apply object store copy optimization when 'cross storage' copy is within the same object store ... Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Roeland Jago Douma	618805b14f	Fix RedundantPropertyInitializationCheck ... For #25839 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>	5 years ago
Robin Appelman	b844293536	catch notfound and forbidden exception in smb::getmetadata ... Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
dependabot-preview[bot]	eb502c02ff	Bump nextcloud/coding-standard from 0.3.0 to 0.5.0 ... Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 0.3.0 to 0.5.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v0.3.0...v0.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	5 years ago
Robin Appelman	d182043e83	explicitly set permissions on newly created folders ... this works around any `umask` that might be set and limiting the folder permissions Signed-off-by: Robin Appelman <robin@icewind.nl>	5 years ago
Jasper Knockaert	40fde94b4d	fix even more brackets ... Signed-off-by: Jasper Knockaert jasper@knockaert.nl	5 years ago
Jasper Knockaert	252d1ae512	fix brakcets	5 years ago
Jasper Knockaert	f9a0380eb1	consolidation of boolean expression	5 years ago