Skip to content
Snippets Groups Projects
Commit 8f551e77 authored by Niko Ehrenfeuchter's avatar Niko Ehrenfeuchter :keyboard:
Browse files

Remember config validation warnings and report to the admin

Fixes #51
parent ef3de89b
No related branches found
No related tags found
No related merge requests found
...@@ -288,6 +288,9 @@ namespace ATxCommon.Serializables ...@@ -288,6 +288,9 @@ namespace ATxCommon.Serializables
} }
} }
[XmlIgnore]
public static string ValidatorWarnings { get; set; }
#endregion #endregion
...@@ -391,8 +394,10 @@ namespace ATxCommon.Serializables ...@@ -391,8 +394,10 @@ namespace ATxCommon.Serializables
SubOptimal(value.ToString(), name, "value is set very high, please check!"); SubOptimal(value.ToString(), name, "value is set very high, please check!");
} }
void SubOptimal(string value, string name, string msg) { void SubOptimal(string value, string name, string message) {
Log.Warn(">>> Sub-optimal setting detected: <{0}> [{1}] {2}", name, value, msg); var msg = $">>> Sub-optimal setting detected: <{name}> [{value}] {message}";
ValidatorWarnings += msg + "\n";
Log.Warn(msg);
} }
void LogAndThrow(string msg) { void LogAndThrow(string msg) {
......
...@@ -415,6 +415,12 @@ namespace ATxService ...@@ -415,6 +415,12 @@ namespace ATxService
} }
Log.Debug(msg); Log.Debug(msg);
// finally check if the validation gave warnings and send them to the admin:
var warnings = ServiceConfig.ValidatorWarnings;
if (string.IsNullOrWhiteSpace(warnings))
return;
SendAdminEmail(warnings);
} }
#endregion #endregion
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment