Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP

Login script (non-mysql) gemaakt, is dit veilig?

Nrzonline - 28/02/2006 08:49

PHP ver gevorderde

Hey SIMA mensen!

Ik ben bezig geweest met een login script voor een vriend. Hij wou
geen gebruik maken van een mysql database omdat hij daar nog
niets van snapt, en dus liever een aanpasbaar php scriptje wil
hebben. Dat heb ik geprobeerd, en dit is de eerste keer dat ik een
non-mysql login scriptje heb gemaakt. Niet afgekeken van andere
scripts of iets dergelijks, dus ik weet niet of hij wel echt veilig is...
Daarom zou ik jullie willen vragen of dit script veilig is..

www.runescapeforum.net/login?page=login

Hier het script:

LOGIN.PHP

php code - Bekijk de code zonder highlighting - Klap code in


<?PHP

error_reporting(E_ALL);

if (isset($_COOKIE['cook_login']))
{
	echo "U bent al ingelogged";
}
else
{
	if (isset($_POST['login_account']))
	{
		$ErrorArray = array();
		
		if (empty($_POST['user_name']))
			$ErrorArray[] = "Vul gebruikersnaam in<br>";
		if (empty($_POST['user_password']))
			$ErrorArray[] = "Vul wachtwoord in<br>";
		
		if (count($ErrorArray) > 0)
		{		
			echo "<ul>";
			for ($i = 0; $i < count($ErrorArray); $i ++)
			{
				echo "<li>" . $ErrorArray[$i] . "</li>";
			}
			echo "</ul>";
		}
		else
		{
			$use_username = strtolower($_POST['user_name']);
			if (in_array($use_username, $username))
			{
				if ($_POST['user_password'] == $password[$use_username])
				{
					setcookie("cook_login", $use_username . ";" . $_POST['user_password'], time()+3600*24*31*1, "/");
					echo stripslashes("Je bent nu ingelogged, je kunt nu naar de beveiligde pagina\'s gaan");
				}
				else
					echo "Gebruikersnaam en Wachtwoord komen niet overeen";
			}
			else
				echo "Ongeldige Gebruikersnaam";
		}
	}
	else
	{
		echo "<table width='500' cellspacing='1'>";
		echo "<form action='' method='post'>";
		
		echo "<tr><td width='300'> <b>Gebruikersnaam:</b> </td>";
		echo "<td> <input name='user_name' type='text' maxlength='20' size='20'> </td>";
		
		echo "<tr><td width='200'> <b>Wachtwoord:</b> </td>";
		echo "<td> <input name='user_password' type='password' maxlength='20' size='20'> </td>";
		
		echo "<tr><td colspan='2'><div align='center'> <input name='login_account' type='submit' value='Inloggen!'> </div></td>";
		
		echo "</form>";
		echo "</table>";
	}
}

?>

<?PHP
 
error_reporting(E_ALL);
 
if (isset($_COOKIE['cook_login']))
{
	echo "U bent al ingelogged";
}
else
{
	if (isset($_POST['login_account']))
	{
		$ErrorArray = array();
 
		if (empty($_POST['user_name']))
			$ErrorArray[] = "Vul gebruikersnaam in<br>";
		if (empty($_POST['user_password']))
			$ErrorArray[] = "Vul wachtwoord in<br>";
 
		if (count($ErrorArray) > 0)
		{		
			echo "<ul>";
			for ($i = 0; $i < count($ErrorArray); $i ++)
			{
				echo "<li>" . $ErrorArray[$i] . "</li>";
			}
			echo "</ul>";
		}
		else
		{
			$use_username = strtolower($_POST['user_name']);
			if (in_array($use_username, $username))
			{
				if ($_POST['user_password'] == $password[$use_username])
				{
					setcookie("cook_login", $use_username . ";" . $_POST['user_password'], time()+3600*24*31*1, "/");
					echo stripslashes("Je bent nu ingelogged, je kunt nu naar de beveiligde pagina\'s gaan");
				}
				else
					echo "Gebruikersnaam en Wachtwoord komen niet overeen";
			}
			else
				echo "Ongeldige Gebruikersnaam";
		}
	}
	else
	{
		echo "<table width='500' cellspacing='1'>";
		echo "<form action='' method='post'>";
 
		echo "<tr><td width='300'> <b>Gebruikersnaam:</b> </td>";
		echo "<td> <input name='user_name' type='text' maxlength='20' size='20'> </td>";
 
		echo "<tr><td width='200'> <b>Wachtwoord:</b> </td>";
		echo "<td> <input name='user_password' type='password' maxlength='20' size='20'> </td>";
 
		echo "<tr><td colspan='2'><div align='center'> <input name='login_account' type='submit' value='Inloggen!'> </div></td>";
 
		echo "</form>";
		echo "</table>";
	}
}
 
?>

CONFIG.INC.PHP

php code - Bekijk de code zonder highlighting - Klap code in

                
<?PHP

$username[] = "edwin";
$username[] = "edzard";

$password['edwin'] = "terhorst";
$password['edzard'] = "poppema";

$page[] = "home";
$page[] = "login";

?>

<?PHP
 
$username[] = "edwin";
$username[] = "edzard";
 
$password['edwin'] = "terhorst";
$password['edzard'] = "poppema";
 
$page[] = "home";
$page[] = "login";
 
?>

Er zit ook een stukje navigatie script in, maar die laat ik
achterwegen, want dat zit wel KnoR...

2 antwoorden

Gesponsorde links

Voldemort - 28/02/2006 10:21
PHP ver gevorderde		Het is NIET veilig. Je moet je POST inhoud controleren. Als men schadelijke dingen erin typt, en jij zet daarmee allemaal dingen kan men rare JS dingen uithalen of als je SQL gebruikt je hele DB droppen. Dus even htmlentities($var,ENT_QUOTES); over je postinhoud gooien en het zal al veel beter zijn.

ikkedikke - 28/02/2006 10:53
PHP expert		wat is dat voor onzin? hij checkt of de waarde precies dat wachtwoord is, hoe kan er dan javascript in zitten? wat wel zo is dat je ook de waarde van de cookie moet checken voordat iemand zomaar ingelogd is. het is namelijk niet zo moeilijk om een cookie na te maken.

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten