remember me problem checked problem can any one help me

Question

0.00/5 (No votes)

See more:

Hi folk's

I have a login form in that if i check the remember me check-box,then i want to remember username and password ..

below code is for remember me.. functionality...it is working fine but only one problem ...After checking remember me check box.. if i reload login page the checked remember not remains can any one help me how can i resolve this problem.....

Can any one help me......
My code is in this way..........for remember me .....

Below code is working fine only problem is checked remember is not remains ....

protected void Page_Load(object sender, EventArgs e)
   {
       if (Page.IsPostBack)
       {
           if (chk_rem.Checked)
           {
               HttpCookie mycookie1 = new HttpCookie("mycookie1");
               //Response.Cookies.Remove("mycookie1");

               Response.Cookies.Add(mycookie1);

               mycookie1.Values.Add("Username", txt_username.Text);
               mycookie1.Values.Add("Password", txt_password.Text);
               DateTime dtex = DateTime.Now.AddDays(15);

               if (chk_rem.Checked == true)
               {
                   if (Request.Cookies["mycookie1"] != null)
                   {
                       HttpCookie getcookie1 = Request.Cookies.Get("mycookie1");
                       string un = getcookie1.Values["Username"].ToString();
                       string ps = getcookie1.Values["Password"].ToString();
                   }

               }

           }
           if (chk_rem.Checked == false)
           {
               Response.Cookies["mycookie1"].Expires = DateTime.Now.AddDays(-1);
           }
       }


       if (HttpContext.Current.Request.Cookies["mycookie1"] != null)
       {
           try
           {
               txt_username.Text = HttpContext.Current.Request.Cookies["mycookie1"]["Username"].ToString();
               string a = HttpContext.Current.Request.Cookies["mycookie1"]["Password"].ToString();
               txt_password.Attributes.Add("value", a);

               chk_rem.Checked = true;
           }
           catch { }
       }

   }

Thank's&Regard's
mahesh.b.p.c

Posted 2-Sep-12 23:31pm

mahesh.b.p.c

Updated 2-Sep-12 23:36pm

Karwa_Vivek

v2

Add a Solution

Comments

L Viljoen 3-Sep-12 6:23am

What you are currently doing poses as a massive security risk as the username and password is being stored in clear text on a cookie.

What i recommend is generating a user id (GUID) and encrypting it in order to automatically log a user in instead of saveing the login details on the client pc., also what you are doing is currently begin done automatically by most mainstream browsers.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

**AmitGajjar** · Answer 1 · 2012-09-02T23:44:00

Hi,

You are storing cookies for checked Remember me button. So you need to add one condition after first if condition. you can check if cookies exist then you need to manually check that checkbox. You need to do that because control never persist it's value after next postback(refresh page).

And even if credential is stored then you do not need to display that page. just redirect to user's home screen. At the time user click logout, you can remove cookies and user can enter username & password(this will depend upon your requirement).

C#

if (Page.IsPostBack)
        {
            if (Response.Cookies["mycookie1"] != null)
            {
                chk_rem.Checked = true;
            }
            if (chk_rem.Checked)
            {
                HttpCookie mycookie1 = new HttpCookie("mycookie1");
                //Response.Cookies.Remove("mycookie1");
               
                Response.Cookies.Add(mycookie1);
               
                mycookie1.Values.Add("Username", txt_username.Text);
                mycookie1.Values.Add("Password", txt_password.Text);
                DateTime dtex = DateTime.Now.AddDays(15);
 
                if (chk_rem.Checked == true)
                {
                    if (Request.Cookies["mycookie1"] != null)
                    {
                        HttpCookie getcookie1 = Request.Cookies.Get("mycookie1");
                        txtUserName.Text = getcookie1.Values["Username"].ToString();
                        txtPassword.Password = getcookie1.Values["Password"].ToString();
                    }
 
                }
 
            }
            if (chk_rem.Checked == false)
            {
                Response.Cookies["mycookie1"].Expires = DateTime.Now.AddDays(-1);
            }
        }

Hope you got the idea from above statement,
Thanks
-Amit Gajjar