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

Null LdapSession given to cleanUpSession #6

Open
Vhex opened this issue Aug 30, 2013 · 4 comments
Open

Null LdapSession given to cleanUpSession #6

Vhex opened this issue Aug 30, 2013 · 4 comments
Assignees

Comments

@Vhex
Copy link

Vhex commented Aug 30, 2013

I haven't looked into it, but my logs are showing "Null LdapSession given to cleanUpSession" fairly often.

Just me?

@bantunes
Copy link

bantunes commented Sep 1, 2013

I also see it:

[21:17:15] WARN [org.apache.directory.server.ldap.LdapProtocolHandler] - Null LdapSession given to cleanUpSession.

@dwimberger
Copy link
Owner

Likely something related to the Apache Directory Server:

https://issues.apache.org/jira/browse/DIRSERVER-1404
https://issues.apache.org/jira/browse/DIRSERVER-1432

Given the amount of API changes between Apache Directory Server versions, it will require quite some work to upgrade....

@ghost ghost assigned dwimberger Dec 10, 2013
@charithdesilva
Copy link

Guys this is what I did. Went on to source of the version used by Spring (i.e http://svn.apache.org/viewvc/directory/apacheds/tags/1.5.5/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java?view=co) and got the LdapRequestHandler.java class source added to my project. Add on same package path (org.apache.directory.server.ldap.handlers). Then just add following null check on handleMessage method just after LdapSession is loaded.

    if( ldapSession == null )
    {
        LOG.info( "ignoring the message {} received from null session", message  );
        return;
    }

This is same fix given by for the next version of ADS.

@Serverfrog
Copy link

What is the Status?

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

5 participants