If you are willing to merge this in, I can modify anything you find missing.

Updated DefaultLdapConnectionFactory to use the LdapNetworkConnection constructor that takes an LdapConnectionConfig

No need to throw LdapException when calling newLdapConnection (that just calls LdapConnectionFactory.newUnboundLdapConnection)

There is such a pool of bound connection in a weird place : LdapApiIntegrationUtils.getPooledAdminConnection()

We should move this method into some more convenient place.

New interface and class were added for connection factory, pooled connection factory was modified in a backwards compatible way to use connection factory instead of config, and connection pool was modified to add a method for getting an unbound connection. The only functional change to previous code was in the implementation of the pooled connection factory in which I modified the passivate (called when a connection is returned) to ensure that it gets re-initialized (in case the borrower bound the connection to a different account, changed the timeout, or the binary attribute detector was changed).

Thanks for the committed code, Lucs.

However, could you add the mandatory AL 2.0 header at the top of the
file, and a header before the created classes, containing a short
explaination and an @Autor tag ( * @author <a
href="mailto:dev@directory.apache.org">Apache Directory Project</a>
)

Many thanks !

Whoops, my bad. Fixed in revision 1590323.