Skip to content

feat(providers): 10 notification providers + HTTP bugfix#6468

Open
wsimon1982 wants to merge 7 commits into
keephq:mainfrom
wsimon1982:providers-final-clean
Open

feat(providers): 10 notification providers + HTTP bugfix#6468
wsimon1982 wants to merge 7 commits into
keephq:mainfrom
wsimon1982:providers-final-clean

Conversation

@wsimon1982
Copy link
Copy Markdown

@wsimon1982 wsimon1982 commented May 12, 2026

10 Notification Providers + HTTP Bugfix

Closes #6420 #6421 #6422 #6423 #6424 #6425 #6426 #6431

@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label May 12, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


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.

@dosubot dosubot Bot added Bug Something isn't working Feature A new feature Provider Providers related issues labels May 12, 2026
@wsimon1982 wsimon1982 force-pushed the providers-final-clean branch from 22e44d4 to 570a37d Compare May 12, 2026 17:59
@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels May 12, 2026
wsimon1882 added 7 commits May 12, 2026 20:38
- 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
…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.
@wsimon1982 wsimon1982 force-pushed the providers-final-clean branch from 21c44b5 to ff97975 Compare May 12, 2026 18:38
@wsimon1982
Copy link
Copy Markdown
Author

🔄 3rd attempt — All 10 providers in clean commits, all authored by wsimon1882

  • ✅ CI Checks: Validate PR title, Validate PR to Issue link, GitGuardian
  • ⏳ CLA: pending (slow bot)
  • Author: wsimon1882

Bounty: ~$575-875

@shahargl @talboren please review/merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working Feature A new feature Provider Providers related issues size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🔌 Provider]: Cisco Webex notification provider

2 participants