diff --git a/website/server/libs/email.js b/website/server/libs/email.js index 27cf2b62b5..23a5c17480 100644 --- a/website/server/libs/email.js +++ b/website/server/libs/email.js @@ -149,7 +149,10 @@ export async function sendTxn (mailingInfoArray, emailType, variables, personalV backoff: { delay: 10 * 60 * 1000, type: 'fixed' }, }, }, - }).json().catch(err => logger.error(err, 'Error while sending an email.')); + }).json().catch(err => logger.error(err, { + extraMessage: 'Error while sending an email.', + emailType, + })); } return null; diff --git a/website/server/libs/webhook.js b/website/server/libs/webhook.js index 738af5b71b..c7d201e829 100644 --- a/website/server/libs/webhook.js +++ b/website/server/libs/webhook.js @@ -19,7 +19,11 @@ function sendWebhook (webhook, body, user) { // Not calling .json() to parse the response because we simply ignore it }).catch(webhookErr => { // Log the error - logger.error(webhookErr, 'Error while sending a webhook request.'); + logger.error(webhookErr, { + extraMessage: 'Error while sending a webhook request.', + userId: user._id, + webhook, + }); let _failuresReset = false; diff --git a/website/server/models/webhook.js b/website/server/models/webhook.js index f9a0f62dc7..e9d4ec12e6 100644 --- a/website/server/models/webhook.js +++ b/website/server/models/webhook.js @@ -58,7 +58,7 @@ export const schema = new Schema({ required: true, validate: [v => validator.isURL(v, { require_tld: !!IS_PRODUCTION, // eslint-disable-line camelcase - require_protocol: true, // TODO migrate existing ones + require_protocol: true, protocols: ['http', 'https'], }), shared.i18n.t('invalidUrl')], },