0chain / 0chain/zwalletcli

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

Đang mở Phù hợp với người mới
#353 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
post-mainnet
Ngôn ngữ chính
Go
Star
21
Fork
19
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Tìm lệnh `auth-register` trong codebase, có thể nằm trong thư mục lệnh hoặc cmd. Xác định trường có tên `client_id` và đổi tên thành `delegate_wallet_id`. Cập nhật mọi xác thực hoặc cách sử dụng liên quan trong lệnh và logic xây dựng giao dịch cho `add_authorizer`. Chạy các bài kiểm tra hiện có để đảm bảo thay đổi không làm hỏng chức năng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
go
Lĩnh vực
cli
Loại issue
Tái cấu trúc
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
70/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.