Skip to content

Commit dcd14f9

Browse files
committed
fix: update createLogRecord and logCustomAction to return a success response for external users
1 parent e622505 commit dcd14f9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ export default class AuditLogPlugin extends AdminForthPlugin {
9191
createLogRecord = async (resource: AdminForthResource, action: AllowedActionsEnum | string, data: Object, user: AdminUser, oldRecord?: Object, extra?: HttpExtra) => {
9292
if (user.isExternalUser) {
9393
// audit log does not support logging for external (non-adminforth) users.
94-
return;
94+
return { ok: true };
9595
}
9696

9797
const recordIdFieldName = resource.columns.find((c) => c.primaryKey === true)?.name;
@@ -187,7 +187,7 @@ export default class AuditLogPlugin extends AdminForthPlugin {
187187
const { resourceId, recordId, actionId, oldData, data, user, headers } = params;
188188

189189
if (user.isExternalUser) {
190-
return;
190+
return { ok: true };
191191
}
192192

193193
// if type of params is not object, throw error

0 commit comments

Comments
 (0)