github/sim
1
1
Fork 0
mirror of https://github.com/simstudioai/sim.git synced 2026-04-06 03:00:16 -04:00
Code Issues Packages Projects Releases Wiki Activity

improvement(audit-log): replace UUIDs with names in descriptions, add missing resourceName fields

Browse Source
This commit is contained in:
waleed
2026-02-18 10:11:11 -08:00
parent 35a69dce07
commit e3bbffd55a
22 changed files with 31 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
apps/sim/app/api/chat/manage/[id]/route.ts
View File

@@ -292,8 +292,8 @@ export async function DELETE(
action: AuditAction.CHAT_DELETED,
resourceType: AuditResourceType.CHAT,
resourceId: chatId,
resourceName: chatRecord?.title || chatId,
description: `Deleted chat deployment "${chatRecord?.title || chatId}"`,
resourceName: chatRecord?.title,
description: `Deleted chat deployment "${chatRecord?.title}"`,
request: _request,
})

2
apps/sim/app/api/credential-sets/[id]/invite/route.ts
View File

@@ -258,7 +258,7 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: result.set.name,
description: `Revoked invitation "${invitationId}" for credential set "${result.set.name}"`,
description: `Revoked an invitation for credential set "${result.set.name}"`,
request: req,
})

2
apps/sim/app/api/form/route.ts
View File

@@ -205,7 +205,7 @@ export async function POST(request: NextRequest) {
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: title,
description: `Created form "${title}" for workflow ${workflowId}`,
description: `Created form "${title}" for workflow "${workflowRecord.name}"`,
request,
})

4
apps/sim/app/api/knowledge/[id]/documents/[documentId]/route.ts
View File

@@ -207,7 +207,7 @@ export async function PUT(
resourceType: AuditResourceType.DOCUMENT,
resourceId: documentId,
resourceName: validatedData.filename ?? accessCheck.document?.filename,
description: `Updated document "${documentId}" in knowledge base "${knowledgeBaseId}"`,
description: `Updated document "${validatedData.filename ?? accessCheck.document?.filename}" in knowledge base "${accessCheck.knowledgeBase?.name}"`,
request: req,
})
@@ -280,7 +280,7 @@ export async function DELETE(
resourceType: AuditResourceType.DOCUMENT,
resourceId: documentId,
resourceName: accessCheck.document?.filename,
description: `Deleted document "${documentId}" from knowledge base "${knowledgeBaseId}"`,
description: `Deleted document "${accessCheck.document?.filename}" from knowledge base "${accessCheck.knowledgeBase?.name}"`,
request: req,
})

4
apps/sim/app/api/knowledge/[id]/documents/route.ts
View File

@@ -254,7 +254,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
resourceType: AuditResourceType.DOCUMENT,
resourceId: knowledgeBaseId,
resourceName: `${createdDocuments.length} document(s)`,
description: `Uploaded ${createdDocuments.length} document(s) to knowledge base "${knowledgeBaseId}"`,
description: `Uploaded ${createdDocuments.length} document(s) to knowledge base "${accessCheck.knowledgeBase?.name}"`,
request: req,
})
@@ -315,7 +315,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
resourceType: AuditResourceType.DOCUMENT,
resourceId: knowledgeBaseId,
resourceName: validatedData.filename,
description: `Uploaded document "${validatedData.filename}" to knowledge base "${knowledgeBaseId}"`,
description: `Uploaded document "${validatedData.filename}" to knowledge base "${accessCheck.knowledgeBase?.name}"`,
request: req,
})

2
apps/sim/app/api/knowledge/[id]/route.ts
View File

@@ -220,7 +220,7 @@ export async function DELETE(
resourceType: AuditResourceType.KNOWLEDGE_BASE,
resourceId: id,
resourceName: accessCheck.knowledgeBase.name,
description: `Deleted knowledge base "${accessCheck.knowledgeBase.name || id}"`,
description: `Deleted knowledge base "${accessCheck.knowledgeBase.name}"`,
request: _request,
})

4
apps/sim/app/api/mcp/servers/[id]/route.ts
View File

