Uploaded image for project: 'OpenAM'
  1. OpenAM
  2. OPENAM-8459

LDAPAuthUtils should set operations timeout in seconds

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 13.0.0, 13.5.0, 14.0.0
    • Fix Version/s: 13.5.1, 14.0.0
    • Component/s: console
    • Labels:
    • Sprint:
      AM Sustaining Sprint 31
    • Story Points:
      0.5
    • Support Ticket IDs:

      Description

      Console info text for the service property 'LDAP operation timeout' states

      Defines the timeout in seconds OpenAM should wait for a response of the Directory Server - 0 means no timeout.
      

      but code sets timeout in milliseconds

      LDAPAuthUtils.java
          private ConnectionFactory createConnectionPool(Map<String, ConnectionFactory> connectionPools,
                  String bindingUser, char[] bindingPwd) throws LdapException, LDAPUtilException {
              ConnectionFactory connPool;
      
              try {
                  String configName = servers.toString() + ":" + bindingUser;
                  connPool = connectionPools.get(configName);
                  if (connPool == null) {
                      synchronized (connectionPools) {
                          connPool = connectionPools.get(configName);
                          Options options = Options.defaultOptions()
                                  .set(REQUEST_TIMEOUT, new Duration((long) operationsTimeout, TimeUnit.MILLISECONDS));
      ...
      

      The operation timeout should be set in seconds instead, just like before 13.0.0.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                sachiko Sachiko Wallace
                Reporter:
                bthalmayr Bernhard Thalmayr
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: