fix: notification same provider (#1190) · openstatus.dev/openstatus@f04928c

openstatus.dev / openstatus

fork atom

Openstatus www.openstatus.dev

fork atom

fix: notification same provider (#1190)

authored by

Maximilian Kaske and committed by

GitHub 1 year ago f04928cf d1a51738

+1 -1

1 changed file

expand all

apps

server

src

routes

checker

alerting.ts

+1 -1

apps/server/src/routes/checker/alerting.ts

··· 42 42 .where(eq(schema.monitor.id, Number(monitorId))) 43 43 .all(); 44 44 for (const notif of notifications) { 45 - const key = `${monitorId}:${incidentId}:${notif.notification.provider}:${notifType}`; 45 + const key = `${monitorId}:${incidentId}:${notif.notification.id}:${notifType}`; 46 46 const r = await redis.setnx(key, "1"); 47 47 if (r === 0) { 48 48 console.log(`🤔 notification already sent for ${key}`);