login  Naam:   Wachtwoord: 
Registreer je!
Scripts > PHP > Beveiliging > Login script met auto. ip block


Reacties op het script Login script met auto. ip block

Offline  Rens
Gepost op: 02 januari 2006 - 17:35
Gouden medaille

Crew algemeen



php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($username == "$user"  && $password == "$pass")


Variabelen niet tussen quotes zetten.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. if($username == $user  && $password == $pass)



php code - Bekijk de code zonder highlighting - Klap code in 
  1. $_SESSION["ingelogd"]="true";


true (en dus ook false) niet tussen quotes.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. $_SESSION["ingelogd"]=true;



php code - Bekijk de code zonder highlighting - Klap code in 
  1. }elseif($username == '' && $password == '')


Waarom niet gewoon gebruik maken van empty?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. }elseif(empty($username) && empty($password)



php code - Bekijk de code zonder highlighting - Klap code in 
  1. }else if ($username == '')


Waarom niet gewoon gebruik maken van empty?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. }else if(empty($username)



php code - Bekijk de code zonder highlighting - Klap code in 
  1. }else if ($password == '')


Waarom niet gewoon gebruik maken van empty?

php code - Bekijk de code zonder highlighting - Klap code in 
  1. }else if(empty($password)



php code - Bekijk de code zonder highlighting - Klap code in 
  1. mysql_query("INSERT INTO banned_ips (ip, datum) VALUES ('$ip', now())");


Variabelen niet tussen quotes zetten.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. mysql_query("INSERT INTO banned_ips (ip, datum) VALUES ('".$ip."', now())");

Offline  eddaredevil
Gepost op: 03 januari 2006 - 12:53
PHP interesse



Waarom zet je geen variabelen tussen quotes neer? Geeft dat een andere waarde mee of is het gewoon makkelijker?

Bedankt voor je reactie.

ps. hoe veilig is het om op deze manier in te loggen?

(ik weet dat de ipblock wel te omzeilen is)

Offline  Rens
Gepost op: 03 januari 2006 - 16:09
Gouden medaille

Crew algemeen



Volgens mij is het behoorlijk veilig.
Ik zou geen manier weten om $user en/of $pass op het scherm te printen...

En over die variabelen, het is gewoon netter en ik geloof dat het ook wat scheelt in parse tijd, zal niet veel zijn, maar op een grote site kan het wel verschil maken...

Offline  Robert_Dam
Gepost op: 30 januari 2006 - 09:12
Nieuw lid



Ik krijg een nare foutmelding:

Warning: session_start(): Cannot send session cookie - headers already sent by (output started at /var/www/html/reg/inloggen.php:6) in /var/www/html/reg/inloggen.php on line 20

Warning: session_start(): Cannot send session cache limiter - headers already sent (output started at /var/www/html/reg/inloggen.php:6) in /var/www/html/reg/inloggen.php on line 20

Misschien kan iemand mij vertellen wat er fout gaat??

Thnx
Robert

Offline  eddaredevil
Gepost op: 31 januari 2006 - 11:11
PHP interesse



Probeer het even opnieuw als het goed is werkt t nu wel.

Offline  sayanora
Gepost op: 14 april 2006 - 16:18
Nieuw lid



is wel raar, zo een goed script, maar dan zo beginnersfoutjes maken:P

mysql_real_escape_string(); gebruiken kan ook altijd nuttig zijn.

Offline  eddaredevil
Gepost op: 17 april 2006 - 20:55
PHP interesse



Citaat:
is wel raar, zo een goed script, maar dan zo beginnersfoutjes maken

mysql_real_escape_string(); gebruiken kan ook altijd nuttig zijn.


het is mijn eerste script.. dus beginners foutjes zijn toch niet zo gek?

Waarom zou ik dat moeten gebruiken? mbt sql injections?


Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.
 
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.171s