passing a variable from page to another page

[RomeoX]

Hi there, I would like to ask a question about how to pass a variable from page to page. For example, I have my first page for login and used a session with cookie and after submitting successfully and redirect my page to another page I want show statement like Hello Smith or Welcome Smith, and it doesn't work properly. Anyone knows the soulution would be appreciated. Thanks

<?php
if (loggedin())
    {
    header("Location: redirectpage.php");
    exit();
    }
if ($_POST["login"])
{
    global $username;
    $username = $_POST['username'];
    $password = $_POST['password'];
    $rememberme = $_POST['rememberme'];
 
 
    if($username&&$password)
    {
 
    $login = mysql_query("SELECT * FROM usersystem where username='$username'");
    while ($row = mysql_fetch_assoc($login))
    {
        $db_password = $row['password'];
        if($password==$db_password)
 
    else
        setcookie("username", $username, time()+7200);
        else if ($rememberme=="")
        $_SESSION['logged_in']== $username;
        $_SESSION['username'] =$_POST['username'];
 
//userarea.php
        header("Location: redirectpage.php");
        exit();
 
        }
 
    }
 
    }
    else
    die("Please enter a username and password");
 
}
?>
 

This is the code for the second page that I want the name of the user to show up.

<?php
session_start();
 
$username = $_SESSION['username'];
 
echo $_SESSION['username'];
 
 
 
echo "Hello ". $username ;
?>

So what's wrong with my code. Thanks again

[Atli]

Hey.

There is a syntax error there on line 24. The if, else and else if statements don't match up.

It is generally best to always use brackets with if statements, even if they only execute on line of code. It helps to avoid problems like these.

// Avoid this:
if(true == true)
    echo "It is true!";
echo "A completely unrelated line...";
 
// Rather do this:
if(true == true) {
    echo "It is true!";
}
echo "A completely unrelated line...";

There are a couple of extra charachters required, but it makes it a whole lot clearer to read, especially if your indentations are sloppy.

[RomeoX]

Actually the I wrote the code and I knew that it's wrong because I picked up just for example of how to pass session variable to another page and I was asking about this. Anyway thanks a lot of passing my thread.

[johny10151981]

you can try this

redirectpage.php?user_name=johny&user_age=100

then in redirectpage.ph p usign $_GET you can have the data. i.e

$name=$_GET[user_name];
$age=$_GET[user_age];

Regards
johny

[RomeoX]

Thanks for reply I'm really thankful to u

by the way, actually the first code that I submitted in the first topic that will be in the login screen and I can't pass the login and password in the tool bar of the browser so I'm using POST instead of get because it's much secure. I tried your code but I used POST but couldn't get the username.

So what should I do?

[johny10151981]

I dont know what is your plan.

But if you try session it may help. After user log in you can save the user_name and Password using session. your php page will be able to use these session anytime if session is valide. Cookie is a solution but a very easy solution to destroy the security.

Regards,
Johny

[Atli]

OK, to answer the original question: there are a couple of methods you can use to pass things between pages.

1. Sessions
   These are the most secure of the methods, because the actual data is stored on the server, while only a session identifier is passed to the browser (usually as a cookie). Any page on the domain can then activate the session and use the data, given that the cookie is not destroyed. This is highly recommended for storing sensitive data, such as user login information.
2. Cookies
   Less secure and reliable than sessions, but more long-term. Whereas a session is destroyed every time the browser is closed (by default), a cookie can remain indefinitely. It is usually best not to use these to store sensitive info, only info that would not be a security threat if it fell into the wrong hands.
3. GET parameters
   Adding variables to the URL can serve as a (very) short-term method of passing data between pages. This is usually reserved for things like passing paging information via navigation links. In general, if the same piece of data needs to be passed to more than a couple of pages, a session or a cookie is a better choice.

There are a few more possible methods, but they are generally so situational and questionable that they are hardly worth mentioning.

For the purposes of storing user login data, a session would be ideal. Putting the user name and ID into the session is pretty standard.

This is a pretty generic example of how that would be done.

