0chain / 0chain/zwalletcli

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

Ouverte Adaptée aux débutants
#353 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
post-mainnet
Langage dominant
Go
Étoiles
21
Forks
19
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Trouvez la commande `auth-register` dans la base de code, probablement dans un répertoire de commandes ou cmd. Localisez le champ nommé `client_id` et renommez-le en `delegate_wallet_id`. Mettez à jour toute validation ou utilisation associée dans la commande et la logique de construction de transaction pour `add_authorizer`. Exécutez les tests existants pour vous assurer que le changement ne casse pas la fonctionnalité.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
go
Domaine
cli
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
70/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.