aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_external/lib
Commit message (Collapse)AuthorAgeFilesLines
* fix(filesexternal): Remove unneeded 3rdparty library usebackport/48540/stable30Joas Schilling2024-10-035-13/+13
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix: Use hashed password in files_external settingsbackport/48359/stable30Louis Chemineau2024-09-261-0/+8
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(files_external): Catch correct exceptionbackport/48345/stable30Marcel Klehr2024-09-261-7/+9
| | | | | | SMB#getFileInfo used to throw an icewind/smb exception, but nowadays throws \OCP\Files\ForbiddenException. This fixes downstream methods to catch the new exception. Signed-off-by: Marcel Klehr <mklehr@gmx.net>
* Merge pull request #47923 from nextcloud/backport/47527/stable30Ferdinand Thiessen2024-09-141-0/+2
|\ | | | | [stable30] fix(files_external): Handling in SFTP UI for custom ports
| * fix: custom port handling in UI for SFTPbackport/47527/stable30Josh Richards2024-09-121-0/+2
| | | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | fix(files_external): Check key exists before accessing itGit'Fellow2024-09-141-4/+8
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | fix(files_external): broken credentials dialogJohn Molakvoæ (skjnldsv)2024-09-131-28/+0
|/ | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* fix: Fix unmodified placeholder replacing the actual value when updatingDaniel Calviño Sánchez2024-09-042-0/+12
| | | | | | | | | | | | | | When updating global storages and user storages a property is not updated by "StoragesService::updateStorage()" if the value matches the unmodified placeholder. However, userglobal storages are not updated through the "StoragesService"; as only the authentication mechanism is updated it is directly done with "saveBackendOptions()" in "IUserProvided" or "UserGlobalAuth". Due to this the unmodified placeholder value needs to be explicitly checked in those cases and replaced by the actual value (note that in this case it is not possible to just skip updating a specific property). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* refactor(files_external): Replace security annotations with respective ↵provokateurin2024-08-144-24/+16
| | | | | | attributes Signed-off-by: provokateurin <kate@provokateurin.de>
* feat: add a specialized writeStream implementation for s3 external storageRobin Appelman2024-07-241-0/+21
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #46073 from nextcloud/fix/save_global_credentialsBenjamin Gaussorgues2024-06-251-3/+9
|\
| * fix: allows admin to edit global credentialsBenjamin Gaussorgues2024-06-241-3/+9
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | fix: Do not log an error when connecting to SFTP without a logged in userDaniel Calviño Sánchez2024-06-181-5/+7
|/ | | | | | | | | | | | | | | | | | | | | When connecting to a SFTP server from a SFTP storage the host key is checked against the known host keys stored in a file in the data directory of the logged in Nextcloud user. The path to the file is (indirectly) got using "OC_App::getStorage", which logs an error if called when there is no logged in user; this can happen, for example, if the storage is used from a background job or a command. Not being able to read or write the file just causes the host key check to be skipped, but it has no other consequence. Moreover, even with logged in users it is likely that the file can not be read either and the check is also skipped, as the file needs to have been manually created by an admin. Due to all that now the path to the file is directly created using a View rather than relying on "OC_App::getStorage" to prevent the unneeded error from being logged. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* chore: Add SPDX headerAndy Scherzinger2024-06-06115-2433/+308
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* fix(files_external): Be extra safe with null handling to please psalmCôme Chilliet2024-04-301-0/+3
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #44658 from nextcloud/fix/migrate-away-from-resource-typeCôme Chilliet2024-04-161-6/+3
|\ | | | | fix: Remove obsolete resource typing
| * fix: Fix small psalm errors in FTP and LDAP connectionsCôme Chilliet2024-04-091-4/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix: Remove obsolete resource typingCôme Chilliet2024-04-091-2/+2
| | | | | | | | | | | | In PHP>=8.1, LDAP and FTP resources are always typed objects Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix: Pass the mountpoint target user to storages without ownerJulius Härtl2024-04-091-0/+1
|/ | | | | | | | | | | | | Storages that do not have a dedicated owner (e.g. groupfolders, external storages) currently always assume the current session user as the owner. This leads to several issues when there is no user session but a node is obtained through a user folder. In order to have the correct user available we need to pass the user that is used to setup a mountpoint along to the storage layer as we generally assume that an owner is available for those. Signed-off-by: Julius Härtl <jus@bitgrid.net>
* chore: Improve phpdoc typing to silence psalm errorsCôme Chilliet2024-04-0211-0/+31
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Apply new coding standard to all filesCôme Chilliet2024-04-0217-17/+17
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Implement option to temporarily set the user sessionJulius Härtl2024-03-191-4/+5
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* feat: add command to scan external storages directlyRobin Appelman2024-03-063-74/+290
| | | | | | the main use case of this over simply scanning through is the ability to provide a username and/or password for cases where login credentials are used Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: Allow to disable multipart copy on external s3 storageJulius Härtl2024-02-241-0/+3
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* mark MOUNT_TYPE_PERSONAl as deprecatedMaxence Lange2024-02-242-0/+4
| | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* full uppercase constMaxence Lange2024-02-247-13/+12
| | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* fix: replace `Account name` by `Login`John Molakvoæ2024-02-1310-11/+11
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
* feat: rename users to account or personVincent Petry2024-02-1313-15/+15
| | | | | | Replace translated text in most locations Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Merge pull request #43349 from nextcloud/fix/case-sensitive-root-smbArthur Schiwon2024-02-121-1/+2
|\ | | | | SMB: allow to list root directory when using case-insensitive option
| * fix(files_external): list root when using SMB case-insensitive optionBenjamin Gaussorgues2024-02-051-1/+2
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | fix(files_external) Add sensitive S3 values redacted by list commandJosh2024-02-071-2/+2
| | | | | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* | chore: Replace OC::$server->getL10N by OCP\Util::getL10N in lib and some appsCôme Chilliet2024-02-051-1/+1
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Fix missing template parameter for IEventListenerCôme Chilliet2024-01-304-1/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor: Migrate IgnoreOpenAPI attributes to OpenAPIprovokateurin2024-01-181-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Correct a typoValdnet2023-11-281-1/+1
| | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* fix(files_external): Fix "Could not find resource main.js to load"Joas Schilling2023-11-271-1/+0
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-2317-40/+35
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* refactor: Rename OpenAPI type aliases to their namespacesjld31032023-11-023-8/+8
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Merge pull request #41053 from nextcloud/bug/smb/in-place-case-renameBenjamin Gaussorgues2023-10-312-0/+24
|\ | | | | fix(files_external): on case insensitive system, block case change
| * fix(files_external): on case insensitive system, block case changeBenjamin Gaussorgues2023-10-232-0/+24
| | | | | | | | | | | | | | | | When a file/directory is renamed to the same name with only case change, the rename fail. We block this kind of rename. The user will have to rename to another name first. Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | fix(external): Get storage before update itJoas Schilling2023-10-261-0/+3
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Return correct status if touch failedLouis Chemineau2023-10-231-11/+13
|/ | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(files_external): basic auth user storage triggerJohn Molakvoæ2023-10-101-0/+1
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
* Merge pull request #39131 from fsamapoor/refactor_lib_files_external_commandsLouis2023-09-2811-204/+135
|\ | | | | Refactors files_external app commands
| * Adds parameter type hints.Faraz Samapoor2023-09-282-2/+2
| | | | | | | | | | | | Co-authored-by: Côme Chilliet <come.chilliet@nextcloud.com> Signed-off-by: Faraz Samapoor <fsa@adlas.at> Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>
| * Refactors files_external app commands.Faraz Samapoor2023-09-2811-206/+137
| | | | | | | | | | | | | | To improve code readability. Signed-off-by: Faraz Samapoor <fsa@adlas.at> Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>
* | Convert isset ternary to null coalescing operatorHamid Dehnavi2023-09-282-4/+4
|/ | | | Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
* Add single status code descriptions for OpenAPIjld31032023-09-271-0/+2
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Merge pull request #40499 from nextcloud/known-mtime-wrapperRobin Appelman2023-09-211-18/+10
|\ | | | | add wrapper for external storage to ensure we don't get an mtime that is lower than we know it is
| * apply known mtime wrapper for external storagesRobin Appelman2023-09-191-18/+10
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>