diff --git a/src/InboxIntel.Infrastructure/Persistence/AppDbContext.cs b/src/InboxIntel.Infrastructure/Persistence/AppDbContext.cs index e4d32ec..47d390a 100644 --- a/src/InboxIntel.Infrastructure/Persistence/AppDbContext.cs +++ b/src/InboxIntel.Infrastructure/Persistence/AppDbContext.cs @@ -8,7 +8,22 @@ namespace InboxIntel.Infrastructure.Persistence; public class AppDbContext : DbContext, IAppDbContext { - public AppDbContext(DbContextOptions options) : base(options) { } + private readonly ICurrentUser? _currentUser; + + public AppDbContext(DbContextOptions options, ICurrentUser? currentUser = null) : base(options) + => _currentUser = currentUser; + + /// + /// Tenant id used by the global query filters below. Resolves to the + /// authenticated user during an HTTP request. It is + /// when there is no current user (background workers, design-time tooling, + /// startup migration) — in which case filtering is DISABLED, because those + /// paths are trusted server code that already scope their own queries by a + /// userId passed in explicitly. The security value is on the HTTP attack + /// surface: a forgotten manual WHERE UserId == can no longer leak + /// another tenant's rows, since the filter restricts to the caller. + /// + public Guid CurrentUserId => _currentUser?.UserId ?? Guid.Empty; public DbSet Users => Set(); public DbSet Emails => Set(); @@ -26,6 +41,41 @@ public class AppDbContext : DbContext, IAppDbContext protected override void OnModelCreating(ModelBuilder modelBuilder) { modelBuilder.ApplyConfigurationsFromAssembly(Assembly.GetExecutingAssembly()); + + // Defense-in-depth tenant isolation (systemic IDOR safeguard). Every + // user-owned entity is filtered to the current user so a query that forgets + // its manual `WHERE UserId ==` clause cannot leak across tenants. Applied + // uniformly to all user-scoped entities so EF sees no filtered/unfiltered + // navigation mismatch. Bypassed when CurrentUserId is Guid.Empty (workers). + modelBuilder.Entity().HasQueryFilter(e => CurrentUserId == Guid.Empty || e.UserId == CurrentUserId); + modelBuilder.Entity().HasQueryFilter(e => CurrentUserId == Guid.Empty || e.UserId == CurrentUserId); + modelBuilder.Entity().HasQueryFilter(e => CurrentUserId == Guid.Empty || e.UserId == CurrentUserId); + modelBuilder.Entity().HasQueryFilter(e => CurrentUserId == Guid.Empty || e.UserId == CurrentUserId); + modelBuilder.Entity().HasQueryFilter(e => CurrentUserId == Guid.Empty || e.UserId == CurrentUserId); + modelBuilder.Entity