From 9a8607b07ee7cb0a1dfc24cdbcabca45e9e3967a Mon Sep 17 00:00:00 2001 From: David Leek Date: Wed, 5 Feb 2025 15:45:51 +0100 Subject: [PATCH] chore: clear scim fields when deleting user + migration for existing cases (#9217) --- src/lib/db/user-store.ts | 2 ++ ...50205072305-clean-scim-id-from-deleted-users.js | 14 ++++++++++++++ 2 files changed, 16 insertions(+) create mode 100644 src/migrations/20250205072305-clean-scim-id-from-deleted-users.js diff --git a/src/lib/db/user-store.ts b/src/lib/db/user-store.ts index 04d2ab73a6..4edaaacb07 100644 --- a/src/lib/db/user-store.ts +++ b/src/lib/db/user-store.ts @@ -197,6 +197,8 @@ class UserStore implements IUserStore { deleted_at: new Date(), email: null, username: null, + scim_id: null, + scim_external_id: null, name: this.db.raw('name || ?', '(Deleted)'), }); } diff --git a/src/migrations/20250205072305-clean-scim-id-from-deleted-users.js b/src/migrations/20250205072305-clean-scim-id-from-deleted-users.js new file mode 100644 index 0000000000..144ed05be1 --- /dev/null +++ b/src/migrations/20250205072305-clean-scim-id-from-deleted-users.js @@ -0,0 +1,14 @@ +exports.up = (db, cb) => { + db.runSql(` + UPDATE users + SET + scim_id = NULL, + scim_external_id = NULL + WHERE deleted_at IS NOT NULL AND scim_id IS NOT NULL; + `, cb); + + }; + + exports.down = (db, cb) => { + cb(); + };