@stamhoofd/backend 2.57.0 → 2.58.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@stamhoofd/backend",
-    "version": "2.57.0",
+    "version": "2.58.0",
     "main": "./dist/index.js",
     "exports": {
         ".": {
@@ -35,16 +35,16 @@
         "@mollie/api-client": "3.7.0",
         "@simonbackx/simple-database": "1.25.0",
         "@simonbackx/simple-encoding": "2.17.0",
-        "@simonbackx/simple-endpoints": "1.14.0",
+        "@simonbackx/simple-endpoints": "1.15.0",
         "@simonbackx/simple-logging": "^1.0.1",
-        "@stamhoofd/backend-i18n": "2.57.0",
-        "@stamhoofd/backend-middleware": "2.57.0",
-        "@stamhoofd/email": "2.57.0",
-        "@stamhoofd/models": "2.57.0",
-        "@stamhoofd/queues": "2.57.0",
-        "@stamhoofd/sql": "2.57.0",
-        "@stamhoofd/structures": "2.57.0",
-        "@stamhoofd/utility": "2.57.0",
+        "@stamhoofd/backend-i18n": "2.58.0",
+        "@stamhoofd/backend-middleware": "2.58.0",
+        "@stamhoofd/email": "2.58.0",
+        "@stamhoofd/models": "2.58.0",
+        "@stamhoofd/queues": "2.58.0",
+        "@stamhoofd/sql": "2.58.0",
+        "@stamhoofd/structures": "2.58.0",
+        "@stamhoofd/utility": "2.58.0",
         "archiver": "^7.0.1",
         "aws-sdk": "^2.885.0",
         "axios": "1.6.8",
@@ -64,5 +64,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "9e6e3c22154937a4977cc0d1b669d9577067b832"
+    "gitHead": "4159f0bc858d301854c4e38a3e315053aec08399"
 }

package/src/endpoints/global/payments/StripeWebhookEndpoint.ts

@@ -7,6 +7,8 @@ import { isDebouncedError, QueueHandler } from '@stamhoofd/queues';
 import { StripeHelper } from '../../../helpers/StripeHelper';
 import { ExchangePaymentEndpoint } from '../../organization/shared/ExchangePaymentEndpoint';
 import { PaymentService } from '../../../services/PaymentService';
+import { AuditLogService } from '../../../services/AuditLogService';
+import { AuditLogType } from '@stamhoofd/structures';
 
 type Params = Record<string, never>;
 class Body extends AutoEncoder {
@@ -85,8 +87,17 @@ export class StripeWebookEndpoint extends Endpoint<Params, Query, Body, Response
                     const id = account.id as string;
                     const [model] = await StripeAccount.where({ accountId: id }, { limit: 1 });
                     if (model) {
+                        const beforeMeta = model.meta.clone();
                         model.setMetaFromStripeAccount(account);
-                        await model.save();
+                        if (await model.save()) {
+                            // Track audit log
+                            await AuditLogService.log({
+                                type: AuditLogType.StripeAccountEdited,
+                                stripeAccount: model,
+                                oldData: beforeMeta,
+                                patch: model.meta,
+                            });
+                        }
                     }
                     else {
                         console.warn('Could not find stripe account with id', id);

package/src/endpoints/global/platform/PatchPlatformEnpoint.ts

@@ -49,10 +49,9 @@ export class PatchPlatformEndpoint extends Endpoint<
         }
 
         const platform = await Platform.getShared();
+        const initialStruct = (await Platform.getSharedPrivateStruct()).clone();
 
         if (request.body.privateConfig) {
-            const oldConfig = platform.privateConfig.clone();
-
             // Did we patch roles?
             if (request.body.privateConfig.roles) {
                 if (!Context.auth.canManagePlatformAdmins()) {
@@ -77,12 +76,6 @@ export class PatchPlatformEndpoint extends Endpoint<
                     request.body.privateConfig.emails,
                 );
             }
-
-            await AuditLogService.log({
-                type: AuditLogType.PlatformSettingsChanged,
-                oldConfig,
-                patch: request.body.privateConfig,
-            });
         }
 
         let shouldUpdateSetupSteps = false;
@@ -116,12 +109,6 @@ export class PatchPlatformEndpoint extends Endpoint<
                 else {
                     platform.config = patchObject(platform.config, newConfig);
                 }
-
-                await AuditLogService.log({
-                    type: AuditLogType.PlatformSettingsChanged,
-                    oldConfig,
-                    patch: newConfig,
-                });
             }
 
             if (newConfig.tags && isPatchableArray(newConfig.tags) && newConfig.tags.changes.length > 0) {
@@ -236,6 +223,12 @@ export class PatchPlatformEndpoint extends Endpoint<
             SetupStepUpdater.updateSetupStepsForAllOrganizationsInCurrentPeriod().catch(console.error);
         }
 
+        await AuditLogService.log({
+            type: AuditLogType.PlatformSettingsChanged,
+            oldData: initialStruct,
+            patch: request.body,
+        });
+
         return new Response(await Platform.getSharedPrivateStruct());
     }
 

