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

ldapadmin - creating account is broken #1152

Closed
pmauduit opened this issue Dec 15, 2015 · 4 comments
Closed

ldapadmin - creating account is broken #1152

pmauduit opened this issue Dec 15, 2015 · 4 comments
Milestone

Comments

@pmauduit
Copy link
Member

On sdi.georchestra.org, getting the following trace when trying to create a new account:

org.springframework.ldap.NameNotFoundException: [LDAP: error code 32 - No Such Object]; nested exception is javax.naming.NameNotFoundException: [LDAP: error code 32 - No Such Object]; remaining name 'uid=pmauduit2,ou=users'
    at org.springframework.ldap.support.LdapUtils.convertLdapException(LdapUtils.java:183)
    at org.springframework.ldap.core.LdapTemplate.executeWithContext(LdapTemplate.java:820)
    at org.springframework.ldap.core.LdapTemplate.executeReadOnly(LdapTemplate.java:803)
    at org.springframework.ldap.core.LdapTemplate.lookup(LdapTemplate.java:935)
    at org.georchestra.ldapadmin.ds.AccountDaoImpl.findByUID(AccountDaoImpl.java:311)
    at org.georchestra.ldapadmin.ds.AccountDaoImpl.insert(AccountDaoImpl.java:104)
    at org.georchestra.ldapadmin.ws.newaccount.NewAccountFormController.create(NewAccountFormController.java:153)

@vdorut
Copy link
Contributor

vdorut commented Dec 21, 2015

We need to update PENDING_USERS to PENDING in LDAP directory on sdi.georchestra.org.
After updating my local LDAP directory i dont have this issue anymore.
@pmauduit

@fvanderbiest
Copy link
Member

We need to update PENDING_USERS to PENDING in LDAP directory on sdi.georchestra.org.

To my knowledge, this is already the case.

@fvanderbiest
Copy link
Member

See #1109 (comment)

@Vampouille
Copy link
Contributor

It seems that sometimes ldapTemplate.lookup() does not return null but instead throw a NameNotFoundException. So I suggest to change exception to NameNotFoundException here :

https://github.com/georchestra/georchestra/blob/master/ldapadmin/src/main/java/org/georchestra/ldapadmin/ds/AccountDaoImpl.java#L313

Then we should catch NameNotFoundException here :

https://github.com/georchestra/georchestra/blob/master/ldapadmin/src/main/java/org/georchestra/ldapadmin/ds/AccountDaoImpl.java#L108

vdorut added a commit to vdorut/georchestra that referenced this issue Dec 22, 2015
pmauduit added a commit that referenced this issue Dec 22, 2015
@fvanderbiest fvanderbiest added this to the 15.12 milestone Feb 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants