Skip to content
This repository was archived by the owner on Aug 30, 2021. It is now read-only.

Commit 2b0ae86

Browse files
committed
fix(security): fixing possible xss issue in parsed objects
Fixes #1106
1 parent e6a8f0f commit 2b0ae86

File tree

1 file changed

+19
-1
lines changed

1 file changed

+19
-1
lines changed

modules/core/server/controllers/core.server.controller.js

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,29 @@
11
'use strict';
22

3+
var validator = require('validator');
4+
35
/**
46
* Render the main application page
57
*/
68
exports.renderIndex = function (req, res) {
9+
10+
var safeUserObject = null;
11+
if (req.user) {
12+
safeUserObject = {
13+
displayName: validator.escape(req.user.displayName),
14+
provider: validator.escape(req.user.provider),
15+
username: validator.escape(req.user.username),
16+
created: req.user.created.toString(),
17+
roles: req.user.roles,
18+
profileImageURL: validator.escape(req.user.profileImageURL),
19+
email: validator.escape(req.user.email),
20+
lastName: validator.escape(req.user.lastName),
21+
firstName: validator.escape(req.user.firstName)
22+
};
23+
}
24+
725
res.render('modules/core/server/views/index', {
8-
user: req.user || null
26+
user: safeUserObject
927
});
1028
};
1129

0 commit comments

Comments
 (0)