package/src/endpoints/global/registration-periods/PatchRegistrationPeriodsEndpoint.ts

@@ -1,11 +1,12 @@
 import { ConvertArrayToPatchableArray, Decoder, PatchableArrayAutoEncoder, PatchableArrayDecoder, StringDecoder, patchObject } from '@simonbackx/simple-encoding';
 import { DecodedRequest, Endpoint, Request, Response } from '@simonbackx/simple-endpoints';
-import { RegistrationPeriod as RegistrationPeriodStruct } from '@stamhoofd/structures';
+import { AuditLogType, RegistrationPeriod as RegistrationPeriodStruct } from '@stamhoofd/structures';
 
 import { SimpleError } from '@simonbackx/simple-errors';
 import { Platform, RegistrationPeriod } from '@stamhoofd/models';
 import { Context } from '../../../helpers/Context';
 import { PeriodHelper } from '../../../helpers/PeriodHelper';
+import { AuditLogService } from '../../../services/AuditLogService';
 
 type Params = Record<string, never>;
 type Query = undefined;
@@ -62,6 +63,11 @@ export class PatchRegistrationPeriodsEndpoint extends Endpoint<Params, Query, Bo
 
             await period.save();
             periods.push(period);
+
+            await AuditLogService.log({
+                type: AuditLogType.RegistrationPeriodAdded,
+                period,
+            });
         }
 
         for (const patch of request.body.getPatches()) {
@@ -74,6 +80,7 @@ export class PatchRegistrationPeriodsEndpoint extends Endpoint<Params, Query, Bo
                     message: 'Registration period not found',
                 });
             }
+            const initialStructure = model.getStructure().clone();
 
             if (patch.startDate !== undefined) {
                 model.startDate = patch.startDate;
@@ -95,6 +102,13 @@ export class PatchRegistrationPeriodsEndpoint extends Endpoint<Params, Query, Bo
 
             // Schedule patch of all groups in this period
             PeriodHelper.updateGroupsInPeriod(model).catch(console.error);
+
+            await AuditLogService.log({
+                type: AuditLogType.RegistrationPeriodEdited,
+                period: model,
+                patch,
+                oldData: initialStructure,
+            });
         }
 
         for (const id of request.body.getDeletes()) {
@@ -109,6 +123,11 @@ export class PatchRegistrationPeriodsEndpoint extends Endpoint<Params, Query, Bo
             }
 
             await model.delete();
+
+            await AuditLogService.log({
+                type: AuditLogType.RegistrationPeriodDeleted,
+                period: model,
+            });
         }
 
         // Clear platform cache

package/src/endpoints/organization/dashboard/organization/PatchOrganizationEndpoint.ts

@@ -1,4 +1,4 @@
-import { AutoEncoderPatchType, Decoder, isPatchableArray, ObjectData, PatchableArrayAutoEncoder, patchObject } from '@simonbackx/simple-encoding';
+import { AutoEncoderPatchType, cloneObject, Decoder, isPatchableArray, ObjectData, PatchableArrayAutoEncoder, patchObject } from '@simonbackx/simple-encoding';
 import { DecodedRequest, Endpoint, Request, Response } from '@simonbackx/simple-endpoints';
 import { SimpleError, SimpleErrors } from '@simonbackx/simple-errors';
 import { Organization, OrganizationRegistrationPeriod, PayconiqPayment, Platform, RegistrationPeriod, SetupStepUpdater, StripeAccount, Webshop } from '@stamhoofd/models';
@@ -63,6 +63,8 @@ export class PatchOrganizationEndpoint extends Endpoint<Params, Query, Body, Res
             });
         }
 
