Does this seem Logical?

[Lamez]

I have been playing around with checking on users with stale sessions. I decided to give this method a try. When they login a time will be set, in a session, then it will be checked constantly to see if their session is still good. I try putting in 1 as the limit (1 minuet?), but I still get "still chewy".

 

here is the function in question:

<?php
function timeCheck($limit){
if(!isset($_SESSION['~time']) && !isset($_SESSION['~limit'])){
	$_SESSION['~time'] = time();
	$_SESSION['~limit'] = time()+$limit;
}
if($_SESSION['~time'] == $_SESSION['~limit']){
	return "expired";	
}else{
	return "still chewy";
}
}
?>

[oni-kun]

Are you putting session_start(); at the top of your page? 

[Lamez]

Yes, it is being included.

[oni-kun]

Yes, it is being included.

 

if(!isset($_SESSION['~time']) && !isset($_SESSION['~limit'])){

      $_SESSION['~time'] = time();

This sets the time, and does not update it based on the current time.

 

if($_SESSION['~time'] == $_SESSION['~limit']){

What if ~time is over ~limit? You're not checking if it's greater than.

 

[Lamez]

Thanks for the help. That does make a lot of sense. Here is my new code:

 

<?php
function timeCheck($limit){
if(!isset($_SESSION['~limit'])){
	$_SESSION['~limit'] = time()+$limit;
}
if(time() >= $_SESSION['~limit']){
	return "expired";	
}else{
	return "still chewy";
}
}
?>

 

Now I am getting expired. I am setting the limit to 10, would that be 10 seconds or 10 minuets? The manual did not help much.

[oni-kun]

time(): Returns the current time measured in the number of seconds

 

 

[Lamez]

I see, so adding

 

$limit *= 60;

 

will turn it into minuets?

[oni-kun]

I see, so adding

 

$limit *= 60;

 

will turn it into minuets?

 

Well yes, but you may as well just place: for ease.

time()+($limit*60);

[Lamez]

I gotcha, save that RAM.

 

So my final code works:

<?php
function timeCheck($limit){
if(!isset($_SESSION['~limit'])){
	$_SESSION['~limit'] = time()+($limit*60);
}
if(time() >= $_SESSION['~limit']){
	unset($_SESSION['~limit']);
	return "expired";			
}else{
	return "still chewy";
}
}
?>