WBCE CMS Forum

creatore

Mitglied

Fehler beim Registrierungsvorgang - signup2.php

Hallo,

WBCE 1.1.6 - Registrierung aktiviert

Beim Registrierungsvorgang wird jedes Feld korrekt überprüft (schon vorhandene Nutzernnamen, E-Mail etc.)
Das Scheitern der Überprüfung hat jedoch keine Konsequenzen für die Abarbeitung des Scriptes (!!), will heißen, dass der neue Nutzer trotzdem in die Datenbank geschrieben wird und eine Bestätigungsmail verschickt wird, samt Zugangsdaten.

D.h. unter Umständen, doppelte Nutzernamen oder Mailadressen mit allen Konsequenzen (ouchhh!)

Reicht es, allen Fehlermeldungen ein exit() im Script folgen zu lassen? (s.u.)
Damit funktioniert bei mir soweit alles korrekt.

Gruß creatore

[== PHP ==]
// Captcha
if(ENABLED_CAPTCHA) {
	$MESSAGE['MOD_FORM_INCORRECT_CAPTCHA'] = str_replace($search,$replace,$MESSAGE['MOD_FORM_INCORRECT_CAPTCHA']);
	if(isset($_POST['captcha']) AND $_POST['captcha'] != ''){
		// Check for a mismatch
		if(!isset($_POST['captcha']) OR !isset($_SESSION['captcha']) OR $_POST['captcha'] != $_SESSION['captcha']) {
			$wb->print_error($MESSAGE['MOD_FORM_INCORRECT_CAPTCHA'], $js_back, false);
			exit(); // LIEGE ICH HIER RICHTIG ???
		}
	} else {
		$wb->print_error($MESSAGE['MOD_FORM_INCORRECT_CAPTCHA'], $js_back, false);
		exit(); // LIEGE ICH HIER RICHTIG ???
	}
}

florian

Administrator

Re: Fehler beim Registrierungsvorgang - signup2.php

Urgh. Kann ich bestätigen. Danke für den Hinweis, ich gebe das sofort weiter.

---

Wir Benötigen: Cents, Euros... jetzt spenden!

norhei

Developer

Re: Fehler beim Registrierungsvorgang - signup2.php

Ich erinner mich vage den automatischen exit;  in der Methode print_error mal rausgeworfen zu haben.
Im BE hat dieser Exit öffter mal einiges kaputtgehauen.

Insofern ist der Vorschlag  von  creatore auch gleich eine Lösung.

Persönlich würde ich aber etwas weitergehen und die signup2.php etwas umbauen.

Ist jetzt auf die Schnelle zusammen gebaut, leider habe ich hier grade keine Möglichkeit zum testen.

Einfach in /account/ die signup2.php mit der hier angehängten ersetzen.

Wenn die alles richtig macht kommts direkt ins GIT und wird als  Patch veröffentlicht..

Patch war mist , nicht benutzen

florian

Administrator

Re: Fehler beim Registrierungsvorgang - signup2.php

So, ich denke wie von creatore schon vorausgedacht sollte es reichen, großzügig ein paar "exit" zu ergänzen.

Bitte mal testen: Die angehängte Datei bitte in signup2.php umbenennen und die vorhandene unter /account damit überschreiben.

---

Wir Benötigen: Cents, Euros... jetzt spenden!

norhei

Developer

Re: Fehler beim Registrierungsvorgang - signup2.php

Wenn sich das erfolgreich testet kommts gleich ins git :-)