Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • lemonldap-ng lemonldap-ng
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 314
    • Issues 314
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LemonLDAP NGLemonLDAP NG
  • lemonldap-nglemonldap-ng
  • Issues
  • #2183
Closed
Open
Issue created Apr 30, 2020 by Maxime Besson@maxbes🔧Maintainer

AuthLDAPFilter and LDAPFilter are redundant

Concerned version

Version: 2.0.7

Platform: (Nginx/Apache/Node.js)

Summary

I'm not sure how LDAP filters worked before 2.0, but currently, there are two configuration settings that do the same thing:

If AuthLDAPFilter is not defined, LDAPFilter will be used

But LDAPFilter is used nowhere else.

see ::Portal::Lib::LDAP

Possible fixes

One of these options should be removed to avoid confusion. We must provide an upgrade path to prevent installations that relied on either of these settings to upgrade without breaking.

Assignee
Assign to
Time tracking