Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-59735

Mark connections as used for pooled LDAP connections

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.1.0-rc0
    • Component/s: None
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Sprint:
      Security 2021-09-06, Security 2021-09-20

      Description

      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.

        Attachments

          Activity

            People

            Assignee:
            varun.ravichandran Varun Ravichandran
            Reporter:
            varun.ravichandran Varun Ravichandran
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: