fix(gateway): fail loudly when enabled integration omits webhook config - #457
Conversation
yordis
commented
Jul 1, 2026
- Enabled webhook integrations that omitted their webhook (or Slack transport) block were silently dropped, so operators got no signal that required configuration was missing and events simply never arrived.
Enabled integrations missing their webhook or transport block were silently dropped, leaving operators with no signal that required configuration was absent. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryLow Risk Overview For GitHub, Telegram, Twitter, GitLab, Linear, Microsoft Graph, incidentio, Notion, and Sentry, an enabled integration with no New tests assert each source returns the expected validation message for Reviewed by Cursor Bugbot for commit 70f7839. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughThis PR modifies ChangesIntegration missing-webhook validation
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Code Coverage SummaryDetailsDiff against mainResults for commit: 70f7839 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…ig (#457) Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>