+        const initialStruct = (await AuthenticatedStructures.organization(organization)).clone();
+
         const errors = new SimpleErrors();
         let shouldUpdateSetupSteps = false;
         let updateTags = false;
@@ -136,7 +138,7 @@ export class PatchOrganizationEndpoint extends Endpoint<Params, Query, Body, Res
 
                 // Apply payconiq patch
                 if (request.body.privateMeta.payconiqAccounts !== undefined) {
-                    organization.privateMeta.payconiqAccounts = patchObject(organization.privateMeta.payconiqAccounts, request.body.privateMeta.payconiqAccounts);
+                    organization.privateMeta.payconiqAccounts = patchObject(organization.privateMeta.payconiqAccounts, cloneObject(request.body.privateMeta.payconiqAccounts as any));
 
                     for (const account of organization.privateMeta.payconiqAccounts) {
                         if (account.merchantId === null) {
@@ -214,8 +216,6 @@ export class PatchOrganizationEndpoint extends Endpoint<Params, Query, Body, Res
                     await this.validateCompanies(organization, request.body.meta.companies);
                     shouldUpdateSetupSteps = true;
                 }
-                const oldMeta = organization.meta.clone();
-
                 const savedPackages = organization.meta.packages;
                 organization.meta.patchOrPut(request.body.meta);
                 organization.meta.packages = savedPackages;
@@ -283,13 +283,6 @@ export class PatchOrganizationEndpoint extends Endpoint<Params, Query, Body, Res
 
                     updateTags = true;
                 }
-
-                await AuditLogService.log({
-                    type: AuditLogType.OrganizationSettingsChanged,
-                    organization,
-                    oldMeta,
-                    patch: request.body.meta,
-                });
             }
 
             if (request.body.active !== undefined) {
@@ -393,8 +386,16 @@ export class PatchOrganizationEndpoint extends Endpoint<Params, Query, Body, Res
         if (updateTags) {
             await TagHelper.updateOrganizations();
         }
+        const struct = await AuthenticatedStructures.organization(organization);
+
+        await AuditLogService.log({
+            type: AuditLogType.OrganizationSettingsChanged,
+            organization,
+            oldData: initialStruct,
+            patch: struct,
+        });
 
-        return new Response(await AuthenticatedStructures.organization(organization));
+        return new Response(struct);
     }
 
     async validateCompanies(organization: Organization, companies: PatchableArrayAutoEncoder<Company> | Company[]) {

package/src/endpoints/organization/dashboard/registration-periods/PatchOrganizationRegistrationPeriodsEndpoint.ts

@@ -425,6 +425,11 @@ export class PatchOrganizationRegistrationPeriodsEndpoint extends Endpoint<Param
             Member.updateMembershipsForGroupId(model.id);
         }
 
+        if (Object.keys(struct).length === 1 && struct.id) {
+            // Nothing changed
+            return;
+        }
+
         await AuditLogService.log({
             type: AuditLogType.GroupEdited,
             group: model,

package/src/endpoints/organization/dashboard/stripe/ConnectStripeEndpoint.ts

@@ -1,11 +1,12 @@
 import { DecodedRequest, Endpoint, Request, Response } from '@simonbackx/simple-endpoints';
 import { SimpleError } from '@simonbackx/simple-errors';
 import { StripeAccount } from '@stamhoofd/models';
-import { PermissionLevel, StripeAccount as StripeAccountStruct } from '@stamhoofd/structures';
+import { AuditLogType, PermissionLevel, StripeAccount as StripeAccountStruct } from '@stamhoofd/structures';
 import Stripe from 'stripe';
 
 import { Context } from '../../../../helpers/Context';
 import { StripeHelper } from '../../../../helpers/StripeHelper';
+import { AuditLogService } from '../../../../services/AuditLogService';
 type Params = Record<string, never>;
 type Body = undefined;
 type Query = undefined;
@@ -91,6 +92,12 @@ export class ConnectMollieEndpoint extends Endpoint<Params, Query, Body, Respons
         model.setMetaFromStripeAccount(account);
         await model.save();
 
+        // Track audit log
+        await AuditLogService.log({
+            type: AuditLogType.StripeAccountAdded,
+            stripeAccount: model,
+        });
+
         // Return information about the Stripe Account
 
         return new Response(StripeAccountStruct.create(model));

package/src/endpoints/organization/dashboard/stripe/DeleteStripeAccountEndpoint.ts

@@ -1,10 +1,11 @@
 import { DecodedRequest, Endpoint, Request, Response } from '@simonbackx/simple-endpoints';
 import { StripeAccount } from '@stamhoofd/models';
-import { PermissionLevel } from '@stamhoofd/structures';
+import { AuditLogType, PermissionLevel } from '@stamhoofd/structures';
 
 import { Context } from '../../../../helpers/Context';
 import { StripeHelper } from '../../../../helpers/StripeHelper';
 import { SimpleError } from '@simonbackx/simple-errors';
+import { AuditLogService } from '../../../../services/AuditLogService';
 
 type Params = { id: string };
 type Body = undefined;
@@ -63,6 +64,12 @@ export class DeleteStripeAccountEndpoint extends Endpoint<Params, Query, Body, R
         model.status = 'deleted';
         await model.save();
 
+        // Track audit log
+        await AuditLogService.log({
+            type: AuditLogType.StripeAccountDeleted,
+            stripeAccount: model,
+        });
+
         return new Response(undefined);
     }
 }

