Changeset 0496e6e for iknow-api

Timestamp:

09/23/26 13:29:17 (11 hours ago)

Author:

Stefan-Saveski <stefansaveski19@…>

Branches:

master

Children:

2853f2e

Parents:

e7bafc0 (diff), 002cf5f (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge remote-tracking branch 'origin/phase8-transactions-pooling'

Location:

iknow-api

Files:

• Program.cs (modified) (1 diff)
• Services/Implementations/EnrollmentService.cs (modified) (3 diffs)
• appsettings.Development.json.example (modified) (1 diff)

iknow-api/Program.cs

@@ -66 +66 @@
 // Postgres type by name; PgEnumLabels covers the members whose label is not
 // simply the lowercased member name.
-builder.Services.AddDbContext<AppDbContext>(options =>
+// AddDbContextPool reuses the DbContext instances themselves; the connections
+// underneath them are pooled separately by Npgsql, sized in the connection
+// string. Both matter here because every physical connection is one more
+// channel through the SSH tunnel.
+builder.Services.AddDbContextPool<AppDbContext>(options =>
     options.UseNpgsql(
         builder.Configuration.GetConnectionString("DefaultConnection"),

iknow-api/Services/Implementations/EnrollmentService.cs

@@ -3 +3 @@
 using iknow_api.Models;
 using Microsoft.EntityFrameworkCore;
+using Npgsql;
 
 namespace iknow_api.Services
@@ -220 +221 @@
                 };
             }
+            catch (DbUpdateException ex) when (IsUniqueViolation(ex))
+            {
+                // Two enrolments for the same semester sent at the same time both
+                // pass the check above, because each transaction reads the state
+                // from before the other one wrote. UNIQUE (user_id, semester_id)
+                // is what actually settles it, so the loser gets the same
+                // sentence as if the check had caught it.
+                await transaction.RollbackAsync();
+                return Fail("You are already enrolled in that semester.");
+            }
             catch
             {
@@ -226 +237 @@
             }
         }
+
+        /// <summary>23505 is unique_violation.</summary>
+        private static bool IsUniqueViolation(DbUpdateException ex) =>
+            ex.InnerException is PostgresException { SqlState: "23505" };
 
         private static EnrollSemesterResultDto Fail(string message) =>

iknow-api/appsettings.Development.json.example

@@ -7 +7 @@
   },
   "ConnectionStrings": {
-    "DefaultConnection": "Host=localhost;Port=9999;Database=db_202526z_va_prj_know2026;Username=db_202526z_va_prj_iknow2026_owner;Password=YOUR_DB_PASSWORD_HERE;Search Path=project;Include Error Detail=true"
+    "DefaultConnection": "Host=localhost;Port=9999;Database=db_202526z_va_prj_know2026;Username=db_202526z_va_prj_iknow2026_owner;Password=YOUR_DB_PASSWORD_HERE;Search Path=project;Include Error Detail=true;Maximum Pool Size=10;Minimum Pool Size=1;Connection Idle Lifetime=300;Timeout=15"
   },
   "Jwt": {