From ea541a9bb95884876fc2671c0f3c8203d4813ba2 Mon Sep 17 00:00:00 2001 From: Vytor Calixto <vytorcalixto@gmail.com> Date: Fri, 29 Sep 2017 11:02:06 -0300 Subject: [PATCH] Remove unused log messages --- src/libs/middlewares/oauth2.js | 10 +++------- src/libs/routes/class.js | 1 - src/libs/routes/enrollment.js | 1 - src/libs/routes/idhm.js | 2 -- src/libs/routes/idhme.js | 2 -- src/libs/routes/idhml.js | 2 -- src/libs/routes/idhmr.js | 2 -- src/libs/routes/pibpercapita.js | 1 - src/libs/routes/region.js | 1 - src/libs/routes/resetToken.js | 4 ---- src/libs/routes/school.js | 1 - src/libs/routes/teacher.js | 1 - src/libs/routes/user.js | 6 +++--- src/libs/routes/verifyToken.js | 2 -- 14 files changed, 6 insertions(+), 30 deletions(-) diff --git a/src/libs/middlewares/oauth2.js b/src/libs/middlewares/oauth2.js index d0f4a9cc..c8d47ac5 100644 --- a/src/libs/middlewares/oauth2.js +++ b/src/libs/middlewares/oauth2.js @@ -47,10 +47,10 @@ let generateTokens = (data, done) => { token.save((err) => { if (err) { - console.log('erro aqui?'); log.error(err); return done(err); } + log.info(`Criado token ${tokenValue}`); done(null, tokenValue, refreshTokenValue, { 'expires_in': config.security.tokenLife }); @@ -63,12 +63,8 @@ aserver.exchange(oauth2orize.exchange.password((client, username, password, scop if (err) { return done(err); } - console.log('aqui!'); + if (!user || !user.checkPassword(password)) { - console.log('deu ruim'); - console.log(user); - console.log(password); - if(user) console.log(user.checkPassword(password)); return done(null, false); } @@ -76,7 +72,7 @@ aserver.exchange(oauth2orize.exchange.password((client, username, password, scop userId: user._id, clientId: client._id }; - console.log('gerando token'); + log.info(`Gerando token para usuário ${user.name}`); generateTokens(model, done); }) diff --git a/src/libs/routes/class.js b/src/libs/routes/class.js index 4527640d..72d60a80 100644 --- a/src/libs/routes/class.js +++ b/src/libs/routes/class.js @@ -378,7 +378,6 @@ classApp.get('/download', (req, res, next) => { log.error(err); return res.json({error: err}); } - console.log(body); res.json({msg: 'Wait for download email'}); }); }); diff --git a/src/libs/routes/enrollment.js b/src/libs/routes/enrollment.js index 292a9adb..74d29d32 100644 --- a/src/libs/routes/enrollment.js +++ b/src/libs/routes/enrollment.js @@ -515,7 +515,6 @@ next(); log.error(err); return res.json({error: err}); } - console.log(body); res.json({msg: 'Wait for download email'}); }); }); diff --git a/src/libs/routes/idhm.js b/src/libs/routes/idhm.js index d8d013fa..7d6d97de 100644 --- a/src/libs/routes/idhm.js +++ b/src/libs/routes/idhm.js @@ -38,8 +38,6 @@ idhmApp.get('/year_range', (req, res, next) => { req.old_result = req.result; next(); }, query, (req, res, next) => { - // console.log(req.old_result[0].start_year); - // console.log(req.result[0].start_year); if (req.old_result[0].start_year < req.result[0].start_year) { req.result[0].start_year = req.old_result[0].start_year; } diff --git a/src/libs/routes/idhme.js b/src/libs/routes/idhme.js index e061a2a1..d65318bd 100644 --- a/src/libs/routes/idhme.js +++ b/src/libs/routes/idhme.js @@ -36,8 +36,6 @@ idhmeApp.get('/year_range', (req, res, next) => { req.old_result = req.result; next(); }, query, (req, res, next) => { - // console.log(req.old_result[0].start_year); - // console.log(req.result[0].start_year); if (req.old_result[0].start_year < req.result[0].start_year) { req.result[0].start_year = req.old_result[0].start_year; } diff --git a/src/libs/routes/idhml.js b/src/libs/routes/idhml.js index dba2ccc9..08c71630 100644 --- a/src/libs/routes/idhml.js +++ b/src/libs/routes/idhml.js @@ -36,8 +36,6 @@ idhmlApp.get('/year_range', (req, res, next) => { req.old_result = req.result; next(); }, query, (req, res, next) => { - // console.log(req.old_result[0].start_year); - // console.log(req.result[0].start_year); if (req.old_result[0].start_year < req.result[0].start_year) { req.result[0].start_year = req.old_result[0].start_year; } diff --git a/src/libs/routes/idhmr.js b/src/libs/routes/idhmr.js index 9a8bcf7b..4bd16e91 100644 --- a/src/libs/routes/idhmr.js +++ b/src/libs/routes/idhmr.js @@ -38,8 +38,6 @@ idhmrApp.get('/year_range', (req, res, next) => { req.old_result = req.result; next(); }, query, (req, res, next) => { - // console.log(req.old_result[0].start_year); - // console.log(req.result[0].start_year); if (req.old_result[0].start_year < req.result[0].start_year) { req.result[0].start_year = req.old_result[0].start_year; } diff --git a/src/libs/routes/pibpercapita.js b/src/libs/routes/pibpercapita.js index 83d914a3..815f439d 100644 --- a/src/libs/routes/pibpercapita.js +++ b/src/libs/routes/pibpercapita.js @@ -155,7 +155,6 @@ pibpercapitaApp.get('/', rqf.parse(), rqf.build(), (req, res, next) => { next(); }, query, id2str.transform(false), (req, res, next) => { - console.log(req.result); req.result.forEach((i) => { let value = i.total; let res = value.toString().split("."); diff --git a/src/libs/routes/region.js b/src/libs/routes/region.js index 81f655c3..fdffe00f 100644 --- a/src/libs/routes/region.js +++ b/src/libs/routes/region.js @@ -56,7 +56,6 @@ regionApp.get('/', rqf.parse(), (req, res, next) => { req.sql.from('regiao') .field('id') .field('nome', 'name'); - console.log(req.sql.toString()); next(); }, rqf.build(), query, response('region')); diff --git a/src/libs/routes/resetToken.js b/src/libs/routes/resetToken.js index ef0029c1..34ece845 100644 --- a/src/libs/routes/resetToken.js +++ b/src/libs/routes/resetToken.js @@ -12,7 +12,6 @@ const User = require(`${libs}/models/user`); resetTokenApp.get('/:token', (req, res, next) => { let token = req.params.token; - console.log('verificando token'); ResetToken.findOne({token: token}, (err, rToken) => { if(err) { log.error(err); @@ -20,7 +19,6 @@ resetTokenApp.get('/:token', (req, res, next) => { } if(!rToken) { // TODO: generate new reset token - console.log('Token não existe'); res.statusCode = 404; return next({msg: 'Token not found', status:404}); } @@ -48,14 +46,12 @@ resetTokenApp.get('/:token', (req, res, next) => { }); resetTokenApp.post('/:token', (req, res, next) => { let token = req.params.token; - console.log('verificando token'); ResetToken.findOne({token: token}, (err, rToken) => { if(err) { log.error(err); return next(err); } if(!rToken) { - console.log('Token não existe'); res.statusCode = 404; return next({msg: 'Token not found', status:404}); } diff --git a/src/libs/routes/school.js b/src/libs/routes/school.js index 4514e33d..a9656f2d 100644 --- a/src/libs/routes/school.js +++ b/src/libs/routes/school.js @@ -505,7 +505,6 @@ schoolApp.get('/count/download', (req, res, next) => { log.error(err); return res.json({error: err}); } - console.log(body); res.json({msg: 'Wait for download email'}); }); }); diff --git a/src/libs/routes/teacher.js b/src/libs/routes/teacher.js index a652191a..69b69b01 100644 --- a/src/libs/routes/teacher.js +++ b/src/libs/routes/teacher.js @@ -425,7 +425,6 @@ teacherApp.get('/download', (req, res, next) => { log.error(err); return res.json({error: err}); } - console.log(body); res.json({msg: 'Wait for download email'}); }); }); diff --git a/src/libs/routes/user.js b/src/libs/routes/user.js index 4cc5221a..4913b780 100644 --- a/src/libs/routes/user.js +++ b/src/libs/routes/user.js @@ -144,7 +144,7 @@ userApp.post('/', (req, res, next) => { for(let errName in err.errors) { errors.push(err.errors[errName].message); } - console.log(errors); + log.error(errors); res.statusCode = 400; return res.json({err, errors}); } @@ -167,13 +167,13 @@ userApp.post('/', (req, res, next) => { subject: "Confirme seu cadastro - Laboratório de Dados Educacionais", text } - console.log(mailOptions); email(mailOptions, (err, info) => { if(err) { log.error(err); res.json({msg: 'User created'}); } - log.debug(`Message ${info.messageId} sent: ${info.response}`); + log.info(`Message ${info.messageId} sent: ${info.response}`); + log.info(`Usuário ${user.email} foi criado`); res.json({msg: 'User created'}); }); }); diff --git a/src/libs/routes/verifyToken.js b/src/libs/routes/verifyToken.js index 425c3b6e..d54f64aa 100644 --- a/src/libs/routes/verifyToken.js +++ b/src/libs/routes/verifyToken.js @@ -12,7 +12,6 @@ const User = require(`${libs}/models/user`); verifyTokenApp.get('/:token', (req, res, next) => { let token = req.params.token; - console.log('verificando token'); VerificationToken.findOne({token: token}, (err, vToken) => { if(err) { log.error(err); @@ -20,7 +19,6 @@ verifyTokenApp.get('/:token', (req, res, next) => { } if(!vToken) { // TODO: generate new verification token - console.log('Token não existe'); res.statusCode = 404; return next({msg: 'Token not found', status:404}); } -- GitLab