Skip to content

Commit a9e7f36

Browse files
committed
fix(webhooks): add clarifying comment on Slack error path trade-off
1 parent 07822d1 commit a9e7f36

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

apps/sim/lib/webhooks/processor.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1218,6 +1218,8 @@ export async function queueWebhookExecution(
12181218
}
12191219

12201220
if (foundWebhook.provider === 'slack') {
1221+
// Return empty 200 to avoid Slack showing an error dialog to the user,
1222+
// even though processing failed. The error is already logged above.
12211223
return new NextResponse(null, { status: 200 })
12221224
}
12231225

0 commit comments

Comments
 (0)