Mam pewnie skrypt logowania. Chciałbym abyscie ocenili jego bezpieczoeństwo:
index.php<?php
include('includes/config.php');
mysql_connect($db_host,$db_user,$db_pass) or
die ("Nie mogę nawiązać połączenia z bazą danych!");
{
echo("Zaloguj się:<br />"); include("forms/login.php");
echo("<br />Lub <a href=\"forms/register.php\">zarejestruj się</a>"); echo("<br />Lub <a href=\"page1.php\">wejdz na podstrone nr1</a>"); }
else
{
#
#Logowanie
#
$sql = "SELECT * FROM `v_users` WHERE `user_password`='".$password."' AND `user_username`='".$username."'";
{
$getid = "SELECT * FROM `v_users` WHERE `user_username`='".$username."' LIMIT 1";
$userid = $r[userid];
}
$_SESSION['logged']=1;
}
else
{
die("Nazwa użytkownia i/lub hasło są błędne!"); }
#
#---------------------------------------------------------
#
}
if(isset( $_SESSION['logged'])) {
echo("Jestes zalogowany<br /><br />"); echo("<br /><a href=\"page1.php\">wejdz naa podstrone nr1</a><br><br>");
#...
#TRESCI DOSTEPNA PO ZALOGOWANIU
#...
include("forms/logout.php");
}
?>
includes/config.php<?php
#Polaczenie z baza danych
$db_host = "localhost";
$db_user = "root";
$db_pass = "";
$db_name = "database";
#Login/Logout
$username=$_POST['login'];
$password=$_POST['pass'];
$logout=$_POST['logout'];
$password=md5($password); ?>
Czy ten skrypt jest wystarczajaco bezpieczy czy moze powinenem w nim cos zmienic - dodac?