ログインシステム - Node.jsとRedis

Question

node.jsとRedis（Redisパッケージ）を使用してログインシステムを作成したいが、入力データとデータベースデータを比較するたびに、混乱する結果になる。これが最初の方法です：

私はログイン名とパスワードはOKですプラスそれは 第二の例をたerrMsg機能を実行しない場合であっても、偽得続ける

const Redis = require("redis"); 
const client = Redis.createClient(); 
exports.userLogin = (username, password, errMsg, successMsg, res) => { 
    client.keys("user*", (err, replies) => { 
    let isFound = false; 
    for (let i = 0; i < replies.length; i++) { 
     client.hgetall(replies[i], (err, value) => { 
     if (value.user === username && value.password === password) { 
      isFound = true; 
     } 
     }); 
    } 
    console.log(isFound); 
    if (isFound) { 
     successMsg(username, res); 
    } else { 
     errMsg(res); 
    } 
    }) 
)} 

：

exports.userLogin = (username, password, errMsg, successMsg, res) => { 
    client.keys("user*", (err, replies) => { 
     for (let i = 0; i < replies.length; i++) { 
      client.hgetall(replies[i], (err, value) => { 
      if (value.user === username && value.password === password) { 
       successMsg(username, res); 
      } else { 
       console.log("length: ", replies.length - 1); 
       console.log("i: ", i); 
       if(replies[i] === replies.length - 1) { 
       errMsg(res); 
       } else { 
       console.log("else statement: ", replies[i] === replies.length - 1); 
       return true; 
       } 
      } 
      }); 
     } 
     }) 
)} 

パスワードとログインがOKでも動作しますが、ページに何かがロードされていてerrMsg関数が実行されないため、パスワードが正しくない場合は動作しません。あなたはなにか考えはありますか？

Answer 1

可能であれば、redisを使用してユーザーを格納するのに適したものに変更します。

それ以外の場合は、データの保存方法を変更してください。あなたはすべてのユーザーを読んでそのように繰り返すことは望ましくありません。 dbクエリができるだけ世話をするようにしてください。おそらく1つの方法は、ユーザー名をキーの一部にすることで、正しいユーザーを直接検索し、そこからパスワードを確認することができます。

パスワードを保存してログインするセキュリティの部分を忘れないでください。セキュアにする必要がある場合は、パスワードをハッシュし、パスワードソルトを追加します。