-
Type: Improvement
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
Labels:None
-
Fully Compatible
-
Security 2021-09-06, Security 2021-09-20
Currently, connections used in the `LDAPConnectionFactory` only are marked as `success` or `failure` when they are done being used. This helps ensure that good connections are kept in the pool while those that result in errors are thrown out and not used. However, connections should also be marked as used via `indicateUsed()` immediately before each outbound network call to update when the connection was last used. This will improve performance by reducing the frequency of connection refreshes now that usage will be accurately tracked.
Something to note is that bind attempts sometimes require binding multiple times on the same connection if several passwords are provided for the bindDN. This may require rehashing the current manner in which connections are marked as failed for binds in order to be able to call `indicateUsed()` without error.