Vurdering af login script.

Tags:    php

<< < 12 > >>
Hej.

Jeg er forholdsvis ny inden for emnet, så derfor beder jeg jer vurdere mit lille login system jeg har lavet til en personlig side.

Jeg har først og fremmest en form, som beder om brugernavn og adgangskode:

login.php:
Fold kodeboks ind/udKode 


Som I kan se sender den brugeren videre til scr_login.php som indeholder selve koden bag formularen.
scr_login.php forbinder sig til en mysql database og sammenligner det indtastede brugernavn og pass med dem som er i mysql tabellen.
Viser det sig at de indtastede informationer stemmer med en af dem i tabellen, sender den brugeren videre til siden auth. Passer de ikke, sender den brugeren videre til login_fejl siden som indeholder en fejl meddelelse samt en ny formular.

scr_login.php:
Fold kodeboks ind/udKode 


auth:
Fold kodeboks ind/udKode 

Note: sessionen starter [session_start();] på en anden side hvor auth er inkluderet.

login_fejl:
Fold kodeboks ind/udKode 

Ikke så relavant for systemet, men nu har i den :)


Jeg håber jeg har givet nok information til i kan vurdere det. Måske der er noget som kan gøres smartere eller mere sikkert?
Hele systemet virker som det skal.

[Redigeret d. 17/09-05 00:33:10 af Tommy Jakobsen]

[Redigeret d. 17/09-05 00:34:42 af Tommy Jakobsen]



12 svar postet i denne tråd vises herunder
4 indlæg har modtaget i alt 4 karma
Sorter efter stemmer Sorter efter dato
Tak for de mange svar.


Nå, men sagen er den at du bør slå magic_quotes fra og proppe mysql_escape_string rundt om brugernavnet og adgangskoden i din query.


Jeg kan desværre ikke slå magic quotes fra på min server.
Jeg kiggede lidt på funktionen mysql_real_escape_string og fandt frem til funktionen quote_smart
Fold kodeboks ind/udKode 

som er taget fra et af eksemplerne på php.net.
Efter hvad jeg kan se skulle den også løse problemet med magic quotes?

Sætter jeg det ind i mit script - og tilpasser det, kommer den bare med en fejl som hvis jeg havde tastet brugernavn eller pass forkert.

Mit script ser nu sådan her ud:
Fold kodeboks ind/udKode 


Noget i kan hjælpe med? :-)

Mvh. Tommy

[Redigeret d. 18/09-05 16:04:02 af Tommy Jakobsen]



Prøv at udskrive $sql, og se hvad der står i den



<< < 12 > >>
t