[jira] Commented: (DIRSERVER-185) Twix Support for LDAP Controls

["Emmanuel Lecharny (JIRA)" <ji...@apache.org>]

    [ http://issues.apache.org/jira/browse/DIRSERVER-185?page=comments#action_12368392 ] 

Emmanuel Lecharny commented on DIRSERVER-185:
---------------------------------------------

As I said, it as postponed for 1.1 release. So, right now, it's a work in progress :

 - the Control class has been renamed ControlImpl, and it implements the javax.naming.ldap.Control interface.
 - There will be no more twix or snickers implementation. Only a ADS implementation.

I guess that then your suggestion will be fullfiled :)

> Twix Support for LDAP Controls
> ------------------------------
>
>          Key: DIRSERVER-185
>          URL: http://issues.apache.org/jira/browse/DIRSERVER-185
>      Project: Directory ApacheDS
>         Type: Improvement
>     Versions: pre-1.0
>     Reporter: Van Nhu
>     Assignee: Emmanuel Lecharny
>      Fix For: 1.1.0
>  Attachments: TwixProviderTest.java, better_twix_support_for_ldap_controls.patch
>
> 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

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira