Login script

[Xor]

Schick uns mal den kompletten Code. Alles was du mit $passwd und $name anstellst.

[vt1816]

Da sich sowohl Deine Datenbankstruktur und als auch Dein Code geändert haben, ohne das Du uns daran teilhaben läßt, können wir Dir im Moment nicht weiter helfen - sorry!

[level7]

so jetzt entschuldigung,

hab jetzt tabelle in meiner db mit id name passw und email

id funktioniert
name funktioniert
passw wird nichts angezeigt
email funktioniert

also ich weiß echt nicht woran es mit dem passw liegt:

PHP-Code:

mt_srand((double)microtime()*1000000);
$zahl = mt_rand(1000, 9999);

$passzahl = md5($zahl)
$passw = substr($passzahl, "0" ,"6"); 


PHP-Code:

  <?php
    include("db/test.php");

  $name = $_POST["name"];
  $passw = $_POST["passw"];
  $email = $_POST["email"]

  $eintrag = "INSERT INTO lotex (name, passw, email) VALUES ('$name', '$passw', '$email')";
  $eintragen = mysql_query($eintrag) or die(mysql_error());
  ?>

[vt1816]

Schalte mal die Fehlerbehandlung ein:

PHP-Code:

ini_set ('error_reporting', E_ALL);
error_reporting(E_ALL); 


Es fehlen nähmlich einige ;.

Lass Dir mal vor dem eintragen $eintrag (echo $eintrag;) ausgeben. Vermute das $passw leer ist. Denn in Deiner Entwicklungsdatenbank funktioniert die Übergabe mit Deinem INSERT.

[Xor]

Steht in $passw der erwünschte Wert nachdem du ihn aus der $_POST gehohlt hast?

wenn nicht solltest du im Formular ein

<input type="hidden" name="passw" value="<?echo $passw;?>">

einfügen...

[Socrates]

Als erstes solltest du mt_rand() durch rand() ersetzen. Zudem musst du folgende Änderung vornehmen:

PHP-Code:

$eintrag = "INSERT INTO lotex (`id`, `name`, `passw`, `email`) VALUES ('', '{$name}', '{$passw}', '{$email})"; 


Des Weiteren fällt mir auf, dass der erste Teil fragwürdig ist, da du so niemals das Passwort eines Nutzers überprüfen kannst. Ändere es wie folgt ab:

PHP-Code:

$abc = range(a, z);
$num = range(0,255);
$mix = array();
array_push ($mix  , $abc, $num)
shuffle($mix);

$password = '';
for(i=0; i<=6; i++)
{
    $password .= $mix($i);
}

$passw = md5($password); 


Du kannst nicht einfach nur die ersten 6 Zeichen des MD5-Codes in die DB aufnehmen! Zudem würde ich es für sinnvoller halten, den User ein eigenes Passwort wählen zu lassen.
MfG, Andy

[level7]

ich probiers mal aus aber ich bezweifle dass $passw leer ist denn es wird ja auch automatisch eine e-mail an den neuen user geschickt wo das passwort einwandfrei drinsteht und da hab ich auch los so geschrieben:

Zeile:

Ihr zugangskennwort: $passw

[vt1816]

Zitat von Socrates

... Zudem musst du folgende Änderung vornehmen:

PHP-Code:

$eintrag = "INSERT INTO lotex (`id`, `name`, `passw`, `email`) VALUES ('', '{$name}', '{$passw}', '{$email})"; 


Muss er nicht, da es sich um ein auto_increment handelt!

[Socrates]

Sein passwort ist nicht $passw. Das ist ja das Problem. Du sollst zuerst dass Passwort generieren. Das kannst du dann dem Benutzer ausgeben. Danach codierst du es mit md5() und schreibst es in die DB. Wenn du nun das Passwort überprüfen willst codierst du die User-Eingabe und vergleichst das Ergebnis mit dem Eintrag in der DB. Das sorgt dafür, dass niemand außer der User das Passwort kennt. Sollte ein Hacker an den Eintrag in der DB kommen, hilft ihm das recht wenig, da das Passwort, wenn er es eingibt wieder verschlüsselt wird und so etwas vollkommen anderes ergibt (ist ja dann doppelt verschlüsselt).

@vt1816 Du hast recht! Sorry!
MfG, Andy

[level7]

so hab jetzt gemacht dass man sein pw selbst eingeben kann und so funktioniert es jetzt


also schicke ich ihm das pw dann nochmal zu und nach der e-mail das pw per md5 verschlüsseln und in db eintragen?