aboutsummaryrefslogtreecommitdiffstats
path: root/apps/user_ldap/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #34443 from nextcloud/feat/add-enabled-user-backendCôme Chilliet2023-07-035-63/+111
|\ | | | | Add IProvideEnabledStateBackend interface
| * Add method to list disabled users to IProvideEnabledStateBackendCôme Chilliet2023-06-292-0/+16
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * Fix since annotations and add boolean return type for setUserEnabledCôme Chilliet2023-06-202-3/+4
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * Use DI for DeletedUsersIndex and fix testsCôme Chilliet2023-05-252-24/+33
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * [user_ldap] Add per-connection setting for marking remnants as disabledCôme Chilliet2023-05-233-2/+6
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * Use the new IProvideEnabledStateBackend interface in user_ldapCôme Chilliet2023-05-233-40/+58
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #38457 from nextcloud/fix/improve-ldap-offset-search-perfArthur Schiwon2023-06-221-2/+9
|\ \ | | | | | | Use default page size for jumping to desired offset
| * | Fix paged search when offset is not a multiple of default page sizeCôme Chilliet2023-06-201-2/+8
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | Use default page size for jumping to desired offsetCôme Chilliet2023-05-251-2/+3
| |/ | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* / Refactors "strpos" calls in /apps/user_ldap to improve code readability.Faraz Samapoor2023-06-029-16/+16
|/ | | | Signed-off-by: Faraz Samapoor <f.samapoor@gmail.com>
* Merge pull request #37903 from nextcloud/fix/user_ldap-fix-multiple-ldap-supportCôme Chilliet2023-05-022-21/+5
|\ | | | | Fix multiple LDAP configuration support by fixing AccessFactory
| * Add comment about Manager instance not being sharedCôme Chilliet2023-04-271-0/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * Fix multiple LDAP configuration support by fixing AccessFactoryCôme Chilliet2023-04-242-21/+4
| | | | | | | | | | | | | | It must not reuse the same OCA\User_LDAP\User\Manager instance for several Access instances. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Clear up return typesCôme Chilliet2023-04-273-6/+12
| | | | | | | | | | | | usersInGroup index by int for BC, searchInGroup index by uid (string). Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Let OC\Group\Group handle the fallback and remove default implementation ↵Côme Chilliet2023-04-271-11/+2
| | | | | | | | | | | | from ABackend Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Optimize retrieving display name when searching for users in a groupCarl Schwan2023-04-272-4/+17
| | | | | | | | | | | | | | | | | | This is recurrent scenario that we are searching for users and then for each users we fetch the displayName. This is inefficient, so instead try to do one query to fetch everything (e.g. Database backend) or use the already existing DisplayNameCache helper. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Merge pull request #36784 from nextcloud/fix/user_ldap-update-groups-fail-getDaniel2023-04-241-1/+12
|\ \ | |/ |/| Properly test and log when a group is not found
| * Properly test and log when a group is not foundCôme Chilliet2023-02-201-1/+12
| | | | | | | | | | | | | | Avoid an error about passing null instead of IGroup to the event constructor, instead skip the failed group and log the problem. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | code stylingMarc Hefter2023-04-241-5/+5
| | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@march42.net>
* | Update apps/user_ldap/lib/User/User.phpMarc Hefter2023-04-241-1/+1
| | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@march42.net>
* | removed profile data from LDAP will get removedMarc Hefter2023-04-211-12/+18
| | | | | | | | | | | | | | | | If attribute mapping is configured and no value present in LDAP, the according profile field is emptied. Removing an attribute e.g. phone from LDAP will cause the phone number being removed from profile. Signed-off-by: Marc Hefter <marchefter@gmail.com>
* | Merge pull request #36565 from march42/feature/ldap_update_profileSimon L2023-04-204-1/+212
|\ \ | | | | | | [user_ldap] Update profile from LDAP fields
| * | optimized handling of user profile data changeMarc Hefter2023-04-141-1/+23
| | | | | | | | | | | | | | | | | | | | | Check profile data checksum before updating user profile, to ensure data has changed. Write checksum to user settings and cache. Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | error handling in update profile from LDAPMarc Hefter2023-04-111-1/+7
| | | | | | | | | | | | | | | | | | added error message on InvalidArgumentException Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | trying to make github-code-scanning bot happyMarc Hefter2023-04-061-2/+3
| | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | added simple data conversion for LDAP attributesMarc Hefter2023-04-061-7/+19
| | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | handling, formatting of postalAddress attribute syntaxMarc Hefter2023-04-061-24/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | replace '$' with ', ' delimiter for address property reformatted some code to 80 column early check and return, if wasRefreshed('profile') removed FIXMEs after digging and double checking Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | nice up the code handling AccountManagerMarc Hefter2023-04-063-69/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | merging defaultScopes from DEFAULT_SCOPES and account_manager.default_property_scope removing unneccessary profileScope setting (using config.php instead) honoring admin choice 'profile.enabled'=>false in config.php moved checking for empty array to updateProfile function corrected some typos and cleaned some comments Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | refined code, to be independend from OCP\IUserMarc Hefter2023-04-064-42/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | rework updateProfile in user_ldap/lib/User/User.php some cleanup at processAttributes in user_ldap/lib/User/User.php rearranged Fediverse attribute, to match profile layout Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | fixing Psalm messagesMarc Hefter2023-04-061-8/+8
| | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | handling updateProfile with array of valuesMarc Hefter2023-04-064-66/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | using an array to buffer profile updates, like suggested by @come-nc clean some code and remove unneccessary redundancy added the Fediverse profile property Co-Authored-By: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | CleanUp: removed redundant storing profile attributes in user settingsMarc Hefter2023-04-061-2/+0
| | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | CleanUp: removed redundant condition if blockMarc Hefter2023-04-061-4/+1
| | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | CleanUp: tidy checking for empty profileScopeMarc Hefter2023-04-061-1/+1
| | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | Fix: Parameter $scope has no provided typeMarc Hefter2023-04-061-4/+3
| | | | | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | CleanUp: remove unneeded UNSET, to unclutter codeMarc Hefter2023-04-061-7/+0
| | | | | | | | | | | | | | | Signed-off-by: march42 <marchefter@gmail.com> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | cleanup: removed unnecessary imports of IAccountManager, AccountManager, ↵Marc Hefter2023-04-061-4/+1
| | | | | | | | | | | | | | | | | | | | | IDBConnection Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | fixed error: Array to string conversion at user_ldap/lib/User/User.php#639Marc Hefter2023-04-061-1/+1
| | | | | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | bugfix: changed forgotten userName, to getUsernameMarc Hefter2023-04-061-2/+2
| | | | | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | added user profile scope settingMarc Hefter2023-04-063-10/+19
| | | | | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | feature addition: [user_ldap] update user profile from LDAPMarc Hefter2023-04-061-20/+22
| | | | | | | | | | | | | | | Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | feature addition: [user_ldap] update user profile from LDAP; WIP; fixing ↵Marc Hefter2023-04-064-4/+40
| | | | | | | | | | | | | | | | | | | | | some uggly copy-and-paste errors; testing functionality; preparing and editing the documentation Signed-off-by: Marc Hefter <marchefter@march42.net> Signed-off-by: Marc Hefter <marchefter@gmail.com>
| * | feature addition: [user_ldap] update user profile from LDAP; WIP ↵Marc Hefter2023-04-064-0/+133
| | | | | | | | | | | | | | | | | | work-in-progress; TODO update profile Signed-off-by: Marc Hefter <marchefter@march42.net>
* | | perform exact Ldap search as well to make sure it works for attributes ↵Julien Veyssier2023-04-131-0/+5
|/ / | | | | | | | | | | without a substr matching rule Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
* | Merge pull request #37575 from nextcloud/fix/user_ldap-log-suffix-namesArthur Schiwon2023-04-051-0/+9
|\ \ | | | | | | Log a warning when a user or group is mapped with a suffix
| * | Log a warning when a user or group is mapped with a suffixCôme Chilliet2023-04-041-0/+9
| | | | | | | | | | | | | | | | | | | | | Most of the time, this is the result of a misconfiguration by the admin and is not what is expected. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | Fix types for reading and writing config valuesjld31032023-04-052-7/+7
|/ / | | | | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* | fix undefined method getExtStorageHome()Arthur Schiwon2023-03-222-1/+10
| | | | | | | | | | | | | | - adds a type check - defines missing method in OfflineUser Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | Merge pull request #37006 from nextcloud/ldap-unmark-dhArthur Schiwon2023-03-201-2/+0
|\ \ | | | | | | also unmark deleted ldap user when checking the ldap entry
| * | also unmark deleted ldap user when checking the ldap entryRobin Appelman2023-03-021-2/+0
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>