Merge pull request 'fixed' (#13) from testbranch into main

Reviewed-on: #13

docker-compose.yml

@@ -28,14 +28,9 @@ services:
       - DB_PROVIDER=PostgreSQL
       - ConnectionStrings__DefaultConnection=Host=db;Database=timetracker;Username=timetracker_user;Password=SecretPassword123;
       - EnableHttpsRedirect=false
-      - TenantSettings__BaseDomain=timetracker.marc-wieland.de
-    volumes:
-      - dpkeys:/home/app/.aspnet/DataProtection-Keys
     depends_on:
       - db
 
 volumes:
   pgdata:
-    name: timetracker_pgdata
-  dpkeys:
-    name: timetracker_dpkeys
+    name: timetracker_pgdata

timetracker.Server/Data/Migrations/20260624210436_AddTenantBranding.cs

@@ -1,4 +1,4 @@
-using Microsoft.EntityFrameworkCore.Migrations;
+using Microsoft.EntityFrameworkCore.Migrations;
 
 #nullable disable
 
@@ -13,6 +13,7 @@ namespace timetracker.Data.Migrations
             migrationBuilder.AddColumn<bool>(
                 name: "IsTenantAdmin",
                 table: "Users",
+                type: "INTEGER",
                 nullable: false,
                 defaultValue: false);
 

timetracker.Server/Data/TenantProvider.cs

@@ -37,7 +37,7 @@ public class TenantProvider : ITenantProvider
             }
 
             // Resolve and cache in HttpContext.Items
-            ResolveTenant();
+            ResolveTenantAsync().GetAwaiter().GetResult();
 
             if (httpContext.Items.TryGetValue("TenantId", out cachedId))
             {
@@ -48,27 +48,27 @@ public class TenantProvider : ITenantProvider
         }
     }
 
-    public Task<Tenant?> GetCurrentTenantAsync()
+    public async Task<Tenant?> GetCurrentTenantAsync()
     {
         var httpContext = _httpContextAccessor.HttpContext;
-        if (httpContext == null) return Task.FromResult<Tenant?>(null);
+        if (httpContext == null) return null;
 
         if (httpContext.Items.TryGetValue("CurrentTenant", out var cachedTenant))
         {
-            return Task.FromResult((Tenant?)cachedTenant);
+            return (Tenant?)cachedTenant;
         }
 
-        ResolveTenant();
+        await ResolveTenantAsync();
 
         if (httpContext.Items.TryGetValue("CurrentTenant", out cachedTenant))
         {
-            return Task.FromResult((Tenant?)cachedTenant);
+            return (Tenant?)cachedTenant;
         }
 
-        return Task.FromResult<Tenant?>(null);
+        return null;
     }
 
-    private void ResolveTenant()
+    private async Task ResolveTenantAsync()
     {
         var httpContext = _httpContextAccessor.HttpContext;
         if (httpContext == null) return;
@@ -82,9 +82,9 @@ public class TenantProvider : ITenantProvider
 
         if (!string.IsNullOrEmpty(subdomain))
         {
-            using var db = _dbContextFactory.CreateDbContext();
-            var tenant = db.Tenants
-                .FirstOrDefault(t => t.Subdomain.ToLower() == subdomain.ToLower());
+            await using var db = await _dbContextFactory.CreateDbContextAsync();
+            var tenant = await db.Tenants
+                .FirstOrDefaultAsync(t => t.Subdomain.ToLower() == subdomain.ToLower());
 
             if (tenant != null)
             {

timetracker.Server/Program.cs

@@ -195,88 +195,6 @@ using (var scope = app.Services.CreateScope())
     }
 
     await db.Database.MigrateAsync();
-
-    if (dbProvider.Equals("PostgreSQL", StringComparison.OrdinalIgnoreCase))
-    {
-        try
-        {
-            var conn = db.Database.GetDbConnection();
-            if (conn.State != System.Data.ConnectionState.Open)
-            {
-                await conn.OpenAsync();
-            }
-
-            // Check if column IsTenantAdmin in Users table is of type integer
-            using (var cmdCheckType = conn.CreateCommand())
-            {
-                cmdCheckType.CommandText = @"
-                    SELECT data_type 
-                    FROM information_schema.columns 
-                    WHERE table_name = 'Users' AND column_name = 'IsTenantAdmin';";
-                var dataType = await cmdCheckType.ExecuteScalarAsync() as string;
-                if (dataType != null && dataType.Equals("integer", StringComparison.OrdinalIgnoreCase))
-                {
-                    // Alter the column type to boolean
-                    using (var cmdAlterType = conn.CreateCommand())
-                    {
-                        cmdAlterType.CommandText = @"
-                            ALTER TABLE ""Users"" 
-                            ALTER COLUMN ""IsTenantAdmin"" TYPE boolean 
-                            USING (""IsTenantAdmin"" <> 0);";
-                        await cmdAlterType.ExecuteNonQueryAsync();
-                    }
-                }
-            }
-
-            // Check if column IsApproved in Tenants table is of type integer
-            using (var cmdCheckApproved = conn.CreateCommand())
-            {
-                cmdCheckApproved.CommandText = @"
-                    SELECT data_type 
-                    FROM information_schema.columns 
-                    WHERE table_name = 'Tenants' AND column_name = 'IsApproved';";
-                var dataType = await cmdCheckApproved.ExecuteScalarAsync() as string;
-                if (dataType != null && dataType.Equals("integer", StringComparison.OrdinalIgnoreCase))
-                {
-                    // Alter the column type to boolean
-                    using (var cmdAlterApproved = conn.CreateCommand())
-                    {
-                        cmdAlterApproved.CommandText = @"
-                            ALTER TABLE ""Tenants"" 
-                            ALTER COLUMN ""IsApproved"" TYPE boolean 
-                            USING (""IsApproved"" <> 0);";
-                        await cmdAlterApproved.ExecuteNonQueryAsync();
-                    }
-                }
-            }
-
-            // Check if column CreatedAt in Tenants table is of type text or character varying
-            using (var cmdCheckCreatedAt = conn.CreateCommand())
-            {
-                cmdCheckCreatedAt.CommandText = @"
-                    SELECT data_type 
-                    FROM information_schema.columns 
-                    WHERE table_name = 'Tenants' AND column_name = 'CreatedAt';";
-                var dataType = await cmdCheckCreatedAt.ExecuteScalarAsync() as string;
-                if (dataType != null && (dataType.Equals("text", StringComparison.OrdinalIgnoreCase) || dataType.Equals("character varying", StringComparison.OrdinalIgnoreCase)))
-                {
-                    // Alter the column type to timestamp without time zone
-                    using (var cmdAlterCreatedAt = conn.CreateCommand())
-                    {
-                        cmdAlterCreatedAt.CommandText = @"
-                            ALTER TABLE ""Tenants"" 
-                            ALTER COLUMN ""CreatedAt"" TYPE timestamp without time zone 
-                            USING (""CreatedAt""::timestamp without time zone);";
-                        await cmdAlterCreatedAt.ExecuteNonQueryAsync();
-                    }
-                }
-            }
-        }
-        catch (Exception ex)
-        {
-            Console.WriteLine($"Fehler beim Konvertieren der PostgreSQL-Spaltentypen (IsTenantAdmin/IsApproved/CreatedAt): {ex.Message}");
-        }
-    }
 }
 
 var forwardedHeadersOptions = new ForwardedHeadersOptions