Monitors with failed configurations could log credentials
nkringle opened this issue · 2 comments
nkringle commented
https://github.com/marcells/node-build-monitor/blob/master/app/monitor.js#L176
As services include api credentials in their spec, when a configuration fails to load, the log statement with the configuration should either remove credentials, or have an option to disable logging for this line.
marcells commented
Good idea. I'll add a new flag to the configuration.