Merge pull request #694 from PathfinderHonorManager/develop

Add logging to diagnose migration connection string issue

Author: brian-cummings

PathfinderHonorManager/Service/MigrationService.cs

@@ -31,6 +31,13 @@ public async Task StartAsync(CancellationToken cancellationToken)
             using var scope = _serviceProvider.CreateScope();
 
             var migrationConnectionString = _configuration.GetConnectionString("PathfinderMigrationCS");
+            
+            if (string.IsNullOrEmpty(migrationConnectionString))
+            {
+                _logger.LogError("PathfinderMigrationCS connection string is not configured. Cannot apply migrations.");
+                throw new InvalidOperationException("Migration connection string not configured");
+            }
+            
             var optionsBuilder = new DbContextOptionsBuilder<PathfinderContext>();
             optionsBuilder.UseNpgsql(migrationConnectionString, 
                 npgsqlOptions => 
@@ -47,8 +54,8 @@ public async Task StartAsync(CancellationToken cancellationToken)
             try
             {
                 var builder = new Npgsql.NpgsqlConnectionStringBuilder(migrationConnectionString);
-                _logger.LogDebug("Connecting to database for migrations: Host={Host}, Database={Database}, Username={Username}", 
-                    builder.Host, builder.Database, builder.Username);
+                _logger.LogWarning("Using migration connection: Username={Username} (Password length: {Length})", 
+                    builder.Username, builder.Password?.Length ?? 0);
 
                 var canConnect = await context.Database.CanConnectAsync(cancellationToken);
                 if (!canConnect)
@@ -57,8 +64,6 @@ public async Task StartAsync(CancellationToken cancellationToken)
                     throw new InvalidOperationException("Database connection failed");
                 }
 
-                _logger.LogInformation("Database connection verified. Checking migration status...");
-
                 // Check if we have an existing database without migration history (baseline scenario)
                 if (await NeedsBaselineAsync(context, cancellationToken))
                 {