[DIRSERVER-588] ldap server is never shut down, since ldapStarted flag is never set

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.0-RC1
• Fix Version/s: 1.0-RC4, 1.5.0
• Component/s: ldap

Environment

Don't think it's relevant, but here's the details anyway:
-Windows XP
-Java java version "1.5.0_04"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_04-b05)
Java HotSpot(TM) Client VM (build 1.5.0_04-b05, mixed mode, sharing)

• Assignee: Alex Karasulu
• Reporter: George Spalding
• Created: 01/Mar/06 17:24
• Updated: 27/Aug/06 00:54
• Resolved: 27/Aug/06 00:54

Description

Class org.apache.directory.server.jndi.ServerContextFactory

internally uses a flag ldapStarted (and ldapsStarted which I think suffers of the same problem)
that probably should be set when ldap is started, otherwise the ldap server will not shut down when in SeverContextFactory.afterShutdown(...)

I Fixed the code for my purposes and hope you will be able to make use of it:

In ServerContextFactory.startLDAPS( ServerStartupConfiguration, Hashtable):
...
chain = ( IoFilterChainBuilder ) Class.forName( "org.apache.directory.server.ssl.LdapsInitializer", true,
ServerContextFactory.class.getClassLoader() ).getMethod( "init", new Class[]

{ ServerStartupConfiguration.class }

).invoke( null, new Object[]

{ cfg }

);
ldapsStarted=true;//FIX Geospa added this line
...

In ServerContextFactory.startLDAP0( ServerStartupConfiguration, Hashtable):
...
tcpAcceptor.bind( new InetSocketAddress( port ), protocolProvider.getHandler(), acceptorCfg );
ldapStarted=true;//FIX Geospa added this line
...