Skip to content
This repository was archived by the owner on Apr 13, 2023. It is now read-only.

Commit 2877379

Browse files
authored
Merge pull request #104 from SpringRoll/release/1.9.0
Release/1.9.0
2 parents 162ff07 + 531871e commit 2877379

File tree

7 files changed

+39
-130
lines changed

7 files changed

+39
-130
lines changed

app/helpers/sendmail.js

Lines changed: 27 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,28 @@
1-
var nodemailer = require('nodemailer');
1+
const nodemailer = require('nodemailer');
22

3-
module.exports = function(mailOptions, callback)
4-
{
5-
var transporter = nodemailer.createTransport({
6-
service: 'Gmail',
7-
auth: {
8-
user: process.env.GMAIL_USER,
9-
pass: process.env.GMAIL_PASSWORD
10-
}
11-
});
12-
transporter.sendMail(mailOptions, callback);
13-
};
3+
module.exports = function(mailOptions, callback) {
4+
let transportConfig;
5+
6+
const canSendSmtp = process.env.SMTP_USER !== undefined && process.env.SMTP_PASSWORD !== undefined && process.env.SMTP_HOST !== undefined;
7+
if (canSendSmtp) {
8+
transportConfig = {
9+
host: process.env.SMTP_HOST,
10+
secure: true,
11+
auth: {
12+
user: process.env.SMTP_USER,
13+
pass: process.env.SMTP_PASSWORD
14+
}
15+
};
16+
} else {
17+
transportConfig = {
18+
service: 'Gmail',
19+
auth: {
20+
user: process.env.GMAIL_USER,
21+
pass: process.env.GMAIL_PASSWORD
22+
}
23+
};
24+
}
25+
26+
const transporter = nodemailer.createTransport(transportConfig);
27+
transporter.sendMail(mailOptions, callback);
28+
};

app/public/js/embed.js

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

app/routes/forgot.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ router.post('/', function(req, res, next)
3434
{
3535
var mailOptions = {
3636
to: user.email,
37-
from: 'passwordreset@demo.com',
37+
from: process.env.SMTP_FROM_USER || 'passwordreset@demo.com',
3838
subject: 'Node.js Password Reset',
3939
text: 'You are receiving this because you (or someone else) have requested the reset of the password for your account.\n\n' +
4040
'Please click on the following link, or paste this into your browser to complete the process:\n\n' +
@@ -64,4 +64,4 @@ router.get('/', function(req, res)
6464
});
6565
});
6666

67-
module.exports = router;
67+
module.exports = router;

app/routes/reset.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ router.post('/:token', function(req, res)
5555
{
5656
var mailOptions = {
5757
to: user.email,
58-
from: 'passwordreset@demo.com',
58+
from: process.env.SMTP_FROM_USER || 'passwordreset@demo.com',
5959
subject: 'Your password has been changed',
6060
text: 'Hello,\n\n' +
6161
'This is a confirmation that the password for your account ' + user.email + ' has just been changed.\n'
@@ -73,4 +73,4 @@ router.post('/:token', function(req, res)
7373
});
7474
});
7575

76-
module.exports = router;
76+
module.exports = router;

npm-shrinkwrap.json

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

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"version": "1.8.1",
2+
"version": "1.9.0",
33
"private": true,
44
"devDependencies": {
55
"@babel/cli": "^7.2.3",
@@ -70,7 +70,7 @@
7070
"mongoose-type-email": "^1.0.5",
7171
"mongoose-types": "^1.0.3",
7272
"mongoose-unique-validator": "^0.4.1",
73-
"nodemailer": "^1.3.2",
73+
"nodemailer": "^6.4.3",
7474
"passport": "^0.2.1",
7575
"passport-local": "^1.0.0",
7676
"semver": "^5.0.1",

src/embed/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ const start = () => {
172172
query: queryArgs.length ? '?' + queryArgs.join('&') : ''
173173
})
174174
.catch(err => {
175-
err.json().then(({ error }) => alert(error));
175+
alert(err.message);
176176
})
177177
};
178178

0 commit comments

Comments
 (0)