0chain / 0chain/zwalletcli

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

未关闭 适合新手
#353 0 条评论 0 个 reaction 已指派 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 摘要。