Admin-Login lokal: 2FA-PIN ohne Mailversand (Dev) + Mailer-Fehler abfangen

Problem: lokal konnte man sich nicht als Admin anmelden — der 2FA-PIN kam nie
an (kein SMTP), und Mailer::sendAdminPin() warf sogar eine Exception, die den
Login komplett abbrach (leere Antwort → „MFA fehlt").

- admin.php: Mailversand nur auf Prod + in try/catch (Mail-Fehler darf den
  Login NIE blocken). Lokal (!IS_PRODUCTION) wird der PIN als `devPin` in der
  Login-Antwort zurückgegeben — auf Prod NIEMALS (2FA bleibt Pflicht).
- admin.html: devPin wird lokal automatisch ins PIN-Feld vorausgefüllt + als
  Hinweis angezeigt.

Verifiziert (temp Test-Admin): login → devPin → verify → authenticated:true,
admin-weltkueche.html erreichbar. Prod unberührt (IS_PRODUCTION-Gate).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
2026-08-19 19:13:22 +02:00
parent 0555a2d245
commit 5b91a5bf84
2 changed files with 22 additions and 4 deletions
+8
View File
@@ -83,6 +83,14 @@ async function doLogin() {
document.getElementById('step-login').classList.add('hidden');
document.getElementById('step-pin').classList.remove('hidden');
document.getElementById('msg').className = 'msg';
// Dev-Umgebung (localhost): PIN kommt in der Antwort mit → direkt vorausfüllen,
// damit man sich ohne Mailversand anmelden kann. Auf Prod gibt es kein devPin.
if (r.devPin) {
document.getElementById('a-pin').value = r.devPin;
document.getElementById('pin-email').textContent = 'Dev-Umgebung';
var hint = document.querySelector('#step-pin .step-hint');
if (hint) hint.innerHTML = '🔧 <b>Dev:</b> kein Mailversand — PIN <b>' + r.devPin + '</b> ist vorausgefüllt. Auf „Bestätigen".';
}
document.getElementById('a-pin').focus();
}
+14 -4
View File
@@ -32,9 +32,12 @@ if ($action === 'login') {
[$admin['id'], $pin]
);
// PIN per E-Mail senden
if (class_exists('Mailer')) {
Mailer::sendAdminPin($admin['email_2fa'], $pin);
// PIN per E-Mail senden — Fehler dürfen den Login NIE abbrechen (lokal geht
// kein SMTP raus; früher warf das hier und die Anmeldung schlug fehl → „MFA
// fehlt"). Lokal ohnehin überspringen, da wir den PIN als devPin zurückgeben.
if (IS_PRODUCTION && class_exists('Mailer') && !empty($admin['email_2fa'])) {
try { Mailer::sendAdminPin($admin['email_2fa'], $pin); }
catch (\Throwable $e) { error_log('[admin] Mailer::sendAdminPin fehlgeschlagen: ' . $e->getMessage()); }
}
// Admin-ID in Session speichern (aber noch nicht authentifiziert)
@@ -42,7 +45,14 @@ if ($action === 'login') {
$_SESSION['admin_pending'] = (int)$admin['id'];
unset($_SESSION['admin_id']); // Sicherstellen dass nicht schon eingeloggt
Response::ok(['message' => 'PIN wurde an Ihre E-Mail gesendet.', 'email' => maskEmail($admin['email_2fa'])]);
$resp = ['message' => 'PIN wurde an Ihre E-Mail gesendet.', 'email' => maskEmail($admin['email_2fa'])];
// Dev-Umgebung (localhost): es geht keine Mail raus → PIN direkt zurückgeben,
// damit man sich lokal anmelden kann. Auf Prod (IS_PRODUCTION) NIEMALS.
if (!IS_PRODUCTION) {
$resp['devPin'] = $pin;
$resp['message'] = 'Dev-Umgebung kein Mailversand. PIN steht unten.';
}
Response::ok($resp);
}
// === VERIFY PIN (Schritt 2: PIN eingeben → Admin-Session) ===