Forum PHP.pl > [PHP][MYSQL] Zabezpieczenie danych pochodzących z formularza

Woytek950

22.07.2009, 18:38:59

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

22.07.2009, 18:41:28

Cytat(Woytek950 @ 22.07.2009, 18:38:59 )

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

22.07.2009, 18:41:34

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

22.07.2009, 18:46:45

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

Pawel_W

22.07.2009, 18:47:30

Cytat

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

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

Wicepsik

22.07.2009, 18:54:19

Cytat(mac_fly @ 22.07.2009, 19:46:45 )

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

22.07.2009, 19:38:23

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

mac_fly

22.07.2009, 19:45:54

[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

23.07.2009, 10:32:27

Cytat(mac_fly @ 22.07.2009, 20:45:54 )

[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

Pawel_W

23.07.2009, 10:35:11

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

Woytek950

23.07.2009, 10:54:33

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