PHP login, is it secure? -


i hoping take @ first php login script , give me constructive criticism on may of done wrong , if secure. thank you.

i wasn't sure if had used password rehash correctly.

if (isset($_post['submit'], $_post['username'], $_post['password'])) { $username = null; if (isset($_post['username'])) $username = strip_tags(trim($_post['username'])); $password = null; if (isset($_post['password'])) $password = strip_tags(trim($_post['password']));  $sql = "select * login username=?";  $get = $connect->prepare($sql);  $get->execute(array(     $username )); // execute query if ($get->rowcount() === 1)     {     $row = $get->fetch(pdo::fetch_assoc); // fetch result     $db_username = $row['username'];     $db_password = $row['password'];     if ((password_verify($password, $db_password)) && (strlen($username) >= 5) && (strlen($username) <= 10) && (strlen($password) >= 5) && (strlen($password) <= 12))         {          if (password_needs_rehash($password, password_default))             {             $hash = password_hash($password, password_default);             $sql = "select * login username=?";              $get = $connect->prepare($sql); // use prepare prevent sql injection             $sql = "update login set password=? username=?";             $statement = $connect->prepare($sql);             $statement->execute(array(                 $hash,                 $username             ));             }          $_session['auth'] = $db_username;         session_regenerate_id(true);          $sql = "update login set last_login=?, ip=? username=?";         $statement = $connect->prepare($sql);         $statement->execute(array(             $dt,             $ip,             $username             ));      $sql2 = "insert log (username,lastlogin,ip) values (:username,:lastlogin,:ip)";         $statement = $connect->prepare($sql2);         $statement->execute(array(':username'=>$username,                               ':lastlogin'=>$dt,                               ':ip'=>$ip                               ));                       reloadpage();          }        else         {         $loginmsg = 'wrong username / password';         }     }   else     {     $loginmsg = 'wrong username / password';     } }

no. 1 flaw

$username = null; if (isset($_post['username'])) $username = strip_tags(trim($_post['username'])); $password = null; if (isset($_post['password'])) $password = strip_tags(trim($_post['password']));

try 

$username = null; if (isset($_post['username'])) {      $username = strip_tags(trim($_post['username'])); } $password = null; if (isset($_post['password'])) {      $password = strip_tags(trim($_post['password'])); }

no. 2 flaw

always use algo contants along password_hash() better usage. 

 if (password_needs_rehash($password, password_default))  {       $cons = array('cost' => 12);       $hash = password_hash($password, password_default, $cons);       $sql = "select * login username=?";

all others seems ok me.


Comments

Post a Comment

Popular posts from this blog

c# - Validate object ID from GET to POST -

i have method allows user access comment have made on site. have post method lets them update comment. my normal solution send entire comment model through, let them update it, update on database when post back. involve sending commentid through in hiddenfor , can manipulated. how can verify commentid sent in get method same i'm getting in post , not able alter comment wish? how can verify commentid sent in method same i'm getting in post? basically have validate following things in post - user logged-in user. authenticated users post comments. commentid in post , should valid commentid , should present in database. userid associated logged-in user should same userid associated comment . comment should contain userid column, can check @ time of update. to make sure update happens comment has been sent in get - hold commentid in session , in post action compare commentid value in session .
Read more

node.js - Custom Model Validator SailsJS -

when using custom model validator checks record within model. return within spread doesn't seem end control flow. //post.js model /** * custom validator **/ types: { isuservalid: function(user_id) { var promise = require('bluebird'); promise.all([ user.findone({id: user_id}) ]) .spread(function(user) { console.log(user); if (user === null || user === undefined) { console.log('failed'); return false; }else{ console.log('passed'); return true; } }); } }, my response standard validation failed response. { "error": "e_validation", "status": 400, "summary": "1 attribute invalid", "model": "post", "invalidattributes": { "owner": [ { "rule": "isuservalid", ...
Read more

php - Find a regex to take part of Email -

i have following emails need extract part from: bestellnummer xxx: 1 von xxx, 1er pack ------------- anfang der nachricht ------------- foo bar baz foo bar baz // <<<<< need text here ------------- ende der nachricht ------------- ------------- anfang der nachricht ------------- foo bar baz foo bar baz ------------- ende der nachricht ------------- there 0 unlimited occurences of ------------- anfang der nachricht ------------- ------------- ende der nachricht ------------- and i'm able extract first part regex: $re = "/------------- .*? -------------.?(.*?).?------------- .*? -------------/s"; but, i'm quite new on learning regex, i'm pretty sure there must better regex extract part (foo bar baz foo bar baz) of text between ------------- anfang der nachricht ------------- ------------- ende der nachricht ------------- as can in different languages, i'm using .? to match between hyphens. i...
Read more