Skip to content

Commit

Permalink
Merge pull request #6211 from gerlingsm/fix/ldap_configuration
Browse files Browse the repository at this point in the history
Fix wrong checks of empty string in LdapUser context
  • Loading branch information
solth authored Sep 4, 2024
2 parents 6b2ded0 + dee50b0 commit 02182a6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Kitodo/src/main/java/org/kitodo/production/ldap/LdapUser.java
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ public static String toHexString(byte[] bytes) {

@Override
public Attributes getAttributes(String name) throws NamingException {
if (StringUtils.isBlank(name)) {
if (StringUtils.isNotBlank(name)) {
throw new NameNotFoundException();
}
return (Attributes) this.attributes.clone();
Expand All @@ -292,7 +292,7 @@ public Attributes getAttributes(Name name) throws NamingException {

@Override
public Attributes getAttributes(String name, String[] ids) throws NamingException {
if (StringUtils.isBlank(name)) {
if (StringUtils.isNotBlank(name)) {
throw new NameNotFoundException();
}

Expand Down
2 changes: 1 addition & 1 deletion Kitodo/src/main/resources/messages/messages_en.properties
Original file line number Diff line number Diff line change
Expand Up @@ -680,7 +680,7 @@ lastPage=Last page
lastUpdatedBy=Last updated by
later=Please try again in a few minutes.
ldapKonfigurationSchreiben=Write LDAP configuration
ldapWritten=LDAP configuration successfully written for user\: ????
ldapWritten=LDAP configuration successfully written for user\:
# ldapgroupSaving is used in ldapgroupEdit.xhtml - line 39
ldapGroupSaving=Saving LDAP group...
ldapGroup=LDAP group
Expand Down

0 comments on commit 02182a6

Please sign in to comment.