Skip to content

Loading…

Ldap improvements #76

Merged
merged 4 commits into from

2 participants

@lyncodev
DSpace Repository member

(Developer appreciation) +1 Code is fine

helix84 added some commits
@helix84 helix84 [DS-1078] LDAP group to DSpace group on login 79c0142
@helix84 helix84 consolidate LDAP and LDAPHierarchical auth
Thanks to the previously merged DS-1180, the LDAPHierarchicalAuthentication
method gaind the capabilities of LDAPAuthentication. This commit renames
LDAPHierarchicalAuthentication to LDAPAuthentication and removes the
original LDAPAuthentication.
3c45c1b
@helix84 helix84 fix the special case of anonymous LDAP search
Hierarchical LDAP originally assumed that when search.user is not
set, anonymous search is performed. We now assume that if it's
not set, simple LDAP authentication is performed. That retains
configuration backward comaptibility for the two most common cases.
The special case of anonymous search now requires search.anonymous
to be set to true, which means old installations using this setup
will have to add this config property.
8c43413
@helix84 helix84 fix whitespace in LDAPAuthentication.java 03f0ed9
@helix84 helix84 merged commit d0fa708 into DSpace:master
@kosarko kosarko pushed a commit to kosarko/DSpace that referenced this pull request
jm fixed #76 by changing the notation from \v{} to {\v} d3e526b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 17, 2012
  1. @helix84
  2. @helix84

    consolidate LDAP and LDAPHierarchical auth

    helix84 committed
    Thanks to the previously merged DS-1180, the LDAPHierarchicalAuthentication
    method gaind the capabilities of LDAPAuthentication. This commit renames
    LDAPHierarchicalAuthentication to LDAPAuthentication and removes the
    original LDAPAuthentication.
  3. @helix84

    fix the special case of anonymous LDAP search

    helix84 committed
    Hierarchical LDAP originally assumed that when search.user is not
    set, anonymous search is performed. We now assume that if it's
    not set, simple LDAP authentication is performed. That retains
    configuration backward comaptibility for the two most common cases.
    The special case of anonymous search now requires search.anonymous
    to be set to true, which means old installations using this setup
    will have to add this config property.
  4. @helix84
Something went wrong with that request. Please try again.