fix: should check team + service + name instead (adding webhook) (#550)

hyperdxio · Jan 10, 2025 · 9daccff · 9daccff
1 parent e6d8501
commit 9daccff
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/tricky-actors-judge.md b/.changeset/tricky-actors-judge.md
@@ -0,0 +1,5 @@
+---
+"@hyperdx/api": patch
+---
+
+fix: should check team + service + name uniq constraint instead (adding webhook)
diff --git a/packages/api/src/routers/api/webhooks.ts b/packages/api/src/routers/api/webhooks.ts
@@ -58,7 +58,7 @@ router.post(
       }
       const { name, service, url, description, queryParams, headers, body } =
         req.body;
-      if (await Webhook.findOne({ team: teamId, service, url })) {
+      if (await Webhook.findOne({ team: teamId, service, name })) {
         return res.status(400).json({
           message: 'Webhook already exists',
         });