node.js - expressjs req and res object parameter about safe -


i use express write app, want implement variables current_user in global, ensure whether user logined.so, add middleware

app.use(require('./controller/user').auth_user);

in user.js, define method called auth_user use res.locals set global local current_user. , set req.session.user=user.

exports.auth_user = function(req, res, next) {   if (req.session.user) {     res.locals('current_user', req.session.user);     return next();   } else {   var cookie = req.cookies['user_cookie'];   if (!cookie){     res.locals('current_user', "");     return next();   };   var auth_token = decrypt(cookie, 'user_session');   var auth = auth_token.split('\t');   var user_email=auth[3].tolowercase();   user.findone({'email': user_email},function(err, result) {     if(err) return next(err);     if (result) {       req.session.user = result;       res.locals('current_user', req.session.user);       return next();     }else{       return next();     }   });   } };

so, put user object req, , when debug, found can user.hashed_password req.i think it's not safe. can user in browser side info?

as long not expose information anywhere else, safe. @ least perspective browser not able access req object.

anyway, question whether it's best idea attach complete user object req object if need specific part of it. why not attach stripped down version of object not include hash?

then issue gone anyway, no matter whether browser hypothetically able access or not.


Comments

Post a Comment

Popular posts from this blog

Why does Ruby on Rails generate add a blank line to the end of a file? -

running rails generate controller foo home results in: class foocontroller < application controller def home end end (there's nothing on line in actual file, blank line) is there purpose blank line? it's common use newline indicate end of line, last line. many editors (like vi, use) add in newline silently. consensus text files (especially in unix world) should end in newline, , there have been problems historically if wasn't present. why should text files end newline? the tool used count lines in file "wc" counts newlines in file, without trailing newline show 3 instead of 4. it improves readability of templates used in generator. consider: https://github.com/rails/rails/blob/master/railties/lib/rails/generators/rails/controller/templates/controller.rb to remove trailing newline, template have last line of: end<% end -%> instead of: end <% end -%> that seems less readable me. the discussion below refers ...
Read more

keyboard - Smiles and long press feature in Android -

i've created android keyboard. how add features inserting smilies, , using long keypresses display symbols? to create long-press behavior allows select several options pop-up menu, follow guide (specifically pop-up menu section): http://developer.android.com/guide/topics/ui/menus.html smileys (also known emoticons) combination of keys, understood programs , parsed same way. have do, when clicks smiley face on keyboard, simulate pressing of key combination matching emoticon. example, when clicks smiley, should simulate key presses ':' , ')' :). here's list of emoticons: http://en.wikipedia.org/wiki/list_of_emoticons
Read more

node.js - Bad Request - node js ajax post -

i'm using express 3.0 , node v0.11.0. have button submits form , i'm using ajax post data json object node server. client code is: $('#contact').submit(function(e) { console.log('submit called'); var formdata = json.stringify($('form').serializeobject()); console.log(formdata); $.ajax({ url: "http://localhost:3000/savedata/", type: "post", datatype: 'json', data: {objectdata: formdata}, contenttype: "application/json", complete: function() { console.log('process complete'); }, success: function(data) { console.log('process sucess'); }, error: function() { console.log('process error'); }, }); return false; }); then on server: var express = require('express'); var app = express(); app.set('views', __dirname + '/views'); app.set('view engine', 'jade'); app.use(express.bodyparser());...
Read more