Issue Details (XML | Word | Printable)

Key: DIRSERVER-242
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Alex Karasulu
Reporter: Emmanuel Lecharny
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Directory ApacheDS

Ldif parser does not accept comments

Created: 28/Jan/05 08:16 AM   Updated: 24/Jun/06 08:44 PM
Return to search
Component/s: ldap
Affects Version/s: pre-1.0
Fix Version/s: pre-1.0

Time Tracking:
Not Specified

Environment: Not relevant

Resolution Date: 12/Aug/05 01:31 PM


 Description  « Hide
LdifParserImpl class does not accept comments, as defined by RFC 2848, p.6, par. 3 :

try {
LdifParserImpl p = new LdifParserImpl();

LdifEntry entry = p.parse("#comment");

System.out.println(entry.toString());
} catch (Exception e) {
System.out.println(e.getMessage());
}

throws a LdapNamingException, with this message :

org.apache.ldap.common.exception.LdapNamingException: Line 0 [# empty file] does not correspond to an LDIF entry attribute value pair.
{# empty file}

I marked it Major even if a workaround could be to strip comments from ldif files, as many of them contain comments.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Alex Karasulu added a comment - 29/Jan/05 01:09 AM
Excellent find! These are the kinds of bug reports that you can't put a price tag on. I will add the entire trial of bugs to the 0.8.1 bug fix release and get to work on them immediately.

If you need to get at the sources SVN is the only option at the moment. This is because we did not have an external release of 0.8 ldap-common with the release of the server. The release we internal. Next time we should release all project to fix these issues people are having.

Great job Emmanuel - keep em comming!

Cheers,

Alex


Alex Karasulu added a comment - 12/Aug/05 01:31 PM

Emmanuel Lecharny added a comment - 24/Jun/06 08:44 PM
done