berhsi/matrix-register
1
0
Fork 1
Code Issues Pull requests Projects Releases Wiki Activity

Bugfix in saveRequest()

Browse source
This commit is contained in:
bernd 2021-03-01 18:12:00 +01:00
parent baafad49ed
commit 5f62b03a4d
1 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
lib/request.php
View file

@ -35,7 +35,7 @@ class Request extends BaseClass {
return false; return false;
} }
$this->log->d("Request started for nick: {$_POST['login']}"); $this->log->i("Request started for nick: {$_POST['login']}");
if (false === $this->checkCaptcha()) { if (false === $this->checkCaptcha()) {
$message = "Captcha invalid"; $message = "Captcha invalid";
return false; return false;
@ -65,12 +65,17 @@ class Request extends BaseClass {
/** /**
* Veranlaßt die Speicherung der Anfrage in der Tabelle requests. * Veranlaßt die Speicherung der Anfrage in der Tabelle requests.
* TODO: Exceptions behandeln. * Bekommt aus der Datenbank (auch im Falle einer PDO Exception)
* einen Boolean zurück.
*/ */
$response = $this->db->saveRequest($this->token); try {
if (!$response) { $response = $this->db->saveRequest($this->token);
return false; } catch (Exception $e) {
$this->log->e("Error: Database returns: {$e->getMessage()}");
}
if ($response === true) {
return true;
} }
return false; return false;
} }
@ -89,6 +94,7 @@ class Request extends BaseClass {
$mailFrom = $this->config->getMailFrom(); $mailFrom = $this->config->getMailFrom();
$mailSubject = $this->config->getMailSubject(); $mailSubject = $this->config->getMailSubject();
$mailClosure = $this->config->getMailClosure(); $mailClosure = $this->config->getMailClosure();
$this->log->d("Try to send verification mail");
$link = $baseurl . $validator . $this->token . "\r\n\r\n"; $link = $baseurl . $validator . $this->token . "\r\n\r\n";
$mailbody = MAILTEXT1 . $mxdomain . MAILTEXT2 . "\r\n\r\n" . $link . $mailClosure; $mailbody = MAILTEXT1 . $mxdomain . MAILTEXT2 . "\r\n\r\n" . $link . $mailClosure;