Skip to content

Ldap improvements #76

Merged
merged 4 commits into from Sep 17, 2012

2 participants

@lyncodev
DSpace Repository member

(Developer appreciation) +1 Code is fine

helix84 added some commits Aug 24, 2012
@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 Sep 17, 2012
@kosarko kosarko pushed a commit to kosarko/DSpace that referenced this pull request May 29, 2015
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
Something went wrong with that request. Please try again.