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

Commit e220244

Browse files
committed
requested fixes
1 parent ffce999 commit e220244

File tree

2 files changed

+15
-20
lines changed

2 files changed

+15
-20
lines changed

lib/users/unlock-account-email.js

Lines changed: 14 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
const moment = require('moment');
2-
const options = require('../../config/config');
32

43
const protocol = process.env.PROTOCOL || 'http';
54
const zenHostname = process.env.HOSTNAME || '127.0.0.1:8000';
@@ -10,24 +9,20 @@ function unlockAccountEmail(args, done) {
109

1110
seneca.act({ role: 'cd-users', cmd: 'get_users_by_email', email }, (err, users) => {
1211
if (err) return done(err);
13-
if (options['email-notifications'].sendemail) {
14-
seneca.act({ role: 'email-notifications', cmd: 'send' }, {
15-
code: 'user-lockout-',
16-
locality: args.locality || 'en_US',
17-
to: email,
18-
subject: 'CoderDojo Zen Account Lockout',
19-
content: {
20-
name: users[0].name,
21-
resetlink: `${protocol}://${zenHostname}/reset_password`,
22-
year: moment(new Date()).format('YYYY'),
23-
},
24-
}, (err, response) => {
25-
if (err) return done(err);
26-
return done(null, { ok: true });
27-
});
28-
} else {
29-
return done(null, { ok: false });
30-
}
12+
seneca.act({ role: 'email-notifications', cmd: 'send' }, {
13+
code: 'user-lockout-',
14+
locality: args.locality || 'en_US',
15+
to: email,
16+
subject: 'CoderDojo Zen Account Lockout',
17+
content: {
18+
name: users[0].name,
19+
resetlink: `${protocol}://${zenHostname}/reset_password`,
20+
year: moment(new Date()).format('YYYY'),
21+
},
22+
}, (err, response) => {
23+
if (err) return done(err);
24+
return done(null, { ok: true });
25+
});
3126
});
3227
}
3328

service.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ require('./migrate-psql-db.js')(function (err) {
5858
logger: log.logger
5959
});
6060
seneca.use(require('./oauth2.js'), {clients: config.oauth2.clients});
61-
seneca.use('user', { failedCount: 3 });
61+
seneca.use('user', { failedLoginCount: 3 });
6262
seneca.use('auth');
6363
seneca.use(require('./users.js'),
6464
{ 'email-notifications': config['email-notifications'],

0 commit comments

Comments
 (0)