uptime-kuma
uptime-kuma copied to clipboard
louislam
Twillo SMS Alerts
⚠️ Please verify that this feature request has NOT been suggested before.
- [X] I checked and didn't find similar feature request
🏷️ 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
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.
@woj-tek Any PR adding such functionality is likely to be added to master.
- please also go into detail what you mean by "more secure and thus recommended"
- The problem with the notification providers is, that they are for the most part untested/untestable. => Screenshots are needed to verify that this change works.
needed change (using curl, as imo this is more readable):
@SecureCPU given that this is in uptime kuma since #2980 (this issue was a duplicate => was missed in the autocloing), please close this issue 😅
@CommanderStorm considering this issue got closed I went with followup: https://github.com/louislam/uptime-kuma/issues/3204