From 64ad1c8d5be1738041816fa561029d5914896f56 Mon Sep 17 00:00:00 2001 From: Carina Wollendorfer <30310907+CarinaWolli@users.noreply.github.com> Date: Sat, 18 May 2024 06:12:07 -0400 Subject: [PATCH] rate limiting fix (#15095) Co-authored-by: CarinaWolli Co-authored-by: Udit Takkar <53316345+Udit-takkar@users.noreply.github.com> --- .../ee/workflows/lib/reminders/reminderScheduler.ts | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts b/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts index 68422b62cc..5a4848249e 100644 --- a/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts +++ b/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts @@ -1,6 +1,10 @@ import type { Workflow, WorkflowsOnEventTypes, WorkflowStep } from "@prisma/client"; -import { isSMSAction, isWhatsappAction } from "@calcom/features/ee/workflows/lib/actionHelperFunctions"; +import { + isSMSAction, + isSMSOrWhatsappAction, + isWhatsappAction, +} from "@calcom/features/ee/workflows/lib/actionHelperFunctions"; import { checkSMSRateLimit } from "@calcom/lib/checkRateLimitAndThrowError"; import { SENDER_NAME } from "@calcom/lib/constants"; import { WorkflowActions, WorkflowMethods, WorkflowTriggerEvents } from "@calcom/prisma/enums"; @@ -46,12 +50,14 @@ const processWorkflowStep = async ( seatReferenceUid, }: ProcessWorkflowStepParams ) => { - if (isSMSAction(step.action)) { + if (isSMSOrWhatsappAction(step.action)) { await checkSMSRateLimit({ identifier: `sms:${workflow.teamId ? "team:" : "user:"}${workflow.teamId || workflow.userId}`, rateLimitingType: "sms", }); + } + if (isSMSAction(step.action)) { const sendTo = step.action === WorkflowActions.SMS_ATTENDEE ? smsReminderNumber : step.sendTo; await scheduleSMSReminder({ evt,