From 7d04da2eae239b9c8ecb2781e65b7c3a02dad992 Mon Sep 17 00:00:00 2001
From: solidDoWant <fred.heinecke@yahoo.com>
Date: Fri, 2 May 2025 23:04:13 +0000
Subject: [PATCH] Backend: Fix pooling configuration always overriding
 connection string parameters

---
 back/src/Kyoo.Postgresql/PostgresModule.cs | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/back/src/Kyoo.Postgresql/PostgresModule.cs b/back/src/Kyoo.Postgresql/PostgresModule.cs
index 2558f36b..f7ad8ff2 100644
--- a/back/src/Kyoo.Postgresql/PostgresModule.cs
+++ b/back/src/Kyoo.Postgresql/PostgresModule.cs
@@ -33,15 +33,19 @@ public static class PostgresModule
 {
 	public static NpgsqlDataSource CreateDataSource(IConfiguration configuration)
 	{
+		var connectionString = configuration.GetValue<string>("POSTGRES_URL");
+
 		// Load the connection string from the environment variable, as well as standard libpq environment variables
 		// (PGUSER, PGPASSWORD, PGHOST, PGPORT, PGDATABASE, etc.)
-		NpgsqlConnectionStringBuilder conBuilder =
-			new(configuration.GetValue<string>("POSTGRES_URL") ?? "")
-			{
-				Pooling = true,
-				MaxPoolSize = 95,
-				Timeout = 30
-			};
+		NpgsqlConnectionStringBuilder conBuilder = new(connectionString ?? "");
+		// Set defaults when no explicit connection string is provided. This cannot be set if the connection string
+		// is provided, or it will override connection string values.
+		if (string.IsNullOrEmpty(connectionString))
+		{
+			conBuilder.Pooling = true;
+			conBuilder.MaxPoolSize = 95;
+			conBuilder.Timeout = 30;
+		}
 
 		string? oldVarUsername = configuration.GetValue<string>("POSTGRES_USER");
 		if (!string.IsNullOrEmpty(oldVarUsername))