Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-44410

Unboundid - add options for setHealthCheckIntervalMillis and setHealthCheck

    XMLWordPrintable

    Details

    • 21 Status:
      Resolved
    • 20 status:
      Please Merge

      Description

      https://groups.google.com/a/apereo.org/g/sakai-dev/c/h7ATfBitN_4/m/XE1hGJ7kAAAJ

      Our ldap server's load balancer has a short connection timeout so we are intermittently seeing some errors in the tomcat logs

      org.sakaiproject.portal.api.PortalHandlerException: java.lang.RuntimeException: authenticateUser(): LDAPException during authentication attempt [userLogin = <userid>][result code = 81 (server down)][error message = LDAPException(resultCode=81 (server down), errorMessage='The connection to server ldap.server.edu:port was closed while waiting for a response to a bind request SimpleBindRequest(dn='<myDN>'):  An I/O error occurred while trying to read the response from the server:  SocketException(Connection reset), ldapSDKVersion=4.0.12,
      

      However, we also enabled retryFailedOperationsDueToInvalidConnections, so I think users are still able to log in. Although intermittent, the error happens often. I think we can fix this by implementing unboundid options

      setHealthCheckIntervalMillis(long healthCheckInterval)
      setHealthCheck(@NotNull LDAPConnectionPoolHealthCheck healthCheck)

      https://docs.ldap.com/ldap-sdk/docs/javadoc/com/unboundid/ldap/sdk/LDAPConnectionPool.html#setHealthCheckIntervalMillis(long)

      https://docs.ldap.com/ldap-sdk/docs/javadoc/com/unboundid/ldap/sdk/LDAPConnectionPool.html#setHealthCheck(com.unboundid.ldap.sdk.LDAPConnectionPoolHealthCheck)

      https://docs.ldap.com/ldap-sdk/docs/javadoc/com/unboundid/ldap/sdk/GetEntryLDAPConnectionPoolHealthCheck.html

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  ottenhoff Sam Ottenhoff
                  Reporter:
                  austinUH Austin
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration