Files
praxis-creutzburg-web/intern/verify_2fa.php
T
clemensandClaude Opus 5 19f1ac7ec8 CSRF-Schutz, Brute-Force-Bremse und wirksame Session-Cookie-Flags
Neu: inc/security.inc.php, eingebunden von inc/config.inc.php.

CSRF
Der Adminbereich hatte keinerlei Schutz gegen fremde POSTs. Jetzt haengt
csrf_inject_output() als Ausgabefilter das Token an jedes POST-Formular und
csrf_require() weist POSTs ohne gueltiges Token ab. Beides wird nur fuer
Skripte unterhalb von admin/ aktiviert, der oeffentliche Bereich bleibt
unveraendert. Der Umweg ueber den Ausgabefilter erspart es, die rund 60
bestehenden Formulare einzeln anzufassen; der AJAX-Aufruf auf
mailtemplate.php schickt das Token als Feld mit.

Session-Cookie-Flags
config.inc.php setzt secure, httponly und samesite - aber 25 Dateien in
admin/ und intern/ riefen session_start() vor dem Include auf, womit die
Parameter wirkungslos waren. Die vorgezogenen Aufrufe sind entfernt,
config.inc.php startet die Sitzung nur noch, wenn keine laeuft.
admin/logout.php musste umgestellt werden, weil dort session_destroy()
vor dem Include stand; die Cookies werden jetzt mit denselben Parametern
geloescht, mit denen sie gesetzt wurden.

Brute-Force
Nach 5 Fehlversuchen je Konto oder 20 je IP ist die Anmeldung 15 Minuten
gesperrt, gezaehlt in der neuen Tabelle login_attempts. Fehlt die Tabelle,
laeuft der Login wie bisher - gleiche Vorgehensweise wie bei
securitytokensHatAblaufspalte(). Eine erfolgreiche Anmeldung raeumt die
Fehlversuche des Kontos ab.

Passwort vergessen
admin/passwortvergessen.php uebergab $mail und $body an SendMailMessage();
beide Variablen gibt es dort nicht, sie heissen $empfaenger und $text. Die
Mail ging deshalb nie raus, obwohl der Reset-Code gesetzt wurde.

Getestet gegen einen lokalen PHP-Server: Token wird eingesetzt, POST ohne
Token liefert 403, mit Token laeuft der Login normal, der sechste
Fehlversuch wird gesperrt, das Sitzungscookie traegt secure/HttpOnly/
SameSite. Oeffentliche Seiten sind unveraendert.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-03 14:18:59 +02:00

120 lines
4.3 KiB
PHP

<?php
ob_start();
require_once(__DIR__ . "/../inc/config.inc.php");
require_once(__DIR__ . "/../inc/functions.inc.php");
/* ---------- Kein direkter Zugriff ---------- */
if (empty($_SESSION['2fa_userid'])) {
header('Location: login.php');
exit;
}
$error_msg = '';
if ($_SERVER['REQUEST_METHOD'] === 'POST' && !empty($_POST['code'])) {
$userId = (int)$_SESSION['2fa_userid'];
// Aus HTML-Mails kopierte Codes enthalten haeufig Leerzeichen oder
// geschuetzte Leerzeichen. Der Code besteht immer aus sechs Ziffern.
$eingabe = preg_replace('/\D/', '', (string)$_POST['code']);
$codeHash = hash('sha256', $eingabe);
// Der Ablauf wird in SQL geprueft (NOW()), nicht in PHP: der Webserver
// und der DB-Server stehen bei verschiedenen Anbietern und koennen in
// unterschiedlichen Zeitzonen laufen. Wuerde die Ablaufzeit in PHP
// erzeugt und gegen NOW() verglichen, waere jeder Code sofort
// "abgelaufen". Ablauf und Tippfehler werden getrennt gemeldet.
$stmt = $pdo->prepare("
SELECT id, (expires_at > NOW()) AS noch_gueltig
FROM intern_2fa_codes
WHERE user_id = :uid
AND code = :code
LIMIT 1
");
$stmt->execute([
'uid' => $userId,
'code' => $codeHash
]);
$row = $stmt->fetch(PDO::FETCH_ASSOC);
if ($row && $row['noch_gueltig']) {
// Code löschen (Einmalverwendung)
$pdo->prepare("DELETE FROM intern_2fa_codes WHERE user_id = :uid")
->execute(['uid' => $userId]);
session_regenerate_id(true);
$_SESSION['userid'] = $userId;
$_SESSION['2fa_verified'] = true;
unset($_SESSION['2fa_userid']);
/* ---------- Gerät merken ---------- */
if (!empty($_POST['remember_device'])) {
$identifier = bin2hex(random_bytes(32));
$token = bin2hex(random_bytes(32));
$pdo->prepare("
INSERT INTO intern_securitytokens
(user_id, identifier, securitytoken, expires_at)
VALUES (:uid, :identifier, :token, :expires)
")->execute([
'uid' => $userId,
'identifier' => $identifier,
'token' => hash('sha256', $token),
'expires' => date('Y-m-d H:i:s', time() + 30 * 24 * 3600)
]);
$opts = [
'expires' => time() + 30 * 24 * 3600,
'path' => '/',
'secure' => true,
'httponly' => true,
'samesite' => 'Lax'
];
setcookie('remember_device', $identifier, $opts);
setcookie('remember_device_token', $token, $opts);
}
header('Location: index.php');
exit;
}
$error_msg = $row
? 'Der Code ist abgelaufen. Bitte melden Sie sich erneut an, damit wir Ihnen einen neuen Code schicken können.'
: 'Der eingegebene Code war nicht korrekt.';
}
include("templates/header.inc.php");
?>
<div class="container small-container-330 form-signin">
<form action="verify_2fa.php" method="post">
<h2 class="form-signin-heading">Zwei-Faktor-Authentifizierung</h2>
<p>Die Praxis Creutzburg sichert diesen Zugang mit einer Zwei-Faktor-Authentifizierung ab.<br>
Sie benötigt neben Ihren Passwort auch den zweiten Faktor: eine zufällig generierte Nummer, die an Ihre E-Mail-Adresse versendet wurde.<br>
Der zweite Faktor ist 5 Minuten gültig.<br>
Prüfen Sie bitte jetzt Ihr Postfach und auch Ihren Spam-Ordner auf eine E-Mail von uns und geben Sie hier die sechsstellige Nummer ein.<br></p>
<?php if(!empty($error_msg)) echo $error_msg; ?>
<label for="inputCode">6-stelliger Code</label>
<input type="text" name="code" class="form-control" placeholder="Code eingeben" required>
<br>
<div class="checkbox">
<label>
<input type="checkbox" name="remember_device" value="1"> Dieses Gerät für 30 Tage merken
</label>
</div>
<button class="btn btn-lg btn-primary btn-block" type="submit">Bestätigen</button>
</form>
</div>
<?php include("templates/footer.inc.php"); ?>