package/src/endpoints/organization/dashboard/stripe/UpdateStripeAccountEndpoint.ts

@@ -1,9 +1,11 @@
 import { DecodedRequest, Endpoint, Request, Response } from '@simonbackx/simple-endpoints';
 import { StripeAccount } from '@stamhoofd/models';
-import { PermissionLevel, StripeAccount as StripeAccountStruct } from '@stamhoofd/structures';
+import { AuditLogType, PermissionLevel, StripeAccount as StripeAccountStruct } from '@stamhoofd/structures';
 
 import { Context } from '../../../../helpers/Context';
 import { StripeHelper } from '../../../../helpers/StripeHelper';
+import { AuditLogService } from '../../../../services/AuditLogService';
+import { Model } from '@simonbackx/simple-database';
 
 type Params = { id: string };
 type Body = undefined;
@@ -43,8 +45,18 @@ export class UpdateStripeAccountEndpoint extends Endpoint<Params, Query, Body, R
         // Get account
         const stripe = StripeHelper.getInstance();
         const account = await stripe.accounts.retrieve(model.accountId);
+        const beforeMeta = model.meta;
         model.setMetaFromStripeAccount(account);
-        await model.save();
+
+        if (await model.save()) {
+            // Track audit log
+            await AuditLogService.log({
+                type: AuditLogType.StripeAccountEdited,
+                stripeAccount: model,
+                oldData: beforeMeta,
+                patch: model.meta,
+            });
+        }
 
         return new Response(StripeAccountStruct.create(model));
     }

package/src/helpers/PeriodHelper.ts

@@ -5,14 +5,17 @@ import { Group as GroupStruct, PermissionLevel } from '@stamhoofd/structures';
 import { PatchOrganizationRegistrationPeriodsEndpoint } from '../endpoints/organization/dashboard/registration-periods/PatchOrganizationRegistrationPeriodsEndpoint';
 import { AuthenticatedStructures } from './AuthenticatedStructures';
 import { MemberUserSyncer } from './MemberUserSyncer';
