Update minConns for PasswdUserDBAuthentication

The PasswdUserDBAuthentication has been updated such that it creates a DS connection only if needed. The LdapAnonConnFactory has been updated to allow no minimum connections.
dogtagpki · Jul 18, 2023 · 372fd6e · 372fd6e
1 parent 5da3c59
commit 372fd6e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.../server/src/main/java/com/netscape/cmscore/authentication/PasswdUserDBAuthentication.java b/.../server/src/main/java/com/netscape/cmscore/authentication/PasswdUserDBAuthentication.java
@@ -101,7 +101,7 @@ public void init(
         DBSubsystem dbSubsystem = engine.getDBSubsystem();
         LdapConnInfo ldapinfo = dbSubsystem.getLdapConnInfo();
 
-        mAnonConnFactory = new LdapAnonConnFactory("PasswdUserDBAuthentication", 3, 20, ldapinfo);
+        mAnonConnFactory = new LdapAnonConnFactory("PasswdUserDBAuthentication", 0, 20, ldapinfo);
         mAnonConnFactory.setCMSEngine(engine);
         mAnonConnFactory.init(socketConfig);
     }

diff --git a/base/server/src/main/java/com/netscape/cmscore/ldapconn/LdapAnonConnFactory.java b/base/server/src/main/java/com/netscape/cmscore/ldapconn/LdapAnonConnFactory.java
@@ -177,7 +177,7 @@ protected void init() throws ELdapException {
         if (mInited)
             return; // XXX should throw exception here ?
 
-        if (mMinConns <= 0)
+        if (mMinConns < 0)
             throw new ELdapException("Invalid minimum number of connections: " + mMinConns);
 
         if (mMaxConns <= 0)