louislam / uptime-kuma

A fancy self-hosted monitoring tool
https://uptime.kuma.pet
MIT License
56.38k stars 5.07k forks source link

Twillo SMS Alerts #1860

Closed SecureCPU closed 1 year ago

SecureCPU commented 2 years ago

⚠️ Please verify that this feature request has NOT been suggested before.

🏷️ Feature Request Type

New Notification

🔖 Feature description

Receiving notification via SMS with Twilio

✔️ Solution

Add Twillo as a SMS option for Notifications

❓ Alternatives

No response

📝 Additional Context

No response

phanxgames commented 2 years ago

This would be awesome.

lctech-zeki commented 1 year ago

plz i need this feat

krisdahl commented 1 year ago

+1

woj-tek commented 1 year ago

Hmm... I'm on a latest Kuma and I see Twilio option so it seems it's already there?

However, I would like to add/ask option to log-in via API Key instead of using master AccountSID and AuthToken. See https://www.twilio.com/docs/usage/requests-to-twilio#creating-or-updating-resources-with-the-post-method

Basically one can either use const client = require('twilio')(accountSid, authToken); or const client = require('twilio')(apiKey, apiSecret, { accountSid: accountSid });, where the second one is more secure and thus recomended.

CommanderStorm commented 1 year ago

@woj-tek Any PR adding such functionality is likely to be added to master.

needed change (using curl, as imo this is more readable): image

CommanderStorm commented 1 year ago

@SecureCPU given that this is in uptime kuma since #2980 (this issue was a duplicate => was missed in the autocloing), please close this issue 😅 image

woj-tek commented 1 year ago

@CommanderStorm considering this issue got closed I went with followup: https://github.com/louislam/uptime-kuma/issues/3204