fix(admin): Laenge des Seed-Passworts loggen, $-Falle dokumentieren
Ein $ im Wert wird von Docker Compose als Variable gelesen und schneidet ab: Sicher$Passwort2026 kommt als Sicher an (gemessen). Angelegt wird ein Konto mit einem Passwort, das niemand kennt; die Anmeldung scheitert mit 'invalid credentials', obwohl alles richtig eingetippt ist. Die Datenbank laeuft dabei weiter, weil Postgres denselben abgeschnittenen Wert bekommt. Der Start nennt jetzt die Laenge des Passworts (nicht den Wert) und warnt unter 12 Zeichen mit dem Hinweis auf $$. Ohne diese Zahl war der Fehler von aussen nicht zu sehen: Compose meldet zwar 'variable is not set', aber auf stderr beim up, nicht im Container-Log. Escape-Tabelle in README und beiden .env.example. Ein Backslash hilft nicht. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
1 parent
370d8ce0c7
commit
218b238afc
5 files changed
+92
-1
No files matched your search
@@ -257,7 +257,29 @@ func main() {
|
||||
if err := seedAdmin(ctx, authStore, cfg.AdminUsername, cfg.AdminPassword); err != nil {
|
||||
log.Warn().Err(err).Msg("admin seed skipped")
|
||||
} else {
|
||||
log.Info().Str("username", cfg.AdminUsername).Msg("admin account ready")
|
||||
// Die **Länge** mit ins Log, nicht das Passwort.
|
||||
//
|
||||
// Anlass: Ein `$` in der .env verschluckt bei Docker Compose
|
||||
// lautlos den Rest des Wertes – `geheim$wort` kommt als `geheim`
|
||||
// an. Angelegt wird dann ein Konto mit einem Passwort, das
|
||||
// niemand kennt, und die Anmeldung scheitert mit „invalid
|
||||
// credentials", obwohl alles richtig eingetippt ist. Gemessen,
|
||||
// und im Log war nichts davon zu sehen.
|
||||
//
|
||||
// Eine Zahl verrät nichts Brauchbares, macht den verschluckten
|
||||
// Wert aber sofort sichtbar.
|
||||
log.Info().
|
||||
Str("username", cfg.AdminUsername).
|
||||
Int("password_length", len(cfg.AdminPassword)).
|
||||
Msg("admin account ready")
|
||||
if len(cfg.AdminPassword) < 12 {
|
||||
log.Warn().
|
||||
Int("password_length", len(cfg.AdminPassword)).
|
||||
Msg("ADMIN_PASSWORD ist auffaellig kurz – enthaelt es ein $? " +
|
||||
"In der .env muss es als $$ geschrieben oder der Wert in " +
|
||||
"'einfache Anfuehrungszeichen' gesetzt werden, sonst " +
|
||||
"schneidet Docker Compose dort ab.")
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in new issue
Block a user