Har et loginsystem som ikke virker optimalt

Prøv:
if ($_SESSION['brugernavn'] == "Admin") {

Men hvad sker der hvis brugeren så selv går til admin_panel.php i adresselinien? Har han så adgang? Det er jo ikke så godt :)

Nej, så har han ikke adgang, det er skam session beskyttet ;)

men jeg prøver lige din jaw!

Ja, nu virker det, men er der ikke en smartere måde ?
for den der kan da nemt brydes ?

Kan den nemt brydes? De færreste ting er jo 100% sikre...

Nej, har du fuldstændig ret i, men altså den tjekker jo bare om brugernavnet man er logget ind som er admin. Synes du at det er sikkert nok ? evt har du idéer til forbedring ?

læg da også lige et svar..

Nåå, men selve logindelen kunne jeg jo ikke se. Men du har vel defineret et adgangskodetjek, så den session kun bliver sat hvis adgangskoden også er rigtig. Og så er det såmænd i mine øjne rimelig sikkert - hvis den del af scriptet er :)

Ja ja, det har jeg da gjort ->

if($row['kodeord'] == $_POST['kodeord'])
{
//Brugeren har indtastet rigtige oplysninger, nu skal han gemmes i sessioner
$_SESSION['brugernavn'] = $_POST['brugernavn'];
//Derudover skal han sendes til en hemlig side, hvorpå der bliver tjekket om han er logget ind
header("location: logget_ind.php");
} else {
//Brugeren har skrevet et forkert kodeord
//Sender brugeren tilbage til login.htm ved brug af en header
header("location. login.htm");
exit;
}

men dit svar er accepteret!

der skulle have stået login.php i min kommentar, men det er fordi jeg normalt ikke har kommentare skrev bare lige en nu :)

Jojo, men pointen er der jo, og det tror jeg såmænd ikke er helt tosset, medmindre du skal ud i SSL-forbindelser og hvad der ellers er :)

Og tak...

Det er mig der takker :)

Nok er et svar givet, men man kunne inde i Tabellen lave en row som hedder status, og at den status skule være ADMIN eller et bestemt tal før at den side kom, det bruger jeg i mit Admin system, det er umuligt at bryde da ingen har adgang til selve databasen og kan ændre i det ..

bare en lille kommentar .. :D

Bruger jeg også, men vel egentlig kun nødvendigt i det øjeblik du har flere niveauer :) ?

Jaa , den kan bruges til man kan uddele Status'er , altså admin, coadmin, vedligeholder osv .. jeg synes den er god , men kan oZz bruges selvom man kun har et niveau .. man kan jo altid udvide Status'erne .. :D ;D