DefGuard/defguard

Replace user id in logs with username

Offen

#719 geöffnet am 13.08.2024

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (107 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (2.779 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

If accessible, a username should be displayed in logs (info level) at all times in favor of the user id. If the user object is not accessible, a minimal sql query should be made to access it. This may be also expanded to other objects we refer to by id, which may be confusing for the end user.

Contributor Guide

Research-Richtung
Lokalisieren Sie die Logging Anweisungen, die user id verwenden, und bestimmen Sie, wie auf den Benutzernamen zugegriffen werden kann. Wenn das Benutzerobjekt bereits verfügbar ist, ersetzen Sie die ID durch den Benutzernamen. Wenn nicht, fügen Sie eine minimale SQL Abfrage hinzu, um den Benutzernamen aus der Datenbank abzurufen. Stellen Sie sicher, dass die Logging Stufe info bleibt und Fälle, in denen der Benutzer nicht gefunden wird, angemessen behandelt werden.
Tech Stack
rustsql
Domain
backenddatabase
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitRust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60