0chain / 0chain/zwalletcli

Rename `client_id` field of the `auth-register` command to `delegate_wallet_id`

Offen Anfängerfreundlich
#353 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
post-mainnet
Vorherrschende Sprache
Go
Sterne
21
Forks
19
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

`client_id` field of the `auth_register` is used in the `DelegateWallet` field of the `add_authorizer` transaction, while `client_key` is used in the `PublicKey` field of the txn. They need to be not in correspondence with one another, since this will mean that the authorizer is trying to self-delegate (i.e. use the same wallet for operations and delegation), which is not allowed.
This field should be renamed to `delegate_wallet_id` to be clear that this will not correspond to the `client_key`.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Finden Sie den Befehl `auth-register` im Codebase, wahrscheinlich in einem Verzeichnis für Befehle oder cmd. Suchen Sie das Feld namens `client_id` und benennen Sie es in `delegate_wallet_id` um. Aktualisieren Sie alle damit verbundenen Validierungen oder Verwendungen im Befehl und in der Transaktionserstellungslogik für `add_authorizer`. Führen Sie bestehende Tests aus, um sicherzustellen, dass die Änderung keine Funktionalität beeinträchtigt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
cli
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
70/100

Neue Issues direkt in Ihr Postfach

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