aboutsummaryrefslogtreecommitdiffstats
path: root/apps/user_ldap/lib
Commit message (Collapse)AuthorAgeFilesLines
* fix(user_ldap): Switch to OCP\IAppConfig and fix Helper constructor callsCôme Chilliet2025-06-106-21/+14
| | | | | | | | Using OCP\AppFramework\Services\IAppConfig is not possible because the Helper is queried from places outside of the application DI container (ajax pages, tests, other applications through ILDAPProviderFactory…) Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): Store the list of used configuration prefixed in appconfigCôme Chilliet2025-06-101-36/+60
| | | | | | | This avoids getting all keys from appconfig, which was triggering loading of lazy configuration on all requests. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): Harmonize parameter obfuscation and serialization accross ↵fix/harmonize-ldap-function-loggingCôme Chilliet2025-06-021-18/+28
| | | | | | | | | logging methods Debug log, profiler and ldap debug log had a different logic for sanitizing of parameters, aligning them. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor: Apply rector refactoringschore/deps/rector-2.0provokateurin2025-05-141-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: replace leagcy OC_Helper calls with OCP\Utilchore/oc-helper-filesizeFerdinand Thiessen2025-05-141-2/+2
| | | | | | | | - Replace legacy calls with OCP\Util - Add missing deprecation notices - Inline implementation in OCP\Util and call it from OC_Helper Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(LDAP): inlcude ldapExpertUsernameAttr in general attribute listfix/noid/ensure-userid-attr-presentArthur Schiwon2025-04-171-0/+1
| | | | | | | | fixes corner cases in which an LDAP record might be loaded and used, where the user is still not mapped - and then this information is missing though expected. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix: Replace all usages of OC_User backend method calls by IUserManagerfix/cleanup-user-backendsCôme Chilliet2025-04-101-1/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(l10n): put placeholder like string into variablefix/l10n-placeholderFerdinand Thiessen2025-04-071-2/+2
| | | | | | Otherwise this is wrongly handled by our translation community. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Migrate all uses of OCP\Template to OCP\Template\ITemplateManagerCôme Chilliet2025-03-061-6/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #51096 from ↵Arthur Schiwon2025-03-061-13/+14
|\ | | | | | | | | nextcloud/fix/noid/ldap-setopt-for-disabling-certcheck fix(LDAP): use ldap_set_option over putenv to disable cert check
| * fix(LDAP): use ldap_set_option over putenv to disable cert checkfix/noid/ldap-setopt-for-disabling-certcheckArthur Schiwon2025-03-031-13/+14
| | | | | | | | | | | | the putenv option was not working reliable anymore anyway Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | feat: Add bases and gidNumber attribute to ldap:test-user-settings outputenh/add-info-to-ldap-test-user-settingsCôme Chilliet2025-03-041-2/+14
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix: remove caching in fetchListOfGroupsDaniel Kesselberg2025-02-271-16/+0
|/ | | | | | | | | | | | When using nested groups without a memberof overlay, then fetchListOfGroups is called from getGroupsByMember without applying the group filter. In some setups, the "unfiltered" result is then written back to the group mapping table. That might cause random "An administrator removed you from group" activities. I was unable to replicate it locally, but we got the feedback that the random activities stopped with the patch applied. Ref: https://github.com/nextcloud/server/issues/42195 Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix(l10n): Improve english source stringsJoas Schilling2025-02-261-1/+1
| | | | | | | - No leading/trailing whitespace - Use asci single quote Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(user_ldap): Check that all user and group bases are in the global oneCôme Chilliet2025-02-251-9/+35
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): Add missing use in ajax endpoint and fix L10N injectionCôme Chilliet2025-02-251-1/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(user_ldap): Improve error detail when saving an incorrect configurationCôme Chilliet2025-02-252-31/+64
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Apply new rector configuration to apps folderCôme Chilliet2025-02-1312-25/+44
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(user_ldap): Add a --clearcache option to ldap:test-user-settingsenh/ldap-add-test-settings-commandCôme Chilliet2025-01-301-0/+9
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): Remove redundant if checkCôme Chilliet2025-01-301-3/+1
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(user_ldap): Add command test-user-settings to test LDAP user settingsCôme Chilliet2025-01-301-0/+227
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(user_ldap): upstream common code into Proxy class and add public ↵Côme Chilliet2025-01-303-47/+59
| | | | | | getters for backends Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): Do not map groups we do not know if they match filterfix/ldap-avoid-false-positive-mappingCôme Chilliet2025-01-272-4/+11
| | | | | | | | When nesting is enabled, filterValidGroups is supposed to check for each groups if it actually exist, because it may not be visible to Nextcloud. So in this codepath we disable automapping of groups. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(LDAP): also log why the connection to main server failedfix/noid/ldap-no-connection-reasonArthur Schiwon2025-01-241-2/+4
| | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #50162 from nextcloud/fix/improve-ldap-avatar-handlingArthur Schiwon2025-01-165-136/+43
|\ | | | | Improve ldap avatar handling
| * fix(user_ldap): Remove now unused class FilesystemHelperCôme Chilliet2025-01-134-39/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix(user_ldap): Strong type User class and fix most type issuesCôme Chilliet2025-01-132-35/+27
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix(user_ldap): Always update avatar from LDAP when we have the dataCôme Chilliet2025-01-131-16/+1
| | | | | | | | | | | | Should be at login, in sync job, and when running ldap:check-user --update Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore(user_ldap): Improve typing in user_ldap User classCôme Chilliet2025-01-131-43/+16
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore(user_ldap): Rename avatar setting method to a more suited nameCôme Chilliet2025-01-131-7/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | feat(updatenotification): Add a limit to user count from LDAP so save ↵Côme Chilliet2025-01-142-13/+15
|/ | | | | | performances Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(user_ldap): retrieve from LDAP first "email shaped" value instead of ↵Thatoo2024-11-201-1/+8
| | | | | first value Signed-off-by: Thatoo <Thatoo@users.noreply.github.com>
* fix(OpenAPI): Adjust array syntax to avoid ambiguitiesfix/openapi/array-syntaxprovokateurin2024-11-053-4/+8
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(psalm): Fix some newly detected issuesrefactor/apps/constructor-property-promotionprovokateurin2024-10-211-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(apps): Use constructor property promotion when possibleprovokateurin2024-10-2136-351/+166
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(db): Apply query prepared statementsdbQueriesExecStmt2Git'Fellow2024-10-171-9/+5
| | | | | | | | | | | | | | | | Fix: psalm fix: bad file fix: bug chore: add batch chore: add batch chore: add batch fix: psalm
* chore(db): Correctly apply query typesGit'Fellow2024-10-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | fix: psalm fix: error fix: add batch fix: fatal error fix: add batch chore: add batch chore: add batch fix: psalm fix: typo fix: psalm fix: return bool fix: revert Manager
* chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-1517-44/+56
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Use more gender neutral languageprovokateurin2024-10-146-7/+7
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(BackgroundJobs): Adjust intervals and time sensitivitiesprovokateurin2024-10-081-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48223 from nextcloud/feat/add-rector-configCôme Chilliet2024-09-232-3/+3
|\ | | | | Add rector config
| * chore: Apply rector configuration to apps folderCôme Chilliet2024-09-202-3/+3
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | chore(legacy): Introduce public version ct plass and drop version methods ↵clean/version-ocpJulius Knorr2024-09-201-1/+4
|/ | | | | | from OC_Util Signed-off-by: Julius Knorr <jus@bitgrid.net>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-1923-84/+84
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: Remove deprecated `ILogger` logging functionschore/remove-iloggerFerdinand Thiessen2024-09-191-2/+3
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* feat: add profile pronounsJohn Molakvoæ (skjnldsv)2024-09-174-0/+14
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* fix(psalm): Fix @throws annotationsfix/psalm/throws-annotationsprovokateurin2024-09-171-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48042 from nextcloud/refactor/elvisChristoph Wurst2024-09-161-2/+1
|\ | | | | refactor: Use the elvis operator + fix: Handle null checks with the ?? operator
| * refactor: Use the elvis operatorChristoph Wurst2024-09-151-2/+1
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #48028 from nextcloud/fix/fix-ldap-setupcheck-crashAndy Scherzinger2024-09-161-4/+4
|\ \ | | | | | | fix(user_ldap): Use %n in plural translation to avoid a crash in L10N
generated by cgit v1.2.3 (git 2.39.1) at 2025-07-26 12:57:34 +0000