[PHP-MySQL] MD5

[Paradoxia]

Hallo mensen,

Ik heb even een kort vraagje (hoop ik). Ik heb de paswoorden van mijn gebruikers opgeslagen en encrypted in MD5..
Gaat goed, maar nu wil ik via een ander login script. Nu heb ik dit:

code:

<?php

include("config.php");

$connect = mysql_connect("$user_hostname", "$user_username", "$user_password");
mysql_select_db("$user_database", $connect);

session_start();

if(!isset($username)) {

echo "<form method=\"POST\" action=$PHP_SELF>
<center>
<table>
<tr>
<td>Username:</td>
<td><input type=\"text\" name=\"username\" size=\"20\"></td>
</tr>
<tr>
<td>Password:</td>
<td><input type=\"password\" name=\"password\" size=\"20\"></td>
</tr>
<tr>
<td colspan=\"2\">
<p align=\"center\"><input type=\"submit\" value=\"Submit\" name=\"submit\"></td>
</tr>
</table>
</center>
</form>";
exit;
}

session_register("username");
session_register("password");

$sql = "SELECT * FROM _users WHERE username = \"$username\" AND password = \"$password\"";
$result = @mysql_query($sql) or die("No.");

if(mysql_num_rows($result) == "0") {

session_unregister("username");
session_unregister("password");

echo "<h2 align=center>Wrong username and password, try again</h2>";
exit;

}

$username = mysql_result($result,0,"username");

mysql_close($connect);

?>

Dit paswoord is niet encrypted. Waar zet ik de md5 tag in dit script?

ik had zelf dit in gedachten:

code:

session_register(md5("password"));

Maar ik weet niet of dit gaat werken. En aangezien ik al info in de DB heb staan kan ik het niet 123 testen.

Wees gegroet,
/Eddy

[Verwijderd]

Paradoxia schreef op 28 november 2003 @ 00:33:

code:

$sql = "SELECT * FROM _users WHERE username = \"$username\" AND password = \"$password\"";

>> Bedoel je niet

code:

$sql = "SELECT * FROM _users WHERE username = \"$username\" AND password = \"" . md5($password) . "\"";

En aangezien je password al encrypted in je database staat, haal je'm toch ook encrypted eruit. Ben niet echt mee wat nou het probleem is

[Paradoxia]

Ben ik nu echt zo stupide aan het worden? Het zal wel de kilometers lange <? ... ?> zijn die me door me kop spookt. *sigh*

iig hartelijk bedankt

[.oisyn]

Je kan ook gewoon dat hele wachtwoord gebeuren in de query weglaten, en controleren op het moment dat je de userdata hebt opgehaald

[Verwijderd]

Misschien meer een kanttekening dan een antwoord:

Het is nooit verstandig om persoonlijk / kritieke informatie (zoals in jou geval een wachtwoord) op te slaan in een sessie. Je kunt toch ook gewoon een boolean setten een daarin aangeven dat de gebruiker is ingelogd samen met zijn username?
Als je het wachtwoord een paar scripts verder op dan nog een keer nodig hebt dan haal je em opnieuw op met een query.

[Verwijderd]

Ik zou er trouwens dit van maken (netter, handiger en sneller):

code:

<?php

include("config.php");

$connect = mysql_connect($user_hostname, $user_username, $user_password);
mysql_select_db($user_database, $connect);

session_start();

if(!isset($username)) {

?>
<form method="POST" action="<?=$PHP_SELF;?>">
<center>
<table>
<tr>
<td>Username:</td>
<td><input type="text" name="username" size="20"></td>
</tr>
<tr>
<td>Password:</td>
<td><input type="password" name="password" size="20"></td>
</tr>
<tr>
<td colspan="2">
<p align="center"><input type="submit" value="Submit" name="submit"></td>
</tr>
</table>
</center>
</form>
<?
exit;
}

session_register("username");

$sql = "SELECT * FROM _users WHERE username = '".$username."' AND password = MD5('".$password."')";
$result = @mysql_query($sql) or die("No.");

if(mysql_num_rows($result) == "0") {

session_unregister("username");

?><h2 align="center">Wrong username and password, try again</h2><?
exit;
}

$username = mysql_result($result,0,"username");

mysql_close($connect);

?>

Waarom sla je het password op in een session variabele?
Zelfde vraag dus als 6-Pack

[ Voor 51% gewijzigd door Verwijderd op 28-11-2003 16:05 ]

[Explore]

Zie evt. ook dit topic.