++1 personally I never liked this distinction of core and extra this only made the things worse, I don't see
how some control can be treated as 'belongs to core' and other as 'extra', IMHO a very bad move, and
I would like to get rid of this if possible to make it right

On Wed, May 1, 2013 at 2:19 PM, Emmanuel Lécharny <elecharny@gmail.com> wrote:
Hi guys,

I will move the syncrepl controls and the pwdModify extended operation
code from ldap-api-extra-codec(-api) to api-ldap-codec-core. The
rational is that I would like to extend the LdapConnection API to
support the pwdModify() operation directly, without having to go through
the creation of a PwdModifyRequestImpl instance like in :

        PwdModifyRequestImpl pwdModifyRequest = new PwdModifyRequestImpl();
        pwdModifyRequest.setUserIdentity( Strings.getBytesUtf8(
"cn=User1,ou=system" ) );
        pwdModifyRequest.setOldPassword( Strings.getBytesUtf8( "secret" ) );
        pwdModifyRequest.setNewPassword( Strings.getBytesUtf8(
"BetterSecret" ) );

        // Send the request
        PwdModifyResponse pwdModifyResponse = ( PwdModifyResponse )
connection.extended( pwdModifyRequest );

but instead have :

        // Change the password
        PwdModifyResponse pwdModifyResponse = connection.pwdModify(
"cn=User1,ou=system", "secret", "BetterSecret" );

I currently can't do that as the PwdModifyResponse and
PwdModifyRequestImpl classes aren't visible from the LdapConnection
class. This is the very sae problem for the SYncrepl controls.

thoughts ?

Emmanuel Lécharny

Kiran Ayyagari