fix: cicd

config.yaml

@@ -72,6 +72,6 @@ postgresql_config_version: 0
 # to be closed.
 # postgresql_idle_connection_timeout:
 
-# (DIFFERENT_ACROSS_TENANTS | OPTIONAL | Default: 1) integer value. Minimum number of idle connections to be kept
-# active.
+# (DIFFERENT_ACROSS_TENANTS | OPTIONAL | Default: null) integer value. Minimum number of idle connections to be kept
+# active. If not set, minimum idle connections will be same as the connection pool size.
 # postgresql_minimum_idle_connections:

devConfig.yaml

@@ -74,6 +74,6 @@ postgresql_password: "root"
 # to be closed.
 # postgresql_idle_connection_timeout:
 
-# (DIFFERENT_ACROSS_TENANTS | OPTIONAL | Default: 1) integer value. Minimum number of idle connections to be kept
-# active.
+# (DIFFERENT_ACROSS_TENANTS | OPTIONAL | Default: null) integer value. Minimum number of idle connections to be kept
+# active. If not set, minimum idle connections will be same as the connection pool size.
 # postgresql_minimum_idle_connections:

src/main/java/io/supertokens/storage/postgresql/ConnectionPool.java

@@ -82,7 +82,9 @@ private synchronized void initialiseHikariDataSource() throws SQLException {
         config.setMaximumPoolSize(userConfig.getConnectionPoolSize());
         config.setConnectionTimeout(5000);
         config.setIdleTimeout(userConfig.getIdleConnectionTimeout());
-        config.setMinimumIdle(userConfig.getMinimumIdleConnections());
+        if (userConfig.getMinimumIdleConnections() != null) {
+            config.setMinimumIdle(userConfig.getMinimumIdleConnections());
+        }
         config.addDataSourceProperty("cachePrepStmts", "true");
         config.addDataSourceProperty("prepStmtCacheSize", "250");
         config.addDataSourceProperty("prepStmtCacheSqlLimit", "2048");

src/main/java/io/supertokens/storage/postgresql/config/PostgreSQLConfig.java

@@ -118,7 +118,7 @@ public class PostgreSQLConfig {
 
     @JsonProperty
     @ConnectionPoolProperty
-    private int postgresql_minimum_idle_connections = 1;
+    private Integer postgresql_minimum_idle_connections = null;
 
     @IgnoreForAnnotationCheck
     boolean isValidAndNormalised = false;
@@ -246,7 +246,7 @@ public long getIdleConnectionTimeout() {
         return postgresql_idle_connection_timeout;
     }
 
-    public int getMinimumIdleConnections() {
+    public Integer getMinimumIdleConnections() {
         return postgresql_minimum_idle_connections;
     }
 
@@ -356,15 +356,17 @@ public void validateAndNormalise() throws InvalidConfigException {
                     "'postgresql_connection_pool_size' in the config.yaml file must be > 0");
         }
 
-        if (postgresql_minimum_idle_connections <= 0) {
-            throw new InvalidConfigException(
-                    "'postgresql_minimum_idle_connections' must be a positive value");
-        }
+        if (postgresql_minimum_idle_connections != null) {
+            if (postgresql_minimum_idle_connections < 0) {
+                throw new InvalidConfigException(
+                        "'postgresql_minimum_idle_connections' must be >= 0");
+            }
 
-        if (postgresql_minimum_idle_connections > postgresql_connection_pool_size) {
-            throw new InvalidConfigException(
-                    "'postgresql_minimum_idle_connections' must be less than or equal to "
-                            + "'postgresql_connection_pool_size'");
+            if (postgresql_minimum_idle_connections > postgresql_connection_pool_size) {
+                throw new InvalidConfigException(
+                        "'postgresql_minimum_idle_connections' must be less than or equal to "
+                                + "'postgresql_connection_pool_size'");
+            }
         }
 
         // Normalisation