Issue Details (XML | Word | Printable)

Key: DIRSERVER-185
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Emmanuel Lecharny
Reporter: Van Nhu
Votes: 0
Watchers: 0
Operations

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

Twix Support for LDAP Controls

Created: 30/Nov/05 03:49 PM   Updated: 21/Apr/07 11:19 AM
Return to search
Component/s: None
Affects Version/s: pre-1.0
Fix Version/s: 1.5.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works better_twix_support_for_ldap_controls.patch 2005-11-30 04:23 PM Van Nhu 33 kB
Java Source File Licensed for inclusion in ASF works TwixProviderTest.java 2005-11-30 03:53 PM Van Nhu 9 kB

Resolution Date: 26/Aug/06 11:00 AM


 Description  « Hide
Please excuse my limited understanding of the LDAP Common source base, but it seems that the support for LDAP controls in the TwixTransformer is not complete. So I've made the following modifications to the LDAP Common module in order to have my Virtual Directory to correctly process the various LDAP controls:

1. Get the Twix Control class to implement the javax.naming.ldap.Control interface. Firstly, this is because Twix Controls are really LDAP Controls, and secondly, this makes it easier to create an org.apache.ldap.common.message.ControlImpl instance from a Twix Control object.

2. Remove the extraneous method declarations in org.apache.ldap.common.message.Control because they are already defined in javax.naming.ldap.Control

3. Fix a NullPointerException in org.apache.asn1new.ldap.pojo.LdapMessage.addControl() method

4. Implement the method org.apache.asn1new.ldap.TwixTransformer.transformControls() so that the Response Controls are propagated to the LDAP client properly.

The patch and the unit tests will be attached here shortly for your reference.

Thanks and Regards,
Van

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.