문자열 ""에서 유형 '부울'로의 변환이 올바르지 않습니다.

Question

ASP.NET 로그인 양식의 제목에 오류 메시지가 나타납니다. 누구든지 어떻게 정렬 할 수 있는지 알고 있습니까?

Dim isValidLogin = reader.GetValue(1) 

에 : 도움이 많이

Protected Sub Button1_Click(ByVal sender As Object, ByVal e As EventArgs) Handles Button1.Click 
    Dim conn As New MySqlConnection 

    conn.ConnectionString = ("server=localhost;port=3307;user=user;password=password;database=DB;") 

    Try 

    Dim SQL As String = "select * from users3 where uname = '" & txtUName.Text & "' AND password = '" & txtPwd.Text & "'" 

    conn.Open() 

    Dim cmd As New MySqlCommand(SQL, conn) 
    Dim reader As MySqlDataReader = cmd.ExecuteReader 

     reader.Read() 
     Dim isValidLogin As Boolean 
     Boolean.TryParse(reader.GetValue(1), isValidLogin) 

     If isValidLogin Then 
      Session("UserName") = txtUName.Text 
     Response.Redirect("REGISTERPROP.aspx") 
    Else 
     Response.Write("Invalid Login") 
    End If 

    Catch ex As Exception 
    Response.Write("An Error Occurred: " & ex.Message.ToString()) 
    End Try 
End Sub 

Answer 1

변경 감사

Dim isValidLogin As Boolean 
Boolean.TryParse(reader.GetValue(1), isValidLogin) 

Answer 2

음, 메시지는 거의 자명하다. 빈 (nil) 문자열을 부울로 변환하려고합니다. Boolean의 유효한 문자열 값은 대문자와 소문자를 구분하지 않습니다 (true 및 false). 그래도 다른 문제가 있습니다. 사용자/암호가 발견되면

• 귀하의 질의는 사용자/암호가 발견되지 않으면 0 행 (빈 세트), 그리고 (아마도) 1 개 행을 반환합니다. 그러나 행을 읽으면 true을, false otherwise을 반환하면 Read() 메서드에서 반환 값을 확인하지 않습니다. 쿼리는 일치하는 행만 반환합니다. 데이터를 검색하기 전에 해당 행을 확인해야합니다.

• 또한 쿼리에는 SQL Injection 취약점이 있습니다. 매개 변수가있는 쿼리 또는 저장 프로 시저 사용을 고려하십시오. 당신이 (또는 단순히 게시물) 누군가가 암호를 필드 페이지에이 다시는 경우 발생할 수있는 생각 : 그것은 나타납니다

  ; drop table users3 ; 
  

• 당신이 당신의 데이터베이스에 맑은에 암호를 저장하고 있습니다. SQL Injection 취약점과 관련하여 시스템 및 사용자가 침해당하는 것을 방지 할 수 있습니다. SHA-256과 같은 secure hashing algorithm을 사용하여 암호를 해시하고 해시 된 암호를 해시하는 것이 좋습니다.

는

`select 'true' from user3 where ...` 

에 조회를 변경하고 결과 세트가 비어있는 경우 첫 번째 결과 집합의 행 또는 null의 첫 번째 열을 반환 DbReader.ExecuteScalar()을 사용하여 실행

.

Dim isValidLogin = cmd.ExecuteScalar() 
If isValidLogin IsNot Nothing And isValidLogin Then 
    Session("UserName") = txtUName.Text 
    Response.Redirect("REGISTERPROP.aspx") 
Else 
    Response.Write("Invalid Login") 
End If 

의