Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow enabling LDAP paging for LDAP synchronization queries #3675

Closed
2 of 7 tasks
wcarson opened this issue Mar 15, 2018 · 0 comments · Fixed by #3895
Closed
2 of 7 tasks

Allow enabling LDAP paging for LDAP synchronization queries #3675

wcarson opened this issue Mar 15, 2018 · 0 comments · Fixed by #3895
Labels
type/enhancement An improvement of existing functionality
Milestone

Comments

@wcarson
Copy link

wcarson commented Mar 15, 2018

  • Gitea version (or commit ref): 1.4.0+rc1-17-g93f34fd
  • Git version: 2.16.2
  • Operating system: Windows Server 2016
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

We have a fairly large LDAP with thousands of users that is configured to allow a max of 1000 results at a time. Without using LDAP paging we get an error like the following:

[...gitea/models/user.go:1388 SyncExternalUsers()] [E] LDAP Search failed unexpectedly! (LDAP Result Code 4 "Size Limit Exceeded": )

As far as I can tell (I'm not familiar with Go), search results paging is supported, so this is definitely doable.

@lunny lunny added the type/enhancement An improvement of existing functionality label Mar 16, 2018
@lafriks lafriks added this to the 1.5.0 milestone May 4, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants