[PHP][MYSQL] Zabezpieczenie danych pochodzących z formularza Opcje

[Woytek950]

Witam. Czy dobrze zabezpieczyłem skrypt...?

Kod

  $haslo = sha1(md5(mysql_real_escape_string(trim(addslashes($_POST['haslo'])))));
                       $login =  addslashes(trim($_POST['login']));
                       $email =  addslashes(trim($_POST['email']));

Mógłby ktoś mi objśnić jak zrobić:

aby login, hasło musiły mieć więcej niż 3 znaki;

aby sprawdzalo poprawność emailal;

i ogólnie jakieś porady dotyczące bezpieczeństwa skryptu

będę bardzo wdzięczny !

dziękuje z góry i pozdrawiam

[Fifi209]

Witam. Czy dobrze zabezpieczyłem skrypt...?

Kod

  $haslo = sha1(md5(mysql_real_escape_string(trim(addslashes($_POST['haslo'])))));
                       $login =  addslashes(trim($_POST['login']));
                       $email =  addslashes(trim($_POST['email']));

Mógłby ktoś mi objśnić jak zrobić:

aby login, hasło musiły mieć więcej niż 3 znaki;

aby sprawdzalo poprawność emailal;

i ogólnie jakieś porady dotyczące bezpieczeństwa skryptu

będę bardzo wdzięczny !

dziękuje z góry i pozdrawiam

Nie hashuje się dwa razy - czemu? Było o tym.
Poczytaj jak działa mysql_real_escape_string i potem zastanawiaj się czy używać addslashes

Co do pytania o login, hasło i email (poprawność) to wyrażenia regularne.

Ogólne porady: własny system sesji

[Pawel_W]

poprawność email -> preg_match()" title="Zobacz w manualu PHP" target="_manual
login i hasło -> strlen()" title="Zobacz w manualu PHP" target="_manual

[mac_fly]

Aby ustawić limit znaków to dajesz pętle if bądź inną, np:

[PHP] pobierz, plaintext 
<?php
$haslo2 = strlen($haslo);
if($haslo2<3)
{
echo"Hasło musi zawierac najmniej 3 znaki";
}
?>
[PHP] pobierz, plaintext 

login robisz tak samo.
Odnośnie maila użyj np, coś takiego:

[PHP] pobierz, plaintext 
<?php
if(!eregi("^([a-z0-9_-]+([.a-z0-9_-]+)*)@([a-z0-9_-]{1,}(.[a-z0-9_-]{1,})*.[a-z]{2,3})$",$email))
{
echo "Error";
}
?>
[PHP] pobierz, plaintext 

albo innej funkcji która sprawdza po znakach:strpos

Ten post edytował mac_fly 22.07.2009, 18:48:14

[Pawel_W]

Aby ustawić limit znaków to dajesz pętle if bądź inną, np:

pętlę? if to instrukcja warunkowa...

[Wicepsik]

Aby ustawić limit znaków to dajesz pętle if bądź inną, np:

[PHP] pobierz, plaintext 
<?php
$haslo2 = strlen($haslo);
if($haslo2<3)
{
echo"Hasło musi zawierac najmniej 3 znaki";
}
?>
[PHP] pobierz, plaintext 

login robisz tak samo.
Odnośnie maila użyj np, coś takiego:

[PHP] pobierz, plaintext 
<?php
if(!eregi("^([a-z0-9_-]+([.a-z0-9_-]+)*)@([a-z0-9_-]{1,}(.[a-z0-9_-]{1,})*.[a-z]{2,3})$",$email))
{
echo "Error";
}
?>
[PHP] pobierz, plaintext 

albo innej funkcji która sprawdza po znakach:strpos

Lepiej użyć preg_match" title="Zobacz w manualu PHP" target="_manual, ponieważ eregi nie będzie w PHP6

[Woytek950]

Mam jeszcze jeden problem...

Chcę aby poprawność email i hasło sprawdzane było w tym samym momęcie. jak to zrobić?

dla zobrazowania

...

Kod

 if (!empty($pass))
                   { cos tam...}

else                      {
                          echo"nie wpisałes hasła !";
                      }
                    
if (!empty($email))
                   { cos tam...}

else                      {
                          echo"nie wpisałes meila !";
                      }

jezeli te 2 warunki sie spelnia przejdz dalej... ale oczywiscie warunek nie będzie  sie konczył na !empty

Ten post edytował Woytek950 22.07.2009, 19:38:46

[mac_fly]

[PHP] pobierz, plaintext 
<?php
if(!empty($pass)&& !empty($email))
{
echo"Cos tam";
}
else
{
echo"Nie wpisałes hasła bądź maila";
}
?>
[PHP] pobierz, plaintext 

[Woytek950]

[PHP] pobierz, plaintext 
<?php
if(!empty($pass)&& !empty($email))
{
echo&#092;"Cos tam\";
}
else
{
echo&#092;"Nie wpisałes hasła bądź maila\";
}
?>
[PHP] pobierz, plaintext 

Przeciez napisałem:


"jezeli te 2 warunki sie spelnia przejdz dalej... ale oczywiscie warunek nie będzie sie konczył na !empty" - będzie bardziej rozbudowny







Chodzi mi o to że np. mam sprawdzanie poprawnosci emaila, to jak email jest poprany przejdz dalej i sprawdz haslo, jak haslo jest ok to sprawdz nick itd

A jak zrobić aby sprawdzał wszystko na raz i ewentualnie pokazywał blędy 

Ten post edytował Woytek950 23.07.2009, 10:35:34

[Pawel_W]

powiedz dokładnie co chcesz, bo w ciemno nie możemy napisać ci skryptu, btw przecież instrukcja warunkowa to podstawy...

[Woytek950]

Czy addslashes dodać po usunięciu spacji czy przed?