PHP isset($_SESSION)

[ncurran217]

So I am new on using the SESSION variables in PHP. I have created a simple login screen. Not trying to be to secure with it, just wanted it to be a simple login so a certain field was filled in and the user cannot change the Rep field. Anyways, I have put this at the top of the page where it goes to when they login in the username and password are correct:

 

<?php
session_start();
if(isset($_SESSION['forteid']))
{
header("location: index.php");
}
else
{
header("location: login.php");
}
?>

 

When I log in the page comes up with:

 

This webpage has a redirect loop. The webpage at http://cslogs:8081/test/index.php has resulted in too many redirects. I am new with this so not sure how to fix this one or haven't found a way to attempt to fix it.

 

Basically I want it to where they can't just type in http://cslogs:8081/test/index.php in the address bar and get to the screen without logging in, which the login page is just login.php.

 

Thanks in advance.

[Jessica]

You'll have to check if you're at login.php before executing that code.

[ncurran217]

Ok, makes sense, would you be able to point me in the direction on how I would do that?

 

Or a bit of the code that I can research?!

[ncurran217]

Well I found out what I was doing wrong. Thanks for the help anyways.

 

<?php
session_start();
if (!(isset($_SESSION['forteid']) && $_SESSION['forteid'] != '')) {
header ("Location: login.php");
}
?>

 

And it works!

[PFMaBiSmAd]

You need a exit; statement after your header() redirect to prevent the rest of the code on your 'protected' page from running while the browser is preforming the redirect. Without an exit; to stop the script, all anyone needs to do is ignore the redirect and they can access your 'protected' pages the same as if you didn't have a log in system at all.

[bashy]

You do indeed need exit(); after a header() as said above.

You should also do this;

 

if(!isset($_SESSION)) session_start();

[PFMaBiSmAd]

If you are conditionally starting your session, it means you might have already started it somewhere else in your code. That would indicate code that is out of control, requiring extra code to compensate for a messy design.