aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #49277 from nextcloud/backport/48675/stable30Côme Chilliet2024-11-141-2/+5
|\ | | | | [stable30] DNS: do not query CNAME if A succeeded already
| * fix: remove superflous linebackport/48675/stable30georglauterbach2024-11-141-1/+0
| | | | | | | | Signed-off-by: georglauterbach <44545919+georglauterbach@users.noreply.github.com>
| * fix: do not query CNAME if A succeeded alreadygeorglauterbach2024-11-141-1/+5
| | | | | | | | Signed-off-by: georglauterbach <44545919+georglauterbach@users.noreply.github.com>
* | fix(textprocessing): accept scheduling a task if there are equivalent ↵backport/49260/stable30Julien Veyssier2024-11-141-1/+5
|/ | | | | | taskprocessing providers only Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* Merge pull request #49098 from nextcloud/backport/48933/stable30Richard Steinmetz2024-11-131-1/+7
|\ | | | | [stable30] Clear pending two factor tokens also from configuration
| * fix: Handle exception when clearing previously removed two factor tokensbackport/48933/stable30Daniel Calviño Sánchez2024-11-051-1/+5
| | | | | | | | | | | | | | | | If a token was already removed from the database but not from the configuration clearing the tokens will try to remove it again from the database, which caused a DoesNotExistException to be thrown. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * fix: Clear pending two factor tokens also from configurationDaniel Calviño Sánchez2024-11-051-0/+2
| | | | | | | | | | | | | | | | | | Otherwise as the tokens were removed from the database but not from the configuration the next time that the tokens were cleared the previous tokens were still got from the configuration, and trying to remove them again from the database ended in a DoesNotExistException being thrown. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Merge pull request #49241 from nextcloud/backport/38630/stable30yemkareems2024-11-132-2/+2
|\ \ | | | | | | [stable30] Fix remaining readdir() calls in loops with undesirable false evaluation potential
| * | fix: Clean-up some remaining readdir calls with undesirable false evaluation ↵Josh Richards2024-11-132-2/+2
| | | | | | | | | | | | | | | | | | potential Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | | fix(Middleware): log deprecation when annotation was actually usedbackport/49237/stable30Arthur Schiwon2024-11-131-1/+1
|/ / | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | Merge pull request #49207 from nextcloud/backport/48736/stable30Andy Scherzinger2024-11-121-0/+1
|\ \ | | | | | | [stable30] fix: log a warning when we can't build a background job
| * | fix: log a warning when we can't build a background jobbackport/48736/stable30Robin Appelman2024-11-111-0/+1
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Fix Undefined variable: response notice in case of non-404 errorbackport/49146/stable30Victor Dubiniuk2024-11-081-2/+3
|/ / | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | fix(tests): Add IpAddressClassifier v6 zone ID testbackport/49039/stable30Josh2024-11-071-1/+1
| | | | | | | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* | fix(net): Add IPv6 zone handling to IpAddressClassifierJosh2024-11-061-1/+1
| | | | | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* | fix(security): Handle IPv6 zone IDs used in link-local addressesJosh2024-11-061-0/+1
| | | | | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* | fix(security): Handle IPv6 zone IDs used in link-local addressesJosh2024-11-061-2/+3
|/ | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* Merge pull request #49076 from nextcloud/backport/49075/stable30Benjamin Gaussorgues2024-11-041-7/+5
|\
| * fix(owncloud): Fix ownCloud migration with oauth2 appbackport/49075/stable30Joas Schilling2024-11-041-7/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix(appinfo): Make sure screenshot, author and category are always arraysbackport/48912/stable30Joas Schilling2024-11-011-0/+11
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #49001 from nextcloud/backport/48559/stable30Benjamin Gaussorgues2024-10-302-29/+38
|\
| * fix(sharding): typo in addOrderBybackport/48559/stable30Benjamin Gaussorgues2024-10-302-22/+5
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| * fix(users): improve recently active searchBenjamin Gaussorgues2024-10-301-7/+33
| | | | | | | | | | | | | | | | | | - Remove DISTINCT clause to fix PgSQL - Join user table only if necessary - Don't show people who never connected in active list - Add test Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | Merge pull request #48694 from nextcloud/backport/48689/stable30Git'Fellow2024-10-301-1/+1
|\ \ | |/ |/| [stable30] fix(locking): Accept mixed as value on setTTL
| * fix(locking): Accept mixed as value on setTTLbackport/48689/stable30Git'Fellow2024-10-301-1/+1
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | Merge pull request #48622 from nextcloud/backport/48538/stable30Benjamin Gaussorgues2024-10-301-0/+3
|\ \ | |/ |/|
| * fix: Subadmin can access selfChristopher Ng2024-10-091-0/+3
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | fix: Shipped apps should include the Nextcloud version in the cache busterFerdinand Thiessen2024-10-301-59/+64
| | | | | | | | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix: Fix copying or moving from shared groupfoldersDaniel Calviño Sánchez2024-10-291-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When copying or moving between two local storages the source path (on disk) to copy or move from is got from the unjailed path of the source storage. However, if the source storage has more than one jail getting the unjailed path resolves the most external jail, but the source path needs to be got from the most internal jail instead (the one closer to the local storage). This can happen, for example, with a shared groupfolder: in that case there is an external jail for the shared storage, and one internal jail for the groupfolder storage wrapped by the shared storage. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | fix(config): Mark more configs as sensitivebackport/48947/stable30Joas Schilling2024-10-282-0/+10
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix: crypto made inline for constructor and decrypt error handled in exceptionyemkareems2024-10-281-2/+4
| | | | | | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com> [skip ci]
* | fix: crypto made inline for constructor and decrypt error handled in exceptionyemkareems2024-10-281-6/+7
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: use Icrypto in place of Cyptoyemkareems2024-10-282-4/+4
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: crypto type made not nullable and tests run using ICryptoyemkareems2024-10-282-4/+4
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: encrypt and store password, decrypt and retrieve the sameyemkareems2024-10-282-2/+11
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: Allow overriding shouldApplyQuota check from child classesbackport/48623/stable30Julius Knorr2024-10-281-1/+1
| | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | feat(textprocessing): factorize taskProcessingCompatibleTaskTypesbackport/48863/stable30Julien Veyssier2024-10-241-22/+11
| | | | | | | | Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* | fix(textprocessing): also list types that are available in task processingJulien Veyssier2024-10-241-0/+29
| | | | | | | | Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* | fix(logger): Remove more parameters of other methodsbackport/48603/stable30Joas Schilling2024-10-211-2/+19
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #48763 from ↵Joas Schilling2024-10-211-5/+5
|\ \ | | | | | | | | | | | | nextcloud/dependabot/composer/stable30/symfony-6.4 [stable30] build: Bump symfony/* to 6.4
| * | fix(command): Make method compatible with upstream class (requires 3rdparty ↵Joas Schilling2024-10-211-4/+4
| | | | | | | | | | | | | | | | | | bump) Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | fix(command): Make method compatible with upstream class (backportable)Joas Schilling2024-10-211-3/+3
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #48710 from nextcloud/backport/48682/stable30Kate2024-10-211-0/+1
|\ \ \ | |/ / |/| |
| * | fix(AppConfig): Add external JWT private key to sensitive keysbackport/48682/stable30provokateurin2024-10-151-0/+1
| |/ | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(comments): Document supported types and provide psalm typingJoas Schilling2024-10-181-19/+21
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | feat(comments): Support mentioning emailsJoas Schilling2024-10-181-1/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #48703 from nextcloud/backport/48673/stable30Ferdinand Thiessen2024-10-171-2/+2
|\ \ | | | | | | [stable30] fix(share): Return empty string if no label is set
| * | fix(share): Return empty string if no label is setFerdinand Thiessen2024-10-141-2/+2
| |/ | | | | | | | | | | | | | | | | * Resolves: https://github.com/nextcloud/server/issues/48629 While the database supports NULL, the typing has always said it only returns *string*. So to not break any apps that might trust the typings we should return `''` if the database is set to `NULL`. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix: Skip users that still exist in backendFerdinand Thiessen2024-10-083-9/+28
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix: Make user removal more resilientFerdinand Thiessen2024-10-088-72/+237
|/ | | | | | | | | | | | | | Currently there is a problem if an exception is thrown in `User::delete`, because at that point the user is already removed from the backend, but not all data is deleted. There is no way to recover from this state, as the user is gone no information is available anymore. This means the data is still available on the server but can not removed by any API anymore. The solution here is to first set a flag and backup the user home, this can be used to recover failed user deletions in a way the delete can be re-tried. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>