Duplicate root entry gets added everytime restarting the server

• Key: DIRSERVER-307
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Major
• Assignee: Alex Karasulu
• Reporter: Endi S. Dewata

Description

Duplicate root entry gets added everytime restarting the server.

Steps to reproduce:
1. Start the server:
    java -jar main/target/apacheds-main-0.9.3-SNAPSHOT.jar main/server.xml

2. Use a command line ldapsearch, execute the following:
    ldapsearch -h localhost -p 10389 -b "dc=apache,dc=org"
    It returns an entry: dc=apache,dc=org.

3. Restart the server.

4. Execute the same search operation.
    This time it returns 2 entries, both are: dc=apache,dc=org.

The more you restart the server, the more duplicate entries you will get. Note: if you use an LDAP browser you probably won't see this problem.

Comments

[Stefan Zoerner]

It is quite easy to reproduce this one, thanks to Endi's detailed instractions.
Comments:
My complete search command looked like this

$ ldapsearch -h localhost -p 10389 -b "dc=apache,dc=org" -s sub "(objectclass=*)"

Using the base scope displays only one entry -- this is probably the cause, why the problem does not show up in the GUI tools I checked (as Endi assumed).
I was not able to create yet another "dc=apache,dc=org" entry on an already running ApacheDS from outside by using ldapmodify (entry is rejected as expected). But it is possible to create entries with different attribute value sets by changing the server.xml configuration between startups.

This leads to the question, what the correct behaviour is, if someone changes the configuration here (add some attributes, for example). The LDAP servers I know have the suffixes as a part of the configuration, but adding a new suffix does not automatically add a corresponding entry. This has to be done after server startup, manually.

[Stefan Zoerner]

Two more comments:
1. The issue exists on the system partition as well.
2. I made a change to the BTreeContextPartition.java file (patch attached)

The patch adds a conditional to the init method. It checks wether the entry for the suffix exists. If not, it creates it.
This should remove at least the problem Endi described in this issue (a skilled person shall check, whether this is a clever change).

But I am unsure, whether it helps to fix other problems, which may arise if sb changes the configuration for the partition between startups in the server.xml (e.g. add some attributes in contextEntry or indexedAttributes ...).

[Alex Karasulu]

make the link for tracking in the future

[Alex Karasulu]

Committed in revision 280166 here:

http://svn.apache.org/viewcvs.cgi?view=rev&rev=280166

[Endi S. Dewata]

The fix has been verified. Thanks.