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

Commit 2984358

Browse files
committed
Remove sensitive data
1 parent 7b000cc commit 2984358

File tree

3 files changed

+21
-1
lines changed

3 files changed

+21
-1
lines changed

lib/controllers/users/load.js

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
/**
2+
* This is the act you should call when the result is exposed to the front-end
3+
*/
4+
5+
var _ = require('lodash');
6+
/**
7+
* Load a sys_user without any sensitive data
8+
* @param {String} id user Id
9+
*/
10+
function cmd_load (args, done) {
11+
var seneca = this;
12+
var ENTITY_NS = 'sys/user';
13+
var sensitiveData = ['lmsId', 'pass', 'salt'];
14+
seneca.make(ENTITY_NS).load$(args.id, function (err, user) {
15+
return done(err, _.omit(user, sensitiveData));
16+
});
17+
}
18+
19+
module.exports = cmd_load;

lib/profiles/user-profile-data.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ function cmd_user_profile_data (args, done) {
170170
}
171171

172172
function getUser (profile, done) {
173-
seneca.act({role: 'cd-users', cmd: 'load', id: query.userId, user: args.user}, function (err, user) {
173+
seneca.act({role: 'cd-users', ctrl: 'user', cmd: 'load', id: query.userId}, function (err, user) {
174174
if (err) return done(err);
175175
profile.user = user;
176176
return done(null, profile);

users.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ module.exports = function (options) {
3737
seneca.add({role: plugin, cmd: 'kpi_number_of_youth_females_registered'}, cmd_kpi_number_of_youth_females_registered);
3838
seneca.add({role: plugin, cmd: 'is_self'}, require('./lib/users/is-self'));
3939
seneca.add({role: plugin, cmd: 'is_parent_of'}, require('./lib/users/is-parent-of'));
40+
seneca.add({role: plugin, ctrl: 'user', cmd: 'load'}, require('./lib/controllers/users/load'));
4041
// LMS Integration
4142
seneca.add({role: plugin, cmd: 'get_lms_link'}, require('./lib/users/lms/get-lms-link'));
4243
seneca.add({role: plugin, cmd: 'update_lms_user'}, require('./lib/users/lms/update-user'));

0 commit comments

Comments
 (0)