@@ -99,8 +99,8 @@ export const PATCH = withMcpAuth<{ id: string }>('write')(
action: AuditAction.MCP_SERVER_UPDATED,
resourceType: AuditResourceType.MCP_SERVER,
resourceId: serverId,
resourceName: updatedServer.name || serverId,
description: `Updated MCP server "${updatedServer.name || serverId}"`,
resourceName: updatedServer.name,
description: `Updated MCP server "${updatedServer.name}"`,
request,
})

2
apps/sim/app/api/mcp/workflow-servers/[id]/tools/[toolId]/route.ts
View File

@@ -131,6 +131,7 @@ export const PATCH = withMcpAuth<RouteParams>('write')(
action: AuditAction.MCP_SERVER_UPDATED,
resourceType: AuditResourceType.MCP_SERVER,
resourceId: serverId,
resourceName: updatedTool.toolName,
description: `Updated tool "${updatedTool.toolName}" in MCP server`,
metadata: { toolId, toolName: updatedTool.toolName },
request,
@@ -195,6 +196,7 @@ export const DELETE = withMcpAuth<RouteParams>('write')(
action: AuditAction.MCP_SERVER_UPDATED,
resourceType: AuditResourceType.MCP_SERVER,
resourceId: serverId,
resourceName: deletedTool.toolName,
description: `Removed tool "${deletedTool.toolName}" from MCP server`,
metadata: { toolId, toolName: deletedTool.toolName },
request,

1
apps/sim/app/api/mcp/workflow-servers/[id]/tools/route.ts
View File

@@ -210,6 +210,7 @@ export const POST = withMcpAuth<RouteParams>('write')(
action: AuditAction.MCP_SERVER_UPDATED,
resourceType: AuditResourceType.MCP_SERVER,
resourceId: serverId,
resourceName: toolName,
description: `Added tool "${toolName}" to MCP server`,
metadata: { toolId, toolName, workflowId: body.workflowId },
request,

1
apps/sim/app/api/organizations/[id]/invitations/[invitationId]/route.ts
View File

@@ -567,6 +567,7 @@ export async function PUT(
resourceId: organizationId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: orgInvitation.email,
description: `Organization invitation ${status} for ${orgInvitation.email}`,
metadata: { invitationId, email: orgInvitation.email, status },
request: req,

1
apps/sim/app/api/organizations/[id]/invitations/route.ts
View File

@@ -557,6 +557,7 @@ export async function DELETE(
resourceId: organizationId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: result[0].email,
description: `Revoked organization invitation for ${result[0].email}`,
metadata: { invitationId, email: result[0].email },
request,

4
apps/sim/app/api/organizations/[id]/members/[memberId]/route.ts
View File

@@ -222,7 +222,7 @@ export async function PUT(
resourceId: organizationId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
description: `Changed role for member ${memberId} to ${role}`,
description: `Changed member role to ${role}`,
metadata: { targetUserId: memberId, newRole: role },
request,
})
@@ -330,7 +330,7 @@ export async function DELETE(
description:
session.user.id === targetUserId
? 'Left the organization'
: `Removed member ${targetUserId} from organization`,
: 'Removed a member from the organization',
metadata: { targetUserId, wasSelfRemoval: session.user.id === targetUserId },
request,
})

1
apps/sim/app/api/organizations/[id]/members/route.ts
View File

@@ -294,6 +294,7 @@ export async function POST(request: NextRequest, { params }: { params: Promise<{
resourceId: organizationId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: normalizedEmail,
description: `Invited ${normalizedEmail} to organization as ${role}`,
metadata: { invitationId, email: normalizedEmail, role },
request,

4
apps/sim/app/api/permission-groups/[id]/members/route.ts
View File

@@ -162,7 +162,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
resourceName: result.group.name,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
description: `Added member ${userId} to permission group "${result.group.name}"`,
description: `Added a member to permission group "${result.group.name}"`,
metadata: { targetUserId: userId, permissionGroupId: id },
request: req,
})
@@ -246,7 +246,7 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
resourceName: result.group.name,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
description: `Removed member ${memberToRemove.userId} from permission group "${result.group.name}"`,
description: `Removed a member from permission group "${result.group.name}"`,
metadata: { targetUserId: memberToRemove.userId, memberId, permissionGroupId: id },
request: req,
})

3
apps/sim/app/api/schedules/[id]/route.ts
View File

@@ -115,7 +115,8 @@ export async function PUT(request: NextRequest, { params }: { params: Promise<{
resourceId: scheduleId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
description: `Reactivated schedule for workflow ${schedule.workflowId}`,
resourceName: authorization.workflow?.name,
description: `Reactivated schedule for workflow "${authorization.workflow?.name}"`,
request,
})

4
apps/sim/app/api/workflows/[id]/deploy/route.ts
View File

@@ -268,7 +268,7 @@ export async function POST(request: NextRequest, { params }: { params: Promise<{
resourceType: AuditResourceType.WORKFLOW,
resourceId: id,
resourceName: workflowData?.name,
description: `Deployed workflow "${workflowData?.name || id}"`,
description: `Deployed workflow "${workflowData.name}"`,
request,
})
@@ -348,7 +348,7 @@ export async function DELETE(
resourceType: AuditResourceType.WORKFLOW,
resourceId: id,
resourceName: workflowData?.name,
description: `Undeployed workflow "${workflowData?.name || id}"`,
description: `Undeployed workflow "${workflowData.name}"`,
request,
})

2
apps/sim/app/api/workflows/[id]/duplicate/route.ts
View File

@@ -71,7 +71,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
resourceType: AuditResourceType.WORKFLOW,
resourceId: result.id,
resourceName: result.name,
description: `Duplicated workflow from ${sourceWorkflowId}`,
description: `Duplicated workflow as "${result.name}"`,
metadata: { sourceWorkflowId },
request: req,
})

1
apps/sim/app/api/workspaces/[id]/byok-keys/route.ts
View File

@@ -264,6 +264,7 @@ export async function DELETE(
actorEmail: session?.user?.email,
action: AuditAction.BYOK_KEY_DELETED,
resourceType: AuditResourceType.BYOK_KEY,
resourceId: providerId,
resourceName: providerId,
description: `Removed BYOK key for ${providerId}`,
metadata: { providerId },

1
apps/sim/app/api/workspaces/[id]/environment/route.ts
View File

@@ -165,6 +165,7 @@ export async function PUT(request: NextRequest, { params }: { params: Promise<{
action: AuditAction.ENVIRONMENT_UPDATED,
resourceType: AuditResourceType.ENVIRONMENT,
resourceId: workspaceId,
resourceName: 'Environment Variables',
description: `Updated environment variables`,
metadata: { keysUpdated: Object.keys(variables) },
request,

2
apps/sim/app/api/workspaces/[id]/permissions/route.ts
View File

@@ -166,7 +166,7 @@ export async function PATCH(request: NextRequest, { params }: { params: Promise<
resourceId: workspaceId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
description: `Changed permissions for user ${update.userId} to ${update.permissions}`,
description: `Changed workspace permissions to ${update.permissions}`,
metadata: { targetUserId: update.userId, newPermissions: update.permissions },
request,
})

2
apps/sim/app/api/workspaces/[id]/route.ts
View File

@@ -298,7 +298,7 @@ export async function DELETE(
resourceType: AuditResourceType.WORKSPACE,
resourceId: workspaceId,
resourceName: workspaceRecord?.name,
description: `Deleted workspace "${workspaceRecord?.name || workspaceId}"`,
description: `Deleted workspace "${workspaceRecord?.name}"`,
request,
})

1
apps/sim/app/api/workspaces/invitations/[invitationId]/route.ts
View File

@@ -238,6 +238,7 @@ export async function DELETE(
resourceId: invitation.workspaceId,
actorName: session.user.name ?? undefined,
actorEmail: session.user.email ?? undefined,
resourceName: invitation.email,
description: `Revoked workspace invitation for ${invitation.email}`,
metadata: { invitationId, email: invitation.email },
request: _request,