Codeigniter: Changer le mot de passe pour l'utilisateur connecté et pour le mot de passe qui utilise md5

Question

Je veux demander comment changer le mot de passe pour l'utilisateur connecté, Je peux changer le mot de passe quand j'entre le mot de passe qui correspond à n'importe qui dans la base de données. Pour exemple, l'utilisateur a "admin" mot de passe, je viens d'entrer le mot de passe actuel, nouveau mot de passe et confirmer le mot de passe.

Mot de passe actuel: admin nouveau mot de passe: Mot de passe NewAdmin actuel: nouvelle administration

Et aussi je ne sais pas comment changer le mot de passe si le mot de passe utilise md5(). J'espère que vous pouvez m'aider, je suis un débutant sur Codeigniter. Je recherche des réponses, mais je ne comprends vraiment pas, donc je veux commenter, mais il est nécessaire 50 réputation donc je poster une nouvelle question.

Voici mon code:

Contrôleur

public function update(){ 
    $this->form_validation->set_rules('password', 'Current Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 
    $this->form_validation->set_rules('newpass', 'New Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 
    $this->form_validation->set_rules('confpassword', 'Confirm Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 

    if($this->form_validation->run()){ 
     $cur_password = $this->input->post('password'); 
     $new_password = $this->input->post('newpass'); 
     $conf_password = $this->input->post('confpassword'); 
     $this->load->model('queries'); 
     $userid = '1'; 
     $passwd = $this->queries->getCurrPassword($userid); 
     if($passwd->password == $cur_password){ 
      if($new_password == $conf_password){ 
       if($this->queries->updatePassword($new_password, $userid)){ 
        echo 'Password updated successfully'; 
       } 
       else{ 
        echo 'Failed to update password'; 
       } 
      } 
      else{ 
       echo 'New password & Confirm password is not matching'; 
      } 
     } 
     else{ 
      echo'Sorry! Current password is not matching'; 

    } 
} 
else{ 
    echo validation_errors(); 
} 

modèle

public function getCurrPassword($userid){ 
    $query = $this->db->where(['id'=>$userid]) 
        ->get('users'); 
    if($query->num_rows() > 0){ 
     return $query->row(); 
    } } 

    public function updatePassword($new_password, $userid){ 
    $data = array(
     'password'=> $new_password 
    ); 
     return $this->db->where('id', $userid) 
         ->update('users', $data); } 

Answer 1

J'ai trouvé une solution à mon problème.

Pour l'utilisateur connecté Je viens de changer le $ userid = '1'; en $ userid = $ this-> session-> userdata ('account_id');

Et pour le mot de passe md5 Je viens d'ajouter md5 sur passwords.Like ce que @sintakonte a fait et @zaph est exact. "N'utilisez que des algorithmes de hachage à mots de passe forts comme BCrypt, qui est utilisé dans les fonctions de hachage par mot de passe de PHP."

Référence: https://www.codeigniter.com/userguide3/general/security.html

Merci pour l'aide les gars!

Answer 2

Je ne vais pas discuter du sujet md5, mais vous devriez éviter les faibles algorithmes parce qu'ils ne sont pas sûrs. Utilisez password_verify et password_hash à cette fin. (Mais comme je l'ai dit je ne suis pas un missionnaire)

vous devez organiser votre code un peu mieux - parce que c'est un gâchis;)

essayez ce qui suit - votre contrôleur

public function update() 
{ 
    $this->form_validation->set_rules('password', 'Current Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 
    $this->form_validation->set_rules('newpass', 'New Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 
    $this->form_validation->set_rules('confpassword', 'Confirm Password', 'required|alpha_numeric|min_length[6]|max_length[20]'); 

    if($this->form_validation->run()) 
    { 
     $cur_password = $this->input->post('password'); 
     $new_password = $this->input->post('newpass'); 
     $conf_password = $this->input->post('confpassword'); 
     $this->load->model('queries'); 
     $userid = '1'; 

     try 
     { 
      $objUser = $this->queries->getUser($userid); 
      if ($objUser->password != md5($cur_password)) throw new Exception('Sorry! Current password is not matching'); 
      if ($new_password != $conf_password) throw new Exception('New password & Confirm password is not matching'); 
      $this->queries->updatePassword($new_password, $userid); 
      echo 'Password updated successfully'; 

     } 
     catch (Exception $e) 
     { 
      echo $e->getMessage(); 
     } 
    } 
    else 
    { 
     echo validation_errors(); 
    } 
} 

et votre modèle

public function getUser($userid) 
{ 
    $query = $this->db->where(id,$userid])->get('users'); 
    if($query->num_rows() == 1) 
    { 
     return $query->row(); 
    } 
    throw new Exception("no user data found"); 
} 

public function updatePassword($new_password, $userid) 
{ 
    $data = array 
    (
     'password'=> md5($new_password) 
    ); 
    if (!$this->db->where('id', $userid)->update('users', $data)) 
    { 
     throw new Exception('Failed to update password'); 
    } 
} 

il n'y a pas besoin de nommer votre modèle foncti sur getCurrPassword s'il retourne en fait un objet utilisateur - donc je l'ai renommé.