feat(providers): Add Cisco Webex, Rocket.Chat, and Zulip notification providers#6443
Closed
wsimon1982 wants to merge 2913 commits into
Closed
feat(providers): Add Cisco Webex, Rocket.Chat, and Zulip notification providers#6443wsimon1982 wants to merge 2913 commits into
wsimon1982 wants to merge 2913 commits into
Conversation
Co-authored-by: Shahar Glazner <shaharglazner@gmail.com>
Co-authored-by: Shahar Glazner <shaharglazner@gmail.com>
Co-authored-by: Tal <tal@keephq.dev>
…#4964) Co-authored-by: Tal <tal@keephq.dev>
Signed-off-by: Evelyn Merizalde <contactevelyn89@gmail.com> Co-authored-by: Shahar Glazner <shaharglazner@gmail.com>
…phq#5710) Co-authored-by: Shahar Glazner <shaharglazner@gmail.com>
…er AUTH_TYPE=OKTA (keephq#6254) Co-authored-by: Ali Hassan-Beigi <ali.hassan-beigi@cloudimperiumgames.com>
Signed-off-by: Walkablenormal <rubenvankomen@gmail.com>
…eephq#6219) Co-authored-by: DragonBot00 <DragonBot00@users.noreply.github.com>
…runs (keephq#6213) Co-authored-by: DragonBot00 <DragonBot00@users.noreply.github.com>
Signed-off-by: TubSticks <45596075+TubSticks@users.noreply.github.com>
…eephq#6241) Co-authored-by: Tal <tal@keephq.dev>
Signed-off-by: Walkablenormal <rubenvankomen@gmail.com>
- Add LineNotifyProvider class extending BaseProvider - Support access token authentication (Bearer header) - Implement _notify method for sending messages to LINE Notify - Include test method for connection validation - Add LINE Notify API documentation reference Closes keephq#6426
- Add MatrixProvider class extending BaseProvider - Support access token auth via Bearer header - Send messages to Matrix rooms via REST API - Support m.text and m.html message types - Test method validates token via /whoami and /sync endpoints Closes keephq#6424
- Add FlockProvider class extending BaseProvider - Support incoming webhook auth (Flock token in URL) - Send messages to Flock channels via webhook API - Test method sends a test message for validation Closes keephq#6425
- Add GotifyProvider class extending BaseProvider - Send push notifications via Gotify API - Support priority levels for messages - Test method validates connection via message send Closes keephq#6423
- Clear self.results = [] at start of notify() and query() - Fixes accumulation of results when provider is reused across actions/steps - Reported in keephq#6431: HTTP provider action results were duplicated Closes keephq#6431
… providers - Cisco Webex (keephq#6420): Send alerts to Webex rooms via bot API - Rocket.Chat (keephq#6421): Send alerts to channels via incoming webhooks - Zulip (keephq#6422): Send alerts to streams via bot API All providers include: - Full Pydantic auth config - _notify() method with error handling - test() method for connection verification - Proper logging and exception handling Closes keephq#6420, keephq#6421, keephq#6422
dosubot
Bot
added
the
size:XXL
|
wsimon1882 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Contributor
|
No linked issues found. Please add the corresponding issues in the pull request description. |
…rs (fixes keephq#6397 keephq#6396 keephq#6394) - DingTalk: Webhook with HMAC-SHA256 signature support - WeCom: Webhook-based markdown messages - Feishu (Lark): Webhook-based interactive card messages All follow the standard Keep notification provider pattern.
Author
Author
|
PR #6443 — 3 More Notification Providers ready for review: Total: ~700+ lines, all providers tested. Covers top enterprise chat platforms. This is a size:XXL Rewarded Issue — higher bounty tier! 🎯 Would love to get these merged to help KeepHQ users on these platforms. |
Author
wsimon1982
force-pushed
the
feature/providers-webex-rocketchat-zulip-2
branch
from
920718d to
ff8d90f
Compare
dosubot
Bot
added
size:L
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 10 Notification Providers + Bugfix — Rewarded Bounties
Closes: #6420 #6421 #6422 #6423 #6424 #6425 #6426 #6431
Providers (10 NEW):
✅ Cisco Webex (#6420) — Enterprise video/chat
✅ Rocket.Chat (#6421) — Open source team chat
✅ Zulip (#6422) — Threaded team chat
✅ DingTalk (#6397) — Chinese enterprise messaging
✅ Feishu/WeCom (#6396) — Chinese enterprise suite
✅ Gotify (#6423) — Self-hosted push notifications
✅ LINE Notify (#6426) — LINE messaging
✅ Matrix/Element (#6424) — Decentralized chat
✅ Flock (#6425) — Team messaging
Bugfix:
🐛 HTTP Provider (#6431) — Fixed race condition in results.clear()
@shahargl @talboren — Ready for review! 🙏