Adgangskontrol via en database

Hvor grundigt har du testet DBAuthenticate, for det kode du har postet er det ligemeget med session variablerne. Så med mindre at du faktisk bliver redirected til CMSstart.aspx der så ikke tjekker på session["godkendt"] og det så er CMSstart der vidersender til default.aspx, så vil jeg tro det er DBAuthenticate den er gal med.

Har du kun 1 række i login tabellen? For du selecter det hele men læser kun 1 række. Hvis du har mere end 1 række og det ikke er den bruger du rammer i første hug, så vil brugeren ikke blive authenticated, på trods af valide credentials.
   

Det kunne løses lidt alla det er:

int DBAuthenticate(string username,string password){

    SqlConnection OurConnection;
    SqlCommand OurCommand;
    SqlDataReader OurDataReader;
    string SQLString;
    int Login_ID=0;
   
    OurConnection = new SqlConnection("Server=localhost;uid=sa;password=;database=Data/Database");
    OurConnection.Open();
    OurCommand = new SqlCommand("Select * from Login where username=@pUsername", OurConnection);

    SqlParameter pUsername  = new SqlParameter();
    pUsername.ParameterName = "@pUsername";
    pUsername.Value        = inputCity;
   
    OurCommand.Parameters.Add(pUsername);

    OurDataReader = OurCommand.ExecuteReader();
   
    if(OurDataReader.HasRows){
    //bruger fundet
    OurDataReader.Read();
    if(OurDataReader["Password"].ToString() == password)
        Login_ID=Int32.Parse(OurDataReader["LoginID"].ToString());
    }   
    OurDataReader.Close();
    OurConnection.Close(); 
   


    return Login_ID;
}


Ellers så prøv at gå ind og sæt forskellige retur værdier, -1

Hov....utilsigtet submit :)

Jeg fortsætter lidt her:
-1, i den ene else og minus 2 i den anden f.eks.

Den må lige forklare lidt mere specifikt! :-\

Ok, jeg tror problemet er med din db metode.

Du checker altid kun 1 række i tabellen, hvilket vil være et problem hvis der er flere brugere.

Det kan også være at din connectionstring ikke virker eller andet eller at brugernavnet slet ikke findes.

Her er en tilrettet metode ( nu med færre slå fejl osv :))
Du kan jo prøve om det hjælper:

int DBAuthenticate(string username,string password)
        {
                /* returnere userid eller følgende fejlkoder:
                * -1 = bruger fundet, forkert password
                * -2 = bruger ikke fundet
                * -3 = fejl
                *
                * */
           
                SqlConnection OurConnection = new SqlConnection("Server=localhost;uid=sa;password=;database=Data/Database");
                SqlCommand OurCommand;
                SqlDataReader OurDataReader;
       
                int Login_ID=0;
            try
            {
                OurConnection = new SqlConnection("Server=localhost;uid=sa;password=;database=Data/Database");
                OurConnection.Open();
                OurCommand = new SqlCommand("Select * from Login where username=@pUsername", OurConnection);

                SqlParameter pUsername  = new SqlParameter();
                pUsername.ParameterName = "@pUsername";
                pUsername.Value        = username;
   
                OurCommand.Parameters.Add(pUsername);

                OurDataReader = OurCommand.ExecuteReader();
   
                if(OurDataReader.HasRows)
                {
                    //bruger fundet
                    OurDataReader.Read();
                    if(OurDataReader["Password"].ToString() == password)
                    {
                        Login_ID=Int32.Parse(OurDataReader["LoginID"].ToString());
                    }
                    else
                    {
                        Login_ID = -1;
                    }
                }
                else
                {
                    Login_ID = -2;
                }
                OurDataReader.Close();
                OurConnection.Close();
            }
            catch
            {
                    Login_ID=-3;
            }
            finally
            {
                if(OurConnection.State == System.Data.ConnectionState.Open)
                    OurConnection.Close(); 
            }
           
   
            return Login_ID;
        }

Uanset hvad jeg gør så sender den mig vidre til default.aspx

void Authorize(Object Sender, EventArgs e){
    int Login_ID = DBAuthenticate(Brugernavn.Text, Adgangskode.Text);
    if (Login_ID > 0){
        Session["Godkendt"] = "Godkendt";
        Response.Redirect("CMSstart.aspx");       
    }
    else
    {     
        Response.Redirect("default.aspx"); <--------- HER SENDER DEN MIG HEN!
    }
}

Så må dit Login_ID komme tilbage mindre end 1.

Men hvor fejler din DBAuthenticate henne?
Er det din connection, username/password eller noget tredje?

Hvis du prøver at se hvad du rent faktisk får tilbage (0,-1,-2 eller -3)

Hvis du ikke lige har mulighed for at debugge (det virker lidt sådan), så lav en lille test og så arbejd dig videre derfra.


  int Login_ID = DBAuthenticate(Brugernavn.Text, Adgangskode.Text);
switch(Login_ID){
case 0:{Response.Redirect("www.google.com"); break;}
case -1:{Response.Redirect("www.dr.dk"); break;}
case -2:{Response.Redirect("www.tv2.dk"); break;}
case -3:{Response.Redirect("www.dmi.dk"); break;}
}

eller lav en response.write(Login_ID.To.String()) istedet for.

Dette script virker nu. Problemet lå i min database. Tabellen var sat til 16 tegn pr. default og derfor redirectede den hvis alle 16 tegn ikke var udfyldt. Dette er lavet om til 8 tegn og virker fuldt ud.

<%@ page language="c#" runat="server" ResponseEncoding="unicode" debug="true" %>
<%@ Import Namespace="System.Data"%>
<%@ Import Namespace="System.Data.SqlClient"%>
<script runat="server">
void Authorize(Object Sender, EventArgs e){
    int LoginID = DBAuthenticate(Brugernavn.Text, Adgangskode.Text);
   
   
    if (LoginID > 0){
        Session["Godkendt"] = "Godkendt";
        Response.Redirect("CMSstart.aspx");
       
    }else{     
        Response.Redirect("default.aspx");
    }
}

int DBAuthenticate(string Username,string Password){
   
    SqlConnection OurConnection;
    SqlCommand OurCommand;
    SqlDataReader OurDataReader;
    string SQLString;
    int LoginID;
   
    SQLString = "Select LoginID,Password from Login where Username = '" + Username + "'";   
    OurConnection = new SqlConnection("Server=(local);uid=Grp2;Password=rrkh;database=Database");
    OurConnection.Open();
    OurCommand = new SqlCommand(SQLString ,OurConnection);
    OurDataReader = OurCommand.ExecuteReader();
   
    if (OurDataReader.Read()){   
        if (OurDataReader["Password"].ToString() == Password){       
                LoginID = Int32.Parse(OurDataReader["LoginID"].ToString());
        }else{
                LoginID = 0;
        }   
    }else{
        LoginID = 0;
    }   
   
    OurDataReader.Close();
    OurConnection.Close();   
   
    return LoginID;
}

</script>

lukket