set alarm on watch failures

elixir-toniq / vapor

Runtime configuration system for Elixir

MIT License

596 stars 37 forks source link

set alarm on watch failures #35

Closed jeffweiss closed 5 years ago

jeffweiss commented 5 years ago

Prior to this commit we had no way of notifying other portions of the system when we have a problem reading a configuration source. This commit uses the sasl alarm_handler to set an alarm of the following format {:vapor, {plan_layer, plan_provider}}. We also redact the alarm reason because passing through lower-level reasons from their error tuple could expose secrets to logging.

Closes #11

keathley commented 5 years ago

I've rebased this branch against master and merged it in there.