aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/External
Commit message (Collapse)AuthorAgeFilesLines
...
| * don't overwrite the etag from storage backends that already provide "good" etagsRobin Appelman2022-04-041-1/+2
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Use querybuilder for share external mountpointCarl Schwan2022-04-061-7/+9
|/ | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* invalidate mount cache after accepting or renaming federated shareRobin Appelman2022-03-251-0/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* adjust tests to new fs setupRobin Appelman2022-03-041-2/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* store mountprovider for each mount in the mounts tableRobin Appelman2022-02-231-1/+1
| | | | | | this enabled more fine grained filesystem setup Signed-off-by: Robin Appelman <robin@icewind.nl>
* improve errors for external share availability checkRobin Appelman2022-02-211-7/+5
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Remove inefficient fed share scannerVincent Petry2022-01-101-70/+0
| | | | | | | | | | Remove scanAll which relies on the "shareinfo" endpoint that returns the full cache tree. The latter can become big for big shares and result in timeouts. Furthermode, the full tree would be retrieved again for each and every detected change which can become expensive quickly. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Handle LocalServerException when scanning external sharesCarl Schwan2022-01-061-2/+11
| | | | | | | | When remoteIsOwnCloud trows LocalServerException, the storage is unavailable and instead of crashing the scanner, ignore the specific storage. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Fix passing on the parameterJoas Schilling2022-01-051-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Improve api documentation for file sharing external storageCarl Schwan2022-01-031-28/+24
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Handle external share with invalid hostCarl Schwan2021-12-201-3/+11
| | | | | | | | remoteIsOwnCloud might throw an exception when the host is localhost. Handle this case instead of aborting completely. The behavior is the same as that is done 10 lines under it Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* move files_sharing to IBootStrapRobin Appelman2021-10-191-14/+17
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Let users choose a share_folderHinrich Mahler2021-09-061-1/+1
|
* Fix remote share deletion when deleting userVincent Petry2021-07-271-23/+54
| | | | | | | When deleting a user, we should only delete the direct remote user shares or the remote group based subshares. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Replace ILogger with LoggerInterface in remote share managerVincent Petry2021-07-271-10/+10
| | | | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com> Co-authored-by: Carl Schwan <carl@carlschwan.eu>
* Fix remote group share decline+accept code pathVincent Petry2021-07-271-1/+7
| | | | | | | | When declining a remote group share through the dialog that appears when notifications are off, the mount point is now correctly saved when re-accepting. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix external share manager with multiple user groupsVincent Petry2021-07-271-8/+15
| | | | | | Use query builder with proper matching for finding the group names. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix remote group share API interactionsVincent Petry2021-07-271-8/+36
| | | | | | | | | | Accepting and declining can now be done repeatedly on both the parent group share and sub-share with the same effects. Added unit tests to cover these cases, and also when the same operation is repeated. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix re-accepting or re-rejecting remote group sharesVincent Petry2021-07-271-42/+97
| | | | | | | | | | | | | When accepting a group share, a sub-share entry is created which also has a different id. When accepting or rejecting the sub-share, simply update the "accepted" flag instead of trying to re-insert the entry. Adjust getShare to also properly validate group share membership when called on a sub-share id. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Add logging to external shares managerVincent Petry2021-07-271-13/+24
| | | | | | | Instead of just returning false, also log the exception to make debugging database issues easier. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix received federated group sharesVincent Petry2021-07-271-9/+41
| | | | | | | | | | | | Fix pending shares endpoint to consider user-specific sub-entries for group shares whenever a share was accepted or declined. Added unit test for adding remote group shares. Fixed "removeUserShares" to not send a remote request as we never send remote requests for group shares. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Correctly check the reception of a remote feedbackLouis Chemineau2021-06-091-2/+2
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-047-10/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Rename External storages to External storageszaimen2021-05-201-1/+1
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* catch notfound and forbidden exception in smb::getmetadataRobin Appelman2021-03-041-1/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Bump nextcloud/coding-standard from 0.3.0 to 0.5.0dependabot-preview[bot]2021-02-182-3/+3
| | | | | | | | | | 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>
* Bump doctrine/dbal from 2.12.0 to 3.0.0Christoph Wurst2021-01-081-81/+105
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update all license headers for Nextcloud 21Christoph Wurst2020-12-164-3/+4
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Adjust defaultJulius Härtl2020-12-081-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Move remoteId of remote reshares to stringJulius Härtl2020-12-081-3/+3
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Improve CertificateManager to not be user context dependentMorris Jobke2020-11-032-2/+2
| | | | | | | | | * removes the ability for users to import their own certificates (for external storage) * reliably returns the same certificate bundles system wide (and not depending on the user context and available sessions) The user specific certificates were broken in some cases anyways, as they are only loaded if the specific user is logged in and thus causing unexpected behavior for background jobs and other non-user triggered code paths. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Format code to a single space around binary operatorsChristoph Wurst2020-10-052-18/+18
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headers for Nextcloud 20 (again)Christoph Wurst2020-09-071-0/+1
| | | | | | | There are still lots of outdated headers, so time for another round of updates. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Do not fail if share for mountpoint is no longer availableJulius Härtl2020-09-031-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Update the license headers for Nextcloud 20Christoph Wurst2020-08-241-0/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* When no user is logged in null is usedJoas Schilling2020-07-291-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Move federated_share_added into a typed eventMorris Jobke2020-07-231-36/+17
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Pass $path to ocmPermissions2ncPermissionsDaniel Kesselberg2020-06-251-1/+1
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Update share type constant usageJoas Schilling2020-06-241-7/+7
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Do not only catch Exceptions but any Throwable during rmt share deleteChristoph Wurst2020-06-021-2/+2
| | | | | | | | | | When a remote share is deleted, we notify the remove instance. This may trigger various error conditions, which we want to catch in order to avoid undeletable shares. The try-catch, however, did only capture exceptions, so things like TypeErrors were not caught and caused the process to fail hard. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headers for 19Christoph Wurst2020-04-292-0/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add method to storage backends to get directory content with metadataRobin Appelman2020-04-201-1/+1
| | | | | | | | | | | | | | | | | Currently you need to use `opendir` and then call `getMetadata` for every file, which adds overhead because most storage backends already get the metadata when doing the `opendir`. While storagebackends can (and do) use caching to relief this problem, this adds cache invalidation dificulties and only a limited number of items are generally cached (to prevent memory usage exploding when scanning large storages) With this new methods storage backends can use the child metadata they got from listing the folder to return metadata without having to keep seperate caches. Signed-off-by: Robin Appelman <robin@icewind.nl>
* Add visibility to all constantsChristoph Wurst2020-04-102-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-103-17/+10
| | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use elseif instead of else ifChristoph Wurst2020-04-102-4/+4
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update the license headers for Nextcloud 19Christoph Wurst2020-03-312-0/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Use the short array syntax, everywhereChristoph Wurst2020-03-262-15/+15
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update license headersChristoph Wurst2019-12-057-7/+8
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* FixesRoeland Jago Douma2019-11-181-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Respect the accepted flag for group and user sharesJoas Schilling2019-11-121-1/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>