Skip to content

feat: add Plivo SMS provider - #6625

Open
sarveshpatil-plivo wants to merge 4 commits into
keephq:mainfrom
sarveshpatil-plivo:plivo-provider
Open

feat: add Plivo SMS provider#6625
sarveshpatil-plivo wants to merge 4 commits into
keephq:mainfrom
sarveshpatil-plivo:plivo-provider

Conversation

@sarveshpatil-plivo

Copy link
Copy Markdown

Adds a Plivo notification provider so Keep can send SMS alerts when a monitor fires, following the existing notification-provider pattern and sending through the Plivo Messages API.

Closes #6624.

Included

  • keep/providers/plivo_provider/plivo_provider.pyPlivoProvider(BaseProvider) with a send_sms scope; _notify sends via messages.create(src, dst, text).
  • Auth config: Auth ID, Auth Token (sensitive), source number / sender ID.
  • Docs page + nav entry + autogenerated snippet, provider icon, and the plivo dependency (with the poetry.lock update).

Notes

  • Scope validation uses a read (messages.list(limit=1)) instead of a test send — a Plivo send is billed and not idempotent.
  • Auto-registers via the providers factory (no factory edit needed).
  • Supersedes the earlier community PR feat: add Plivo SMS provider #5768 (closed with an unsigned CLA); the CLA will be signed here.

sarveshpatil-plivo and others added 3 commits July 8, 2026 03:20
Adds Plivo as an SMS notification provider, mirroring twilio_provider:
sends alert SMS via the Plivo Messages API (RestClient.messages.create).
Scope validated with a read (Plivo sends are billed). Auto-registers via
ProvidersFactory. Includes provider docs + icon.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 9, 2026
@CLAassistant

CLAassistant commented Jul 9, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@dosubot dosubot Bot added Dependencies Pull requests that update a dependency file Documentation Improvements or additions to documentation Feature A new feature Provider Providers related issues labels Jul 9, 2026
@sarveshpatil-plivo

Copy link
Copy Markdown
Author

Hi @talboren (cc @shahargl ), following up here.
This PR adds a Plivo SMS provider and the CLA is signed. It's a clean redo of the earlier community attempt (#5768 ) that was closed for an unsigned CLA.
Could you tell me if anything's missing or should something be changed?
Happy to make any changes required.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dependencies Pull requests that update a dependency file Documentation Improvements or additions to documentation Feature A new feature Provider Providers related issues size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(feat) Plivo provider

2 participants