aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/User
Commit message (Collapse)AuthorAgeFilesLines
* fix(users): improve recently active searchBenjamin Gaussorgues2024-10-301-28/+37
| | | | | | | | | - 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>
* chore(deps): Bump nextcloud/coding-standard in /vendor-bin/cs-fixerdependabot/composer/vendor-bin/cs-fixer/nextcloud/coding-standard-1.3.2dependabot[bot]2024-10-191-7/+7
| | | | | | | | | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 1.3.1 to 1.3.2. - [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/v1.3.1...v1.3.2) --- updated-dependencies: - dependency-name: nextcloud/coding-standard dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(db): Apply query prepared statementsdbQueriesExecStmt2Git'Fellow2024-10-171-8/+8
| | | | | | | | | | | | | | | | Fix: psalm fix: bad file fix: bug chore: add batch chore: add batch chore: add batch fix: psalm
* chore: Use more gender neutral languageprovokateurin2024-10-141-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(BackgroundJobs): Adjust intervals and time sensitivitiesprovokateurin2024-10-081-3/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Skip users that still exist in backendfix/resiliant-user-removalFerdinand Thiessen2024-09-263-9/+28
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Make user removal more resilientFerdinand Thiessen2024-09-264-63/+188
| | | | | | | | | | | | | | 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>
* fix(users): Don't crash if disabled user is missing in the databaseartonge/fix/prevent_missing_users_from_crashing_disabled_users_searchLouis Chemineau2024-09-232-9/+18
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* chore: Add proper deprecation dates where missingFerdinand Thiessen2024-09-201-1/+1
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-193-4/+6
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #45020 from Noodlesalat/fix-lastlogin-update-token-loginJohn Molakvoæ2024-09-171-0/+1
|\
| * fix: update last_login timestamp for token based-loginsFabian Dreßler2024-09-061-0/+1
| | | | | | | | | | | | fixes #31075 and maybe #32953 Signed-off-by: Fabian Dreßler <nudelsalat@clouz.de>
* | fix: Handle null checks with the ?? operatorrefactor/elvisChristoph Wurst2024-09-151-1/+1
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | refactor: Use the elvis operatorChristoph Wurst2024-09-151-1/+1
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-1/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-253-6/+6
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix(SetupCheck): Properly check public access to data directoryFerdinand Thiessen2024-08-081-1/+1
| | | | | | | | | | | | | When checking for public (web) access to the data directory the status is not enough as you might have a webserver that forwards to e.g. a login page. So instead check that the content of the file matches. For this the `.ncdata` file (renamed from `.ocdata`¹) has minimal text content to allow checking. ¹The file was renamed from the legacy `.ocdata`, there is a repair step to remove the old one. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(Session): avoid race conditions on clustered setupsArthur Schiwon2024-07-101-44/+17
| | | | | | | | - re-stablishes old behaviour with cache to return null instead of throwing an InvalidTokenException when the token is cached as non-existing - token invalidation and re-generation are bundled in a DB transaction now Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #46123 from nextcloud/feat/user-password-hashPytal2024-07-093-1/+61
|\ | | | | feat: Allow getting/setting the password hash of a user
| * feat: Validate password hashChristopher Ng2024-07-081-0/+4
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * perf: Return cached password hashChristopher Ng2024-07-081-0/+3
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * feat: Allow getting/setting the password hash of a userChristopher Ng2024-07-082-0/+23
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * feat: Implement IPasswordHashBackend in database user backendChristopher Ng2024-07-081-1/+31
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | fix: getLastLoggedInUsers moved from AllConfig/IConfig to IUserManager/Manageryemkareems2024-07-081-0/+43
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: limit and fixLimit removed. negative limit handled in controller. ↵yemkareems2024-07-081-15/+0
| | | | | | | | | | | | removed getUsersSortedByLastLogin from Manager and instead used the config in controller Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: removed default limit of 25. if null is given all users are fetched or ↵yemkareems2024-07-081-1/+1
| | | | | | | | | | | | if limit is given limit number of users are fetched Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: removed references to old disabled users code. refactored query as per ↵yemkareems2024-07-081-18/+3
| | | | | | | | | | | | getDisplayNames function. limit and offset added to query. default limit set to 25. Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: removed the params related to sortMode and order since it sorts by ↵yemkareems2024-07-081-2/+2
| | | | | | | | | | | | lastLogin Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: doc blocks addedyemkareems2024-07-081-0/+3
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: search and searchDisplayNames reverted to how it was initially as sort ↵yemkareems2024-07-081-2/+1
| | | | | | | | | | | | and order are not required here Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: search and searchDisplayNames reverted to how it was initially as sort ↵yemkareems2024-07-081-68/+9
| | | | | | | | | | | | and order are not required here Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: rebased the branch with master and resolved conflictsyemkareems2024-07-083-28/+37
| | | | | | | | | | | | fix: added a new endpoint users/recent and getting users based on last login info in the same. Reverted old code that was breaking LDAP Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: change orderBy and sort to sortMode and sortOrder. default it to uid ↵yemkareems2024-07-083-36/+41
| | | | | | | | | | | | asc. enable email search by changing query Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: cs fix and psalm ci related changesyemkareems2024-07-082-3/+5
| | | | | | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* | fix: sort the user getDisplayNames based on lastLogin. Default sort order is ↵yemkareems2024-07-083-14/+25
| | | | | | | | | | | | lastLogin DESC Signed-off-by: yemkareems <yemkareems@gmail.com>
* | feat: cs fix run on the changed files and default order by last_login desc. ↵yemkareems2024-07-081-5/+5
| | | | | | | | | | | | Also last_login sort logic changed Signed-off-by: yemkareems <yemkareems@gmail.com>
* | feat: add ability to sort users by last_login, uid or displayName. one of ↵yemkareems2024-07-081-8/+68
|/ | | | | | these needs to be passed as orderBy and sort can be ASC or DESC Signed-off-by: yemkareems <yemkareems@gmail.com>
* Fix: Make out of office replacement nullableHamza Mahjoubi2024-07-031-4/+4
| | | | Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>
* Feat: Allow users to select another user as their out-of-office replacementHamza Mahjoubi2024-07-012-1/+17
| | | | Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>
* Merge pull request #40147 from ↵John Molakvoæ2024-05-311-2/+3
|\ | | | | | | summersab/refactor/OC-Server-getNotificationManager
| * chore: php lint fix importJohn Molakvoæ2024-05-301-1/+1
| | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * Merge branch 'master' into refactor/OC-Server-getNotificationManagerJohn Molakvoæ2024-05-3013-350/+408
| |\ | | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * | Refactor `OC\Server::getNotificationManager`Andrew Summers2023-08-291-2/+3
| | | | | | | | | | | | Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com>
* | | Merge branch 'master' into refactor/OC-Server-getCsrfTokenManagerJohn Molakvoæ2024-05-3013-349/+407
|\ \ \ | | |/ | |/| Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * | Merge pull request #45411 from ↵Daniel2024-05-291-0/+4
| |\ \ | | | | | | | | | | | | | | | | nextcloud/fix/auth/selective-token-activity-update fix(auth): Update authtoken activity selectively
| | * | fix(auth): Update authtoken activity selectivelyChristoph Wurst2024-05-211-0/+4
| | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
| * | | chore: Add SPDX headerAndy Scherzinger2024-05-2413-300/+38
| | | | | | | | | | | | | | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
| * | | fix: Also search in email field for disabled usersCôme Chilliet2024-05-231-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | To match what is done for Database backend for enabled users Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | feat: Add back searching in disabled user listCôme Chilliet2024-05-231-2/+10
| |/ / | | | | | | | | | | | | | | | | | | | | | When disabled users where moved to their own endpoint we overlooked search, so adding it back. The search is done case-insensitive in uid and display name. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | fix: delete user credentials stored in storages_credentials when user gets ↵yemkareems2024-05-161-1/+6
| | | | | | | | | | | | | | | | | | deleted Signed-off-by: yemkareems <yemkareems@gmail.com>