0chain / 0chain/zwalletcli

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

オープン 初心者向け
#353 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
post-mainnet
主要言語
Go
スター
21
フォーク
19
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

コードベース内の `auth-register` コマンドを探します。おそらく commands または cmd ディレクトリにあります。`client_id` という名前のフィールドを見つけ、`delegate_wallet_id` に名前を変更します。コマンド内および `add_authorizer` のトランザクション構築ロジックに関連する検証や使用箇所を更新します。既存のテストを実行して、変更が機能を損なわないことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
go
領域
cli
issue の種類
リファクタリング
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
70/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。