chahine/rebreak-monorepo
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix(mail): DSGVO Art. 17 — manuelles Sample-Cleanup bei Account-Delete

Browse Source 
MailClassificationSample hat keine userId-FK-Cascade im Schema (connectionId
ist nullable). Samples ohne connectionId blieben nach deleteAllMailConnections()
als Orphans stehen. Neuer Helper deleteUserMailClassificationSamples() löscht
explizit nach userId — wird in delete.delete.ts parallel zu anderen Lösch-Ops
ausgeführt.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
chahinebrini 2026-05-14 08:29:32 +02:00
parent bdd93668ae
commit 343f9ab567
2 changed files with 30 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
backend/server/api/user/delete.delete.ts
View File

@ -9,7 +9,10 @@ import {
deleteUserCoachSessions,
} from "../../db/user";
import { deleteProfile } from "../../db/profile";
import { deleteAllMailConnections } from "../../db/mail";
import {
deleteAllMailConnections,
deleteUserMailClassificationSamples,
} from "../../db/mail";
import { writeConsentRevoke } from "../../db/consent";
import { usePrisma } from "../../utils/prisma";
@ -44,6 +47,9 @@ export default defineEventHandler(async (event) => {
}
// Delete all user data (DSGVO Art. 17)
// Reihenfolge: Samples VOR Connections löschen (oder parallel — FK-Reihenfolge
// egal weil wir nach userId filtern). Samples haben keine userId-FK-Cascade
// im Schema (connectionId ist nullable), daher manuelles Cleanup zwingend.
await Promise.all([
deleteUserUrgeLogs(userId),
deleteUserSosSessions(userId),
@ -52,6 +58,7 @@ export default defineEventHandler(async (event) => {
deleteAllUserCustomDomains(userId),
deleteUserTrustedContacts(userId),
deleteUserCoachSessions(userId),
deleteUserMailClassificationSamples(userId),
deleteAllMailConnections(userId),
]);

24
backend/server/db/mail.ts
View File

@ -193,13 +193,33 @@ export async function insertMailBlocked(
// ─── MailClassificationSample ─────────────────────────────────────────────────
/**
* Löscht alle MailClassificationSamples eines Users.
*
* Warum manuell und nicht via Prisma-Cascade:
* MailClassificationSample hat KEINE userId-Relation mit onDelete: Cascade im Schema
* (connectionId hat Cascade, aber connectionId ist nullable). Samples mit
* connectionId=null wären nach deleteAllMailConnections() Orphans.
*
* Muss in delete.delete.ts VOR deleteAllMailConnections() aufgerufen werden
* (oder in Promise.all parallel) — FK-Reihenfolge spielt keine Rolle weil wir
* nach userId filtern, nicht nach connectionId.
*
* DSGVO Art. 17: User-Daten müssen vollständig gelöscht werden.
*/
export async function deleteUserMailClassificationSamples(userId: string) {
const db = usePrisma();
return db.mailClassificationSample.deleteMany({ where: { userId } });
}
/**
* Schreibt einen Klassifikations-Sample-Eintrag für ML-Phase 3.
* Wird nach JEDER Klassifikation aufgerufen (außer Layer 0 / Already-blocked Skips).
*
* DSGVO: Nur Features, keine Mail-Inhalte (kein Body). Subject + Sender sind
* kurzlebige Detection-Signale, kein narrativer Inhalt. Cascade-Delete bei
* User-Löschung (Art. 17).
* kurzlebige Detection-Signale, kein narrativer Inhalt.
* Vollständige Löschung bei Account-Delete via deleteUserMailClassificationSamples()
* (Art. 17) — NICHT via Prisma-Cascade, da userId keine FK-Relation hat.
*/
export async function insertMailClassificationSample(entry: {
userId: string;