mailer: make oom mail contain link to dashboard instead of docs
This commit is contained in:
+16
-6
@@ -33,8 +33,7 @@ var assert = require('assert'),
|
||||
settings = require('./settings.js'),
|
||||
showdown = require('showdown'),
|
||||
translation = require('./translation.js'),
|
||||
smtpTransport = require('nodemailer-smtp-transport'),
|
||||
util = require('util');
|
||||
smtpTransport = require('nodemailer-smtp-transport');
|
||||
|
||||
var NOOP_CALLBACK = function (error) { if (error) debug(error); };
|
||||
|
||||
@@ -386,19 +385,30 @@ function boxUpdateError(mailTo, message) {
|
||||
});
|
||||
}
|
||||
|
||||
function oomEvent(mailTo, program, event) {
|
||||
function oomEvent(mailTo, app, addon, containerId, event) {
|
||||
assert.strictEqual(typeof mailTo, 'string');
|
||||
assert.strictEqual(typeof program, 'string');
|
||||
assert.strictEqual(typeof app, 'object');
|
||||
assert.strictEqual(typeof addon, 'object');
|
||||
assert.strictEqual(typeof containerId, 'string');
|
||||
assert.strictEqual(typeof event, 'object');
|
||||
|
||||
getMailConfig(function (error, mailConfig) {
|
||||
if (error) return debug('Error getting mail details:', error);
|
||||
|
||||
const templateData = {
|
||||
webadminUrl: settings.adminOrigin(),
|
||||
cloudronName: mailConfig.cloudronName,
|
||||
app,
|
||||
addon,
|
||||
event: JSON.stringify(event),
|
||||
format: 'text'
|
||||
};
|
||||
|
||||
var mailOptions = {
|
||||
from: mailConfig.notificationFrom,
|
||||
to: mailTo,
|
||||
subject: `[${mailConfig.cloudronName}] ${program} was restarted (OOM)`,
|
||||
text: render('oom_event.ejs', { cloudronName: mailConfig.cloudronName, program: program, event: JSON.stringify(event), format: 'text' })
|
||||
subject: `[${mailConfig.cloudronName}] ${app ? app.fqdn : addon.name} was restarted (OOM)`,
|
||||
text: render('oom_event.ejs', templateData)
|
||||
};
|
||||
|
||||
sendMail(mailOptions);
|
||||
|
||||
Reference in New Issue
Block a user