Fixes unexpected max acquire-timeout (#5377)

Important for applications where high pool contention is expected, and completion is favored over latency.

Co-authored-by: Matt <matt.goodson.business@gmail.com>

lib/client.js

@@ -213,11 +213,16 @@ class Client extends EventEmitter {
       }
     });
 
+    const DEFAULT_ACQUIRE_TIMEOUT = 60000;
     const timeouts = [
-      this.config.acquireConnectionTimeout || 60000,
-      poolConfig.acquireTimeoutMillis,
+      this.config.acquireConnectionTimeout,
+      poolConfig.acquireTimeoutMillis
     ].filter((timeout) => timeout !== undefined);
 
+    if (!timeouts.length) {
+      timeouts.push(DEFAULT_ACQUIRE_TIMEOUT)
+    }
+
     // acquire connection timeout can be set on config or config.pool
     // choose the smallest, positive timeout setting and set on poolConfig
     poolConfig.acquireTimeoutMillis = Math.min(...timeouts);