Skip to content

Commit 0942425

Browse files
authored
fix the ReferenceError: resolved_color is not defined error (#16)
* fix the 'ReferenceError: resolved_color is not defined' error * hardcode resolved_color to stablize container * remove debug lines
1 parent 3013c36 commit 0942425

File tree

3 files changed

+108
-63
lines changed

3 files changed

+108
-63
lines changed

package-lock.json

Lines changed: 100 additions & 58 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "matrix-alertmanager",
3-
"version": "0.12.0",
3+
"version": "0.12.1",
44
"description": "Prometheus Alertmanager bot for Matrix",
55
"main": "src/app.js",
66
"scripts": {

src/utils.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,12 +74,15 @@ const utils = {
7474
}
7575
})(data.labels.severity);
7676
parts.push('<summary><font color=\"' + color + '\"><b>FIRING</b>: ' + summary + env + '</font></summary>')
77+
7778
} else if (data.status === 'resolved') {
79+
80+
// determine best logging color for resolved alerts
81+
let resolved_color = "#a8fd57";
7882
if (process.env.COLOR_RECOVERED) {
79-
let resolved_color = process.env.COLOR_RECOVERED
80-
} else {
81-
let resolved_color = "#a8fd57"
82-
}
83+
let resolved_color = process.env.COLOR_RECOVERED;
84+
};
85+
8386
parts.push('<summary><font color=\"' + resolved_color + '\"><b>RESOLVED</b>: ' + summary + env + '</font></summary>')
8487
} else {
8588
parts.push('<summary>' + data.status.toUpperCase() + ': ' + summary + env + '</summary>')

0 commit comments

Comments
 (0)