Rename `client_id` field of the `auth-register` command to `delegate_wallet_id`
- 主要言語
- 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