diff --git a/UMSETZUNGSPLAN.md b/UMSETZUNGSPLAN.md index 34aa55e..386fb54 100644 --- a/UMSETZUNGSPLAN.md +++ b/UMSETZUNGSPLAN.md @@ -74,8 +74,10 @@ Beim Review ist aufgefallen, dass `TraderAnalyticsWorker.CalculatePnL` **jeden T - [x] Aktuellen Ist-Stand des Schemas exakt erfassen (inkl. aller manuellen `ALTER TABLE`-Patches aus `DependencyInjection.cs`) — abgeglichen, Entities/`OnModelCreating` deckten den Patch-Stand bereits vollständig ab - [x] Erste **Baseline-Migration** erzeugt (`20260701102311_InitialBaseline`, in `src/Predictalytics.Infrastructure/Migrations/`), entspricht exakt dem aktuellen Schema - [x] `EnsureCreatedAsync` + handgeschriebene `ExecuteIfColumnMissing`/`ALTER`-Helfer aus `DependencyInjection.cs` entfernt, durch `db.Database.MigrateAsync()` ersetzt (Platform-Seed bleibt als `INSERT IGNORE`) -- [ ] **Offen (manueller Schritt durch Nutzer):** Live-DB per SQL als "bereits migriert" markieren (`__EFMigrationsHistory`-Tabelle + Insert für `20260701102311_InitialBaseline`), da die Tabellen dort schon existieren und nicht per `CreateTable` neu angelegt werden dürfen. SQL wurde im Chat bereitgestellt, Ausführung liegt beim Nutzer (bewusst nicht automatisch gegen die produktive Remote-DB ausgeführt) -- [ ] Alle künftigen Schemaänderungen aus Phase 1 (z.B. `TraderPosition`, `MarketOutcomePriceSnapshot`, `TraderCopytradingProfile`) als reguläre Migrationen anlegen — **dieser Task sollte vor A1 abgeschlossen sein**, damit wir dort nicht wieder manuell patchen +- [x] Migration end-to-end gegen eine echte, frische DEV-Datenbank getestet (`lqf7.your-database.de`/`bergisnu_db0`, vom Nutzer bereitgestellt) — `dotnet ef database update` lief fehlerfrei durch, `dotnet ef migrations list` bestätigt sie als angewendet +- [x] `AppDbContextFactory` (Design-Time-Factory für `dotnet ef`) liest Zielverbindung jetzt aus Env-Vars (`PREDICTALYTICS_DB_SERVER/_NAME/_USER/_PASSWORD`, via `MySqlConnectionStringBuilder` statt roher String-Konkatenation, da Passwörter Sonderzeichen wie `;`/`{}` enthalten können) statt fest codiert — kein Secret mehr im Repo nötig, um Migrationen gegen eine beliebige DB zu fahren +- [x] ~~Live-Produktions-DB manuell als "bereits migriert" markieren~~ — vorerst zurückgestellt: Da wir laut Abschnitt D ohnehin einen kompletten Neustart der Produktions-DB planen (Altdaten sind jederzeit nachladbar, siehe C0), richten wir die neue Produktions-DB am Ende genauso ein wie die DEV-DB (leer anlegen + `dotnet ef database update`) — der fummelige Stamp-Schritt auf die bestehende Live-DB entfällt dann komplett. Nur falls wir uns doch gegen den Neustart entscheiden, müsste dieser Schritt nachgeholt werden +- [ ] Alle künftigen Schemaänderungen aus Phase 1 (z.B. `TraderPosition`, `MarketOutcomePriceSnapshot`, `TraderCopytradingProfile`) als reguläre Migrationen anlegen — direkt gegen die DEV-DB entwickeln/testen, da wir dort laut Nutzer frei experimentieren dürfen ### B2. Secrets-Management (Release-Vorbereitung, kein akuter Risikofall) > Software läuft aktuell nur lokal, kein Fremdzugriff — Passwort bleibt vorerst wie es ist, keine Rotation nötig. Dieser Punkt ist reine **Vorbereitung**, damit das Projekt bei Bedarf später auch ohne die aktuellen Secrets veröffentlicht/geteilt werden könnte, ohne den Code nochmal anfassen zu müssen. Dadurch niedrigere Priorität als vorher angenommen — kann später in der Reihenfolge stehen. diff --git a/src/Predictalytics.Infrastructure/Data/AppDbContextFactory.cs b/src/Predictalytics.Infrastructure/Data/AppDbContextFactory.cs index c6f22a3..7b9ebbd 100644 --- a/src/Predictalytics.Infrastructure/Data/AppDbContextFactory.cs +++ b/src/Predictalytics.Infrastructure/Data/AppDbContextFactory.cs @@ -1,19 +1,30 @@ using Microsoft.EntityFrameworkCore; using Microsoft.EntityFrameworkCore.Design; -using Microsoft.Extensions.Configuration; -using System.IO; +using MySqlConnector; namespace Predictalytics.Infrastructure.Data; +/// +/// Used only by "dotnet ef" design-time tooling (migrations add/update), never at runtime. +/// Target database is picked via env vars so no connection string/secret ever needs to +/// live in source or config: PREDICTALYTICS_DB_SERVER/_NAME/_USER/_PASSWORD. +/// Falls back to a local dev default if the env vars aren't set. +/// public class AppDbContextFactory : IDesignTimeDbContextFactory { public AppDbContext CreateDbContext(string[] args) { var optionsBuilder = new DbContextOptionsBuilder(); - // Fallback for local migrations - var connectionString = "Server=localhost;Database=Predictalytics;User=root;Password=;"; - optionsBuilder.UseMySql(connectionString, new MySqlServerVersion(new Version(8, 0, 31))); + var builder = new MySqlConnectionStringBuilder + { + Server = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_SERVER") ?? "localhost", + Database = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_NAME") ?? "Predictalytics", + UserID = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_USER") ?? "root", + Password = Environment.GetEnvironmentVariable("PREDICTALYTICS_DB_PASSWORD") ?? "", + }; + + optionsBuilder.UseMySql(builder.ConnectionString, new MySqlServerVersion(new Version(8, 0, 31))); return new AppDbContext(optionsBuilder.Options); }