ログイン資格を比較してください

Question

こんにちは私はログインフォームで作業しています。私は自分のウェブサイト上のアカウントにサインアップできますが、ユーザー名とパスワードを入力するとログインできません。私は2つのコードを比較していると思われるコードをいくつか持っていますが、何らかの理由で動作していないだけです。ここに私のコードです。何らかの理由でコードの最後にwhileループを実行することさえありません。どんな助けもありがたいです

private bool CompareStrings(string string1, string string2) 
{ 
    return String.Compare(string1, string2, true, System.Globalization.CultureInfo.InvariantCulture) == 0 ? true : false; 
} 

public void LogInAccount(string UserName, string UserPassword, Label InvalidLogIn) 
{ 
    connection.ConnectionString = @"Connection String"; 
    connection.Open(); 

    string compare = "select ISNULL(UserName, '') As UserName, ISNULL(UserPassword, '') As UserPassword from SignUp where UserName= @UserName"; 

    SqlCommand CompareUser = new SqlCommand(compare, connection); 
    //SqlCommand Command2 = new SqlCommand("select * from SignUp where FirstName= @FirstName", connection); 

    //Command2.Parameters.AddWithValue("@FirsName", FirstName.Text); 

    CompareUser.Parameters.AddWithValue("@UserPassword", UserPassword); 
    CompareUser.Parameters.AddWithValue("@UserName", UserName); 

    SqlDataReader dr = CompareUser.ExecuteReader(); 

    //string User = UserName; 
    //string UserPassword = Password; 

    //HtmlAnchor LogIn = (HtmlAnchor)Master.FindControl("LogIn"); 
    //HtmlAnchor SignUp = (HtmlAnchor)Master.FindControl("SignUp"); 

    while (dr.Read()) 
    { 
     if (this.CompareStrings(dr["UserName"].ToString(), UserName) && 
      this.CompareStrings(dr["UserPassword"].ToString(), UserPassword)) 
     { 
      InvalidLogIn.Visible = false; 
      FormsAuthentication.RedirectFromLoginPage(UserName, true); 
     } 
     else 
     { 
      InvalidLogIn.Visible = true; 
     } 
    } 

    connection.Close(); 
} 

Answer 1

ちょうど私がそこに投げ捨てるもの：本当にこのようなユーザー名/パスワードを比較すべきではありません。

は、ログイン認証の基本的な考え方はこれです：パスワードが作成されるとき

1. 、は値をハッシュ化されたユーザーがログインしようとすると テーブル
2. に格納され、彼らはを提出パスワード
3. パスワードがハッシュされました
4. ハッシュ（試行）がデータベースのエントリと比較されます。

...コードを正しく読んでいると、パスワードがSQLのプレーンテキストで保存されているようです。そしてあなたのクエリは値を取得し、それをC＃で比較します。

私は間違ってはいけません。これはちょっとした個人的なサイトであり、パスワードテーブルにはあなたのエントリだけがありますか？それでは問題ありません。しかし、あなたのサイトにユーザー名/パスワードを入れている人がいれば、本当に自分のデータを保護する必要があります。