Skip to content

Commit 34f35bc

Browse files
committed
fix setup validation
1 parent 866cbb0 commit 34f35bc

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

controllers/user.js

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@ var path = require('path'),
22
db = require('../models/webideModel'),
33
scripts_helper = require('../helpers/scripts_helper'),
44
config = require('../config/config'),
5-
check = require('validator').check,
6-
sanitize = require('validator').sanitize;
5+
sanitize = require('validator');
76

87
// Instructional page that displays the setup steps
98
exports.setup = function(req, res) {
@@ -30,9 +29,9 @@ exports.submit_setup = function(req, res) {
3029
}
3130

3231
try {
33-
name = sanitize(req.body.name).trim();
34-
email = sanitize(req.body.email).trim();
35-
check(email).isEmail();
32+
name = sanitize.trim(req.body.name);
33+
email = sanitize.trim(req.body.email);
34+
sanitize.isEmail(email);
3635
} catch (e) {
3736
req.session.message = e.message;
3837
console.log(e.message);
@@ -71,10 +70,10 @@ exports.submit_config = function(req, res) {
7170
req.session.message = undefined;
7271

7372
try {
74-
hostname = sanitize(req.body.hostname).trim();
75-
wifi_ssid = sanitize(req.body.wifi_ssid).trim();
76-
wifi_password = sanitize(req.body.wifi_password).trim();
77-
port = sanitize(req.body.port).trim();
73+
hostname = sanitize.trim(req.body.hostname);
74+
wifi_ssid = sanitize.trim(req.body.wifi_ssid);
75+
wifi_password = sanitize.trim(req.body.wifi_password);
76+
port = sanitize.trim(req.body.port);
7877
if (hostname) {
7978
check(hostname).len(3, 25);
8079
}

0 commit comments

Comments
 (0)