mal wieder java-script Problem

GreenRover

Code

function passwordCheck() {
     var password1 = document.getElementById('password').value;
     var passwordlength = document.getElementById('password').length;
     var password2 = document.getElementById('repeat').value;


     if (password1!=password2) {
      alert('Die von ihnen angegebenen Passwörter sind nicht identisch');
	  document.getElementById('password').value="";
	  document.getElementById('repeat').value="";
	  document.getElementById('repeat').focus();
     }
     elseif (passwordlength<4)
     {
      alert('Die von ihnen angegebenen Passwrter ist zu kurtz  ( min 4 zeichen )');
	  document.getElementById('password').value="";
	  document.getElementById('repeat').value="";
	  document.getElementById('repeat').focus();     
     }
     else
     {
		try {
			document.useradminForm.onsubmit();
		    }
		catch(e){}
		document.useradminForm.submit();
     }


}

Alles anzeigen

ichfinde den Fehelr nicht!! der debugger sagt immer nur

Fehler: syntax error
Quelldatei: https://config.local/template/functions.js
Zeile: 52, Spalte: 5
Quellcode:
else

Ronald

Moin!

In deinem Script sind gleich mehrere Fehler.

1. Dein Hauptproblem war, daß du bei der Überprüfung der Länge des eingegebenen Passwortes vergessen hast, daß "value" zu notiern. Du hast dmit nur die Länge des Input-Tag geprüft un der ist immer "undefined" und damit kleiner als 4 (weil überhaupt kein numerischer Wert).

2. Schönheitsfehler: das Einlesen der PW-Länge in eine eigene Variable ist überflüssig, da es genau 1x benötigt wird. Einfacher kann man die bereits eingelesene Variable "password1" auf ihre Länge überprüfen.

3. Eine zusätzliche Fehlerüberwachung mit "try" ist überflüssig, wenn man die Überprüfung generell erst beim Absenden des Formulars aufruft, siehe mein Beispiel.

4. ein einigermaßen sicheres Passwort sollte mindesten 6 Zeichen haben, besser 8-12.
4 Zeichen sind zu schnell zu knacken.

Im folgenden Beispiel gehe ich davon aus, daß du die Eingabe an ein PHP-Script übergibst. Ich weiß nicht wie dein PHP-Script heißt und habe es im Beispiel einfach einmal "useradmin.php" genannt, das mußt du ggf. noch anpassen.
Generell halte ich es übrigens für besser, die Überprüfung durch das aufzurufende PHP-Script zu machen und dann im Fehlerfall auf die Eingaberoutine zurück zu verweisen.

Quelltext:

HTML

<html>
<head>
<script type="text/javascript" language="JavaScript">
<!--
function passwordCheck()
     {
     var password1 = document.getElementById('password').value;
     var password2 = document.getElementById('repeat').value;


     if (password1!=password2)
        {
        alert('Die von ihnen angegebenen Passwörter sind nicht identisch');
        document.getElementById('password').value="";
        document.getElementById('repeat').value="";
         document.getElementById('password').focus();
         return false;
         }


     if (password1.length < 6)
         {
         alert('Das von ihnen angegebene Passwort ist zu kurz  ( min. 6 Zeichen )');
         document.getElementById('password').value="";
         document.getElementById('repeat').value="";
         document.getElementById('password').focus();
         return false;
         }
     }
//-->
</script>
</head>
<body>
<form name="useradminForm" methode="post" action="useradmin.php" onSubmit="return passwordCheck()">
<input name="password" id="password" type="password"></input>
<input name="repeat" id="repeat" type="password"></input>
<input type="submit" value="Abschicken"></input>
</form>
</body>
</html>

Alles anzeigen

Gg.a.d.M.

Ronald
... der Javascript trotzdem haßt!

GreenRover

danke üfr die überarbeitung. aber das action lasse ich generell lehr da das ganze ien größeres cms ist und alels an die index.php übergeben wird.

Und die 4 Zeichen sicherheits-politk reicht shcon. da man dieses sicher nict ebi 12 versuchen raus bekommen.

PS. GL HF beim bruteforce wenn das PW z.B. lautet: "F*1\g" das dauert dann auch schon so seine Zeit.

Ronald

Zitat von GreenRover

danke üfr die überarbeitung. aber das action lasse ich generell lehr da das ganze ien größeres cms ist und alels an die index.php übergeben wird.

... was dann zur Folge hat, daß bei abgeschaltetem Javascript dein Login nicht mehr funktioniert. Außerdem mußt du dir mit Javascript eine Krücke bauen, anstatt korrekt (W3-Conform) zu coden. Und wie ich schon schrieb: normalerweise macht man diese Überprüfung mit PHP.

Zitat von GreenRover

Und die 4 Zeichen sicherheits-politk reicht shcon. da man dieses sicher nict ebi 12 versuchen raus bekommen.

Ich hab da so meine Zweifel, da die User sich ihre Passworte sicherlich selbst bestimmen und die meisten normalen User über die Sicherheitsproblematik nicht informiert sind, dann Passworte wie "1234" oder "asdf" benutzen. Die sind dann recht schnell zu knacken.

Zitat von GreenRover

PS. GL HF beim bruteforce wenn das PW z.B. lautet: "F*1\g" das dauert dann auch schon so seine Zeit.

Das sind ja schon einmal 5 Zeichen

Igrendwie verstehe ich dich nicht, aber was soll's

Ronald

GreenRover

Ja das wies ich auch das man die Überprüfung normaler weise in PHP macht. Finde es so für den User aber angenehnehmer. Werde vileicht auch noch eine Doppel Überprüfung machen ( also mit PHP auch noch ). aber mal gucken da ic hdoch sehr unter Zeitdruck stehe.

Tags