Permessi login php

[traco]

salve ragazzi,

ho uno script per la registrazione e autenticazione.adesso volevo gestire gli utenti.ho creato un campo nella tabella utenti chiamata auth_level dove ad 1 corrisponde utente registrato e basta,2 Admin e 3 Utentesuper.

nel metodo della classe dove verifico il login ho messo anche :
$_SESSION['auth'] = $row['auth_level'];
dove prelevo il campo di accesso.

ho fatto poi una funzione
public function IsAdmin()
{


if($_SESSION['auth'] == 2)
{
header("Location :database.php");
}
else
{
header("Location :index.php");
}
}

e in ogni pagina inserisco :

<?php
include 'authentication.class.php';

$auth = new UserAuthentication();
$auth->IsAuth();
$auth->IsAdmin();
?>

non mi funziona.Consigli ?

[eiyen]

ma "non mi funziona" che vuol dire? errori o problemi di logica?

[traco]

non funziona.non so se la logica è giusta. vi posto le due funzioni.

        public function VerifyLogin()
        {
            $username = $_POST['username'];
            $password = md5($_POST['password']);
            $this->DbConnect();
            //$sql = "SELECT id FROM utenti WHERE username='$username' AND password='$password' AND ver=1";
            $sql = "SELECT id,auth_level FROM utenti WHERE username='$username' AND password='$password' AND ver=1";
            $res = mysql_query($sql,$this->conn);
 
 
 if($row = mysql_fetch_array($res))
{
    $_SESSION['auth'] = $row['auth_level']; //prendo l'id del permesso
    $_SESSION['user_id'] = $row['id'];
    $sql1 = "SELECT username FROM utenti WHERE id=$_SESSION[user_id]";
            $res1 = mysql_query($sql1,$this->conn);
            $row1 = mysql_fetch_array($res1);
           $_SESSION['u'] = $row1['username'];
    header("Location: ww.php");
    mysql_close($this->conn);
    die;
}
            else
            {
                header("Location: login.php?alert=1");
                mysql_close($this->conn);
                die;
            }
        } 

        public function IsAdmin()
{


   if($_SESSION['auth'] == 2)
    {
        header("Location :database.php");
    }
    else
    {
        header("Location :index.php");
    }
} 

       <?php 
include 'authentication.class.php';
 
$auth = new UserAuthentication();
$auth->IsAuth();
$auth->IsAdmin();
 
 

?>

[eiyen]

diciamo così... cosa ti aspetteresti e cosa invece "vedi" che succede? non è riconosciuto il login come se non fosse effettuato o cos'altro?

[traco]

mi aspetterei che mi rimandasse alla pagina database.php se ad accedere è l'admin che valore = 2.invece ad ogni login vengo riportato alla pagina ww.php che il profilo dell'utente.

[eiyen]

il metodo "isAuth" non vedo dov'è definito, nè "VerifyLogin" vedo dov'è usato...

[traco]

    public function IsAuth()
        {
            if(!isset($_SESSION['auth']))
            {
                header("Location: login.php");
                die;
            }
        } 

sono tutti metodi che fanno parte della classe "class UserAuthentication"
e vengono chiamati in ogni pagina privata.

[eiyen]

un po' come immaginavo... ci può essere un errore nella logica per cui non viene mai raggiunto il redirezionamento voluto... in particolare non metterei mai un redirezionamento così dentro i metodi, secondo me ci vuole un metodo che restituisce l'azione o che ingloba tutti i casi, tipo:

if ($REDIR = $auth->Redir()) {
 header("Location: ".$REDIR); exit;
};

oppure direttamente mettendo header dentro il metodo, che cmq si occupa di controllare la tipologia di utente e di scegliere l'url di destinazione (eventualm. richiamando altri metodi privatamente)... cmq a "IsAuth" e agli altri simili farei solo restituire un booleano, tipo:

public function IsAuth() { 
  return(isset($_SESSION['auth']));
}