Skip to content

Commit 7e394ad

Browse files
committed
fix(audit-log): replace remaining workspaceId '' fallbacks with null
1 parent ea6c276 commit 7e394ad

File tree

7 files changed

+10
-10
lines changed

7 files changed

+10
-10
lines changed

apps/sim/app/api/form/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ export async function POST(request: NextRequest) {
197197
logger.info(`Form "${title}" deployed successfully at ${formUrl}`)
198198

199199
recordAudit({
200-
workspaceId: workflowRecord.workspaceId ?? '',
200+
workspaceId: workflowRecord.workspaceId ?? null,
201201
actorId: session.user.id,
202202
action: AuditAction.FORM_CREATED,
203203
resourceType: AuditResourceType.FORM,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ export async function PUT(
199199
)
200200

201201
recordAudit({
202-
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? '',
202+
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? null,
203203
actorId: userId,
204204
action: AuditAction.DOCUMENT_UPDATED,
205205
resourceType: AuditResourceType.DOCUMENT,
@@ -270,7 +270,7 @@ export async function DELETE(
270270
)
271271

272272
recordAudit({
273-
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? '',
273+
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? null,
274274
actorId: userId,
275275
action: AuditAction.DOCUMENT_DELETED,
276276
resourceType: AuditResourceType.DOCUMENT,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
246246
})
247247

248248
recordAudit({
249-
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? '',
249+
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? null,
250250
actorId: userId,
251251
action: AuditAction.DOCUMENT_UPLOADED,
252252
resourceType: AuditResourceType.DOCUMENT,
@@ -305,7 +305,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
305305
}
306306

307307
recordAudit({
308-
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? '',
308+
workspaceId: accessCheck.knowledgeBase?.workspaceId ?? null,
309309
actorId: userId,
310310
action: AuditAction.DOCUMENT_UPLOADED,
311311
resourceType: AuditResourceType.DOCUMENT,

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ export async function PUT(req: NextRequest, { params }: { params: Promise<{ id:
137137
logger.info(`[${requestId}] Knowledge base updated: ${id} for user ${userId}`)
138138

139139
recordAudit({
140-
workspaceId: accessCheck.knowledgeBase.workspaceId ?? '',
140+
workspaceId: accessCheck.knowledgeBase.workspaceId ?? null,
141141
actorId: userId,
142142
action: AuditAction.KNOWLEDGE_BASE_UPDATED,
143143
resourceType: AuditResourceType.KNOWLEDGE_BASE,
@@ -210,7 +210,7 @@ export async function DELETE(
210210
logger.info(`[${requestId}] Knowledge base deleted: ${id} for user ${userId}`)
211211

212212
recordAudit({
213-
workspaceId: accessCheck.knowledgeBase.workspaceId ?? '',
213+
workspaceId: accessCheck.knowledgeBase.workspaceId ?? null,
214214
actorId: userId,
215215
action: AuditAction.KNOWLEDGE_BASE_DELETED,
216216
resourceType: AuditResourceType.KNOWLEDGE_BASE,

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ export async function PUT(request: NextRequest, { params }: { params: Promise<{
108108
logger.info(`[${requestId}] Reactivated schedule: ${scheduleId}`)
109109

110110
recordAudit({
111-
workspaceId: authorization.workflow.workspaceId ?? '',
111+
workspaceId: authorization.workflow.workspaceId ?? null,
112112
actorId: session.user.id,
113113
action: AuditAction.SCHEDULE_UPDATED,
114114
resourceType: AuditResourceType.SCHEDULE,

apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ export async function POST(
113113
}
114114

115115
recordAudit({
116-
workspaceId: workflowRecord?.workspaceId ?? '',
116+
workspaceId: workflowRecord?.workspaceId ?? null,
117117
actorId: session!.user.id,
118118
action: AuditAction.WORKFLOW_DEPLOYMENT_REVERTED,
119119
resourceType: AuditResourceType.WORKFLOW,

apps/sim/app/api/workflows/[id]/variables/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
8181
.where(eq(workflow.id, workflowId))
8282

8383
recordAudit({
84-
workspaceId: workflowData.workspaceId ?? '',
84+
workspaceId: workflowData.workspaceId ?? null,
8585
actorId: userId,
8686
action: AuditAction.WORKFLOW_VARIABLES_UPDATED,
8787
resourceType: AuditResourceType.WORKFLOW,

0 commit comments

Comments
 (0)