0chain / 0chain/zwalletcli

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

Aperta Adatta ai principianti
#353 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
post-mainnet
Lingua principale
Go
Stelle
21
Fork
19
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

`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`.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Trova il comando `auth-register` nella codebase, probabilmente in una directory di comandi o cmd. Individua il campo chiamato `client_id` e rinominalo in `delegate_wallet_id`. Aggiorna qualsiasi validazione o utilizzo correlato nel comando e nella logica di costruzione della transazione per `add_authorizer`. Esegui i test esistenti per assicurarti che la modifica non comprometta la funzionalità.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
go
Ambito
cli
Tipo di issue
Refactoring
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
70/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.