nextcloud-server

Commit Graph

Author	SHA1	Message	Date
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.	5 years ago
Daniel Kesselberg	be3f4edf1f	explicitly close source stream on encryption storage Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>	5 years ago
Daniel Kesselberg	e60a829b42	explicitly close source stream on local storage Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>	5 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
Johannes Leuker	ac88bcbd5f	Add 'allow Symlinks' as an option to config.php Signed-off-by: Johannes Leuker <j.leuker@hosting.de>	5 years ago
Morris Jobke	24d436cb60	Remove unneeded casts that were found by Psalm In preparation of the update of Psalm from 4.2.1 to 4.3.1+ (see https://github.com/nextcloud/server/pull/24521) Signed-off-by: Morris Jobke <hey@morrisjobke.de>	5 years ago
Jasper Knockaert	81e3ae4753	avoid fread on directories and unencrypted files Reworking the logic in order to first check the filecache and only then reading the fileheader. This in order to solve #21578.	5 years ago
Julius Härtl	98560c80ea	Properly handle result when copying with failure and throw in tests Signed-off-by: Julius Härtl <jus@bitgrid.net>	5 years ago
Christoph Wurst	9ce3ea3368	Update license headers Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	5 years ago
Christoph Wurst	73c7d0dc81	Bump icewind/streams from 0.7.1 to 0.7.2 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	5 years ago
Tigran Mkrtchyan	4f2dc18f58	storage: update IStorage#file_put_contents docs to match usage The current phpdoc of IStorage#file_put_contents doesnt corresponds to it's actual usage in code, e.g. Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>	5 years ago
Christoph Wurst	d89a75be0b	Update all license headers for Nextcloud 21 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>	5 years ago
Tigran Mkrtchyan	f3513f3fe4	files: Local#writeStream should use it's own file_put_contents The OC\Files\Storage\Local#writeStream use system provided file_put_contents. However, it overrides file_put_contents, thus expects that the default behaviour can be different. Use Local#file_put_contents in writeStream to benefit from class specific functionality. Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>	5 years ago
aler9	ac0c7a8fe0	Fix file size computation on 32bit platforms Signed-off-by: aler9 <46489434+aler9@users.noreply.github.com>	5 years ago
John Molakvoæ (skjnldsv)	6a453ac184	Make sure root storage is valid before checking its size Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>	5 years ago

1 2 3 4 5

229 Commits (d9c079937726bfa37a48ef8a2f206741f3fa12e3)