<?php
// Make sure the login info was passed
if(isset($_POST['name'], $_POST['password'])) 
{
    // Fetch the user name from POST.
    // Note the use of the mysql_real_escape_string function.
    // It should ALWAYS be used on data that is to be
    // inserted into a SQL query.
    $name = mysql_real_escape_string($_POST['name']);
 
    // Fetch the password. Note that I hash() the password.
    // This ensures the password is secure, even if
    // the database itself is comprimized. You should
    // ALWAYS hash passwords, and never store them as
    // plain text. The above rule about the escape_string
    // function does not apply here, as a hash is always 
    // safe to put into a SQL query.
    $pwd_hash = hash('sha1', $_POST['password']);
 
    // Verify that the login info is valid.
    // It is better to fetch the user info based on
    // the username and the password, rather than to pass
    // it only the username and verify the password with
    // PHP. This way, if the login is invalid, the real
    // password never enters your PHP code, making it more
    // secure.
    $sql = "SELECT `id` FROM `user` 
            WHERE (`name` = '{$name}')
            AND   (`password` = '{$pwd_hash}')";
 
    $result = mysql_query($sql) or trigger_error(mysql_error(), E_ERROR);
 
    // If a single row was returned, the user info is
    // valid. If more than a single row was returned,
    // odds are that something went rong, or that your
    // code has somehow been comprimized. This is why
    // you should validate ONLY if a single row is returned.
    if(mysql_num_rows($sql) == 1) 
    {
        $row = mysql_fetch_assoc($sql);
 
        // Here we start the session and enter the
        // user info into it. Note that session values
        // can be arrays themselves, so that you can group
        // similar elements together, like I do here.
        session_start();
        $_SESSION['user']['id'] = $row['id'];
        $_SESSION['user']['name'] = $name;
    }
    else
    {
        echo "Login failed. Please try again!";
    }
}
else
{
    echo "Username and/or password were not passed.";
}
 
?>

You can then verify a user as logged in on other pages by doing something like:

<?php
session_start();
 
// Check if the user session element exists.
// If it does, we can assume the client has
// already logged in. If not, we can not.
if(!isset($_SESSION['user']))
{
    // Redirect back to the user login page.
    header('Location: login.php');
    exit;
}
 
// Display the rest of the user-secure content.
echo "Welcome, {$_SESSION['user']['name']}!";
?>

@johny10151981
You should NEVER store the password anywhere, especially in it's plain-text form. If you absolutely can not avoid it, you should at least hash it before doing so. Passwords are one of the more sensitive pieces of info your application will ever handle and they should be used as little as possible.

I mean, consider if a malicious user managed to inject a PHP script to your server. It would be fairly easy for him to hijack sessions and view all the session data, including the password. Limiting this to usernames and IDs makes this sort of breach a lot less damaging.

[kovik]

I disagree. The password should be stored in its hashed form. What if two users are logged into the same account at once? What if the valid user knew that someone knows his password and he needs to change it before the other user harms his account? He would then change his password. If the password was re-authenticated on every page request (as I believe it should be), then the false user would be essentially logged out.

[Atli]

That's an extremely rare scenario, to be honest, and preventing it hardly takes priority over the security of all passwords being used. Even in it's hashed form, in the hands of a malicious user a password would be a major security concern.

But if this scenario is of great concern to you, a far more sensible method - surely - would be to add a "modified" timestamp to the user account that would be updated with the password. The value of that timestamp at the time when a user is logged in would then be stored with the session and checked on every reload.

There is rarely a situation where you need to store the password anywhere - outside normal login and account maintenance - and you really should avoid it wherever possible, for obvious security reasons. That is what I would generally recommend, in any case.

[johny10151981]

Hi Atli,
You have comment out one of my line. Actually I am explaining it. I did tell to store data using SESSION but I didnt mean to store in storage device like hard disk. If I am not wrong session data get stored in the server and in the RAM. My understanding says Session stores run time data.

I also strongly disagree password in server's storage device.

Regards,
Johny