Sicherheit
- install_db.php war ohne Authentifizierung erreichbar und setzte bei jedem
Aufruf das Admin-Passwort auf einen fest im Code stehenden Wert zurück.
Jetzt Auth-Pflicht; ein Konto wird nur bei leerer Benutzertabelle angelegt.
- Stored XSS im Bugtracker-Detail-Modal: Titel, Beschreibung, Fehlermeldung,
Stacktrace und Kommentare gingen ungefiltert durch innerHTML.
- report.php, projects.php und das Veröffentlichen von Releases verlangen jetzt
zwingend ein Token. Publish war zuvor völlig ungeschützt.
- CSRF-Token in allen Formularen, Session-Regenerierung nach Login,
Drosselung fehlgeschlagener Anmeldeversuche.
- Zugangsdaten aus der Versionskontrolle entfernt (Serverdaten.txt,
config.php, .htpasswd, deploy_config.json). Historie enthält sie weiterhin,
Rotation erforderlich (siehe docs/UPGRADE.md).
- Token-Validierung nur noch über SHA-256-Hash; expires_at wird ausgewertet.
Behobene 500er
- Audit::log() war in index.php weder eingebunden noch importiert. Jeder
Klick auf "Aktivierung freigeben" endete in einem Fatal Error.
- Derselbe benannte PDO-Platzhalter mehrfach je Statement (:id in
revokeToken/deleteToken, :q siebenfach in der Volltextsuche). Bei
EMULATE_PREPARES=false ist das nicht zulässig und warf HY093.
- Migration 005 nutzte dynamisches SQL, dessen Semikolons in String-Literalen
vom alten explode(';')-Installer als Statement-Ende gelesen wurden. Sie
schlug still fehl, wodurch push_id/target_agent/tags dauerhaft fehlten.
- Monitor-Umbenennung ohne Transaktion, verschachtelte Transaktionen im
RateLimiter.
Funktionale Korrekturen
- Der Watchdog-Evaluator fehlte vollständig: Monitor-Zustände änderten sich nur
beim Eintreffen eines Heartbeats, ein ausgefallenes System blieb dauerhaft
"up". Erster Lauf auf dem Produktivsystem: 7 von 10 Monitoren waren
tatsächlich seit über einem Tag nicht erreichbar.
- Das Feld "os" fehlte im Monitor-Dialog, wurde aber gespeichert und löschte
damit bei jedem Speichern das Betriebssystem.
- Der Resolve-Dialog existierte im HTML nicht; der Button war funktionslos.
- Versionsvergleich erfolgte lexikografisch, wodurch 1.9.0 als neuer galt
als 1.10.0.
- Schreiboperationen meldeten Erfolg auch für nicht existierende IDs.
- Post/Redirect/Get gegen doppelte Einträge beim Neuladen.
Neue Struktur
- src/bootstrap.php mit PSR-4-Autoloader ersetzt die require-Ketten.
- Core: Config, Http, Csrf, ApiAuth, Logger, Migrator, ErrorReporter.
- Migrator mit zeichenweisem SQL-Parser, dc_migrations und Baseline-Verfahren,
damit bestehende Installationen keine Beispieldaten zurückbekommen.
Agenten-Workflow
- Claim/Lease: Items werden exklusiv übernommen, damit nicht zwei Agenten am
selben Problem arbeiten. action=next holt und reserviert in einem Zug.
- Idempotenz über client_ref, Deduplizierung auch für Feature Requests,
Erkennung von Regressionen, automatische Eskalation des Schweregrads.
- Strukturierter Code-Kontext (repo_url, commit_sha, file_path, line_no).
- Delta-Abfragen über updated_since, Pagination, Bulk-Update.
- Beim Veröffentlichen eines Releases schließen sich Items mit passendem
resolved_in_build selbst.
- Ausgehende Webhooks mit HMAC-Signatur, /api/health, /api/openapi.json.
- Unbehandelte Fehler meldet die Plattform in ihren eigenen Bugtracker.
WebUI
- Serverseitige Filterung mit Pagination statt Rendern aller Datensätze.
- Migrations-Schranke, Evaluator-Warnung, Übersicht aktiver Agenten.
Zeitstempel liegen in der Datenbank durchgängig in UTC und werden für die
Anzeige in die App-Zeitzone umgerechnet.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
94 lines
2.5 KiB
PHP
94 lines
2.5 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Deploymentcenter\Core;
|
|
|
|
/**
|
|
* Minimaler Datei-Logger. Schlaegt das Schreiben fehl, wird still auf
|
|
* error_log() ausgewichen - Logging darf niemals die Anwendung stoppen.
|
|
*/
|
|
final class Logger
|
|
{
|
|
private static ?string $resolvedDir = null;
|
|
|
|
public static function error(string $message, array $context = []): void
|
|
{
|
|
self::write('ERROR', $message, $context);
|
|
}
|
|
|
|
public static function warning(string $message, array $context = []): void
|
|
{
|
|
self::write('WARN', $message, $context);
|
|
}
|
|
|
|
public static function info(string $message, array $context = []): void
|
|
{
|
|
self::write('INFO', $message, $context);
|
|
}
|
|
|
|
public static function exception(\Throwable $e): void
|
|
{
|
|
self::write('ERROR', sprintf(
|
|
'%s: %s in %s:%d',
|
|
get_class($e),
|
|
$e->getMessage(),
|
|
$e->getFile(),
|
|
$e->getLine()
|
|
), ['trace' => $e->getTraceAsString()]);
|
|
}
|
|
|
|
private static function write(string $level, string $message, array $context = []): void
|
|
{
|
|
$line = sprintf(
|
|
"[%s] %s %s%s\n",
|
|
gmdate('Y-m-d H:i:s'),
|
|
$level,
|
|
$message,
|
|
$context !== [] ? ' ' . self::encodeContext($context) : ''
|
|
);
|
|
|
|
$dir = self::dir();
|
|
if ($dir !== null) {
|
|
$file = $dir . '/dc-' . gmdate('Y-m-d') . '.log';
|
|
if (@file_put_contents($file, $line, FILE_APPEND | LOCK_EX) !== false) {
|
|
return;
|
|
}
|
|
}
|
|
|
|
@error_log(rtrim($line));
|
|
}
|
|
|
|
private static function encodeContext(array $context): string
|
|
{
|
|
$json = json_encode($context, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE | JSON_INVALID_UTF8_SUBSTITUTE);
|
|
return $json === false ? '{}' : $json;
|
|
}
|
|
|
|
private static function dir(): ?string
|
|
{
|
|
if (self::$resolvedDir !== null) {
|
|
return self::$resolvedDir === '' ? null : self::$resolvedDir;
|
|
}
|
|
|
|
$dir = defined('DC_VAR') ? DC_VAR . '/log' : null;
|
|
if ($dir === null) {
|
|
self::$resolvedDir = '';
|
|
return null;
|
|
}
|
|
|
|
if (!is_dir($dir) && !@mkdir($dir, 0775, true) && !is_dir($dir)) {
|
|
self::$resolvedDir = '';
|
|
return null;
|
|
}
|
|
|
|
if (!is_writable($dir)) {
|
|
self::$resolvedDir = '';
|
|
return null;
|
|
}
|
|
|
|
self::$resolvedDir = $dir;
|
|
return $dir;
|
|
}
|
|
}
|