Add support for additional notification methods

lrstanley / vault-unseal

auto-unseal utility for Hashicorp Vault

MIT License

230 stars 30 forks source link

Add support for additional notification methods #3

Open lrstanley opened 2 years ago

lrstanley commented 2 years ago

Options:

https://github.com/containrrr/shoutrrr (easier to configure, just allow users to pass in url's via cli or env)
https://github.com/nikoksr/notify

This would likely be a breaking change for existing email support.

zamazan4ik commented 1 year ago

@lrstanley Could you please specify somewhere in the documentation (in the README file, I think) the supported notification mechanisms right now? For now, it's unclear which exact notification ways could be used.

As far as I understand, mail notification is the only supported mechanism, am I right?

lrstanley commented 1 year ago

@zamazan4ik email is the only supported option, that is correct. I will be taking a look at implementing more options (and deprecating how mail notifications currently work) very soon, so I will likely wait until I have the new solution in place before updating the readme.

c33s commented 8 months ago

@lrstanley a webhook notification and/or a shell script notification would be best, having a webhook practically allows to do nearly everything. if it is to complicated to implement a full webhook support (headers, POST, GET), simply calling a custom shell script would be enough. having a shellscript as handler allows admins to really allow every form of notification.