+import { AuditLogService } from '../services/AuditLogService';
 
 export class PeriodHelper {
     static async moveOrganizationToPeriod(organization: Organization, period: RegistrationPeriod) {
-        console.log('moveOrganizationToPeriod', organization.id, period.id);
+        await AuditLogService.disable(async () => {
+            console.log('moveOrganizationToPeriod', organization.id, period.id);
 
-        await this.createOrganizationPeriodForPeriod(organization, period);
-        organization.periodId = period.id;
-        await organization.save();
+            await this.createOrganizationPeriodForPeriod(organization, period);
+            organization.periodId = period.id;
+            await organization.save();
+        });
     }
 
     static async stopAllResponsibilities() {
@@ -169,29 +172,31 @@ export class PeriodHelper {
 
         const batchSize = 100;
         await QueueHandler.schedule(tag, async () => {
-            let lastId = '';
-
-            while (true) {
-                const groups = await Group.where(
-                    {
-                        id: { sign: '>', value: lastId },
-                        periodId: period.id,
-                    },
-                    {
-                        limit: batchSize,
-                        sort: ['id'],
-                    },
-                );
-
-                for (const group of groups) {
-                    await PatchOrganizationRegistrationPeriodsEndpoint.patchGroup(GroupStruct.patch({ id: group.id }), period);
-                    lastId = group.id;
-                }
+            await AuditLogService.disable(async () => {
+                let lastId = '';
+
+                while (true) {
+                    const groups = await Group.where(
+                        {
+                            id: { sign: '>', value: lastId },
+                            periodId: period.id,
+                        },
+                        {
+                            limit: batchSize,
+                            sort: ['id'],
+                        },
+                    );
+
+                    for (const group of groups) {
+                        await PatchOrganizationRegistrationPeriodsEndpoint.patchGroup(GroupStruct.patch({ id: group.id }), period);
+                        lastId = group.id;
+                    }
 
-                if (groups.length < batchSize) {
-                    break;
+                    if (groups.length < batchSize) {
+                        break;
+                    }
                 }
-            }
+            });
         });
     }
 }

package/src/services/AuditLogService.ts

@@ -1,8 +1,9 @@
 import { AutoEncoder, AutoEncoderPatchType } from '@simonbackx/simple-encoding';
-import { AuditLog, Group, Member, Organization, Registration, Event } from '@stamhoofd/models';
+import { AuditLog, Group, Member, Organization, Registration, Event, RegistrationPeriod, StripeAccount } from '@stamhoofd/models';
 import { AuditLogReplacement, AuditLogReplacementType, AuditLogType, GroupType, MemberDetails, OrganizationMetaData, OrganizationPrivateMetaData, PlatformConfig, PlatformPrivateConfig } from '@stamhoofd/structures';
 import { Context } from '../helpers/Context';
 import { explainPatch } from './explainPatch';
+import { AsyncLocalStorage } from 'node:async_hooks';
 
 export type MemberAddedAuditOptions = {
     type: AuditLogType.MemberAdded;
@@ -25,24 +26,16 @@ export type MemberRegisteredAuditOptions = {
 
 export type PlatformConfigChangeAuditOptions = {
     type: AuditLogType.PlatformSettingsChanged;
-} & ({
-    oldConfig: PlatformPrivateConfig;
-    patch: PlatformPrivateConfig | AutoEncoderPatchType<PlatformPrivateConfig>;
-} | {
-    oldConfig: PlatformConfig;
-    patch: PlatformConfig | AutoEncoderPatchType<PlatformConfig>;
-});
+    oldData?: AutoEncoder;
+    patch?: AutoEncoder | AutoEncoderPatchType<AutoEncoder>;
+};
 
 export type OrganizationConfigChangeAuditOptions = {
     type: AuditLogType.OrganizationSettingsChanged;
     organization: Organization;
-} & ({
-    oldMeta: OrganizationMetaData;
-    patch: OrganizationMetaData | AutoEncoderPatchType<OrganizationMetaData>;
-} | {
-    oldMeta: OrganizationPrivateMetaData;
-    patch: OrganizationPrivateMetaData | AutoEncoderPatchType<OrganizationPrivateMetaData>;
-});
+    oldData?: AutoEncoder;
+    patch?: AutoEncoder | AutoEncoderPatchType<AutoEncoder>;
+};
 
 export type EventAuditOptions = {
     type: AuditLogType.EventAdded | AuditLogType.EventEdited | AuditLogType.EventDeleted;
@@ -58,11 +51,46 @@ export type GroupAuditOptions = {
     patch?: AutoEncoder | AutoEncoderPatchType<AutoEncoder>;
 };
 
-export type AuditLogOptions = GroupAuditOptions | EventAuditOptions | MemberAddedAuditOptions | MemberEditedAuditOptions | MemberRegisteredAuditOptions | PlatformConfigChangeAuditOptions | OrganizationConfigChangeAuditOptions;
+export type PeriodAuditOptions = {
+    type: AuditLogType.RegistrationPeriodAdded | AuditLogType.RegistrationPeriodEdited | AuditLogType.RegistrationPeriodDeleted;
+    period: RegistrationPeriod;
+    oldData?: AutoEncoder;
+    patch?: AutoEncoder | AutoEncoderPatchType<AutoEncoder>;
+};
+
+export type StripeAccountAuditOptions = {
+    type: AuditLogType.StripeAccountAdded | AuditLogType.StripeAccountEdited | AuditLogType.StripeAccountDeleted;
+    stripeAccount: StripeAccount;
+    oldData?: AutoEncoder;
+    patch?: AutoEncoder | AutoEncoderPatchType<AutoEncoder>;
+};
+
+export type AuditLogOptions = StripeAccountAuditOptions | PeriodAuditOptions | GroupAuditOptions | EventAuditOptions | MemberAddedAuditOptions | MemberEditedAuditOptions | MemberRegisteredAuditOptions | PlatformConfigChangeAuditOptions | OrganizationConfigChangeAuditOptions;
 
 export const AuditLogService = {
+    disableLocalStore: new AsyncLocalStorage<boolean>(),
+
+    disable<T extends Promise<void> | void>(run: () => T): T {
+        return this.disableLocalStore.run(true, () => {
+            return run();
+        });
+    },
+
+    isDisabled(): boolean {
+        const c = this.disableLocalStore.getStore();
+
+        if (!c) {
+            return false;
+        }
+
+        return true;
+    },
+
     async log(options: AuditLogOptions) {
         try {
+            if (this.isDisabled()) {
+                return;
+            }
             const userId = Context.optionalAuth?.user?.id ?? null;
             const organizationId = Context.organization?.id ?? null;
 
@@ -96,11 +124,18 @@ export const AuditLogService = {
             else if (options.type === AuditLogType.GroupAdded || options.type === AuditLogType.GroupEdited || options.type === AuditLogType.GroupDeleted) {
                 this.fillForGroup(model, options);
             }
+            else if (options.type === AuditLogType.RegistrationPeriodAdded || options.type === AuditLogType.RegistrationPeriodEdited || options.type === AuditLogType.RegistrationPeriodDeleted) {
+                this.fillForPeriod(model, options);
+            }
+            else if (options.type === AuditLogType.StripeAccountAdded || options.type === AuditLogType.StripeAccountEdited || options.type === AuditLogType.StripeAccountDeleted) {
+                if (this.fillForStripeAccount(model, options) === false) {
+                    // do not save
+                    return;
+                }
+            }
 
             // In the future we might group these saves together in one query to improve performance
             await model.save();
-
-            console.log('Audit log', model.id, options);
         }
         catch (e) {
             console.error('Failed to save log', options, e);
@@ -162,7 +197,10 @@ export const AuditLogService = {
         model.objectId = null;
 
         // Generate changes list
-        model.patchList = explainPatch(options.oldConfig, options.patch);
+        if (options.patch) {
+            // Generate changes list
+            model.patchList = explainPatch(options.oldData ?? null, options.patch);
+        }
     },
 
     fillForOrganizationConfig(model: AuditLog, options: OrganizationConfigChangeAuditOptions) {
@@ -178,7 +216,10 @@ export const AuditLogService = {
         ]);
 
         // Generate changes list
-        model.patchList = explainPatch(options.oldMeta, options.patch);
+        if (options.patch) {
+            // Generate changes list
+            model.patchList = explainPatch(options.oldData ?? null, options.patch);
+        }
     },
 
     fillForEvent(model: AuditLog, options: EventAuditOptions) {
@@ -229,4 +270,48 @@ export const AuditLogService = {
             })],
         ]);
     },
+
+    fillForPeriod(model: AuditLog, options: PeriodAuditOptions) {
+        model.objectId = options.period.id;
+
+        if (options.patch) {
+            // Generate changes list
+            model.patchList = explainPatch(options.oldData ?? null, options.patch);
+        }
+
+        model.replacements = new Map([
+            ['p', AuditLogReplacement.create({
+                id: options.period.id,
+                value: options.period.getStructure().nameShort,
+                type: AuditLogReplacementType.RegistrationPeriod,
+            })],
+        ]);
+    },
+
+    fillForStripeAccount(model: AuditLog, options: StripeAccountAuditOptions) {
+        model.objectId = options.stripeAccount.id;
+
+        if (options.patch) {
+            // Generate changes list
+            model.patchList = explainPatch(options.oldData ?? null, options.patch);
+
+            if (model.patchList.length === 0) {
+                // No changes, ignore (only for stripe)
+                return false;
+            }
+        }
+
+        if (options.type === AuditLogType.StripeAccountEdited) {
+            // Never caused by a user
+            model.userId = null;
+        }
+
+        model.replacements = new Map([
+            ['a', AuditLogReplacement.create({
+                id: options.stripeAccount.id,
+                value: options.stripeAccount.accountId,
+                type: AuditLogReplacementType.StripeAccount,
+            })],
+        ]);
+    },
 };