linux-credentials / linux-credentials/credentialsd

Serialize all D-Bus fields and enum values with snake case

Offen
#82 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

help wanted
Vorherrschende Sprache
Rust
Sterne
259
Forks
19
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
12

Beschreibung

We borrow a lot of fields and values from WebAuthn. For internal consistency and consistency with other D-Bus APIs, I think we should convert them to snake_case instead of camelCase, as is more common in JavaScript.

This includes the `publicKey` field of create/get credential requests, `topOrigin` (#81), etc.

This is a breaking change to the API.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die D-Bus-API-Definitionen für Anfragen zum Erstellen/Abrufen von Anmeldedaten sowie die hier erwähnten Felder und Enum-Werte, einschließlich publicKey und topOrigin, zu lokalisieren. Prüfe, wie ihre serialisierten Namen festgelegt sind, und verifiziere anschließend, dass jedes betroffene Feld und jeder Enum-Wert snake_case verwendet und dass die daraus resultierende inkompatible API-Änderung durch die im Projekt verfügbaren Prüfungen abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
api, authentication
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.