[#1515] adm: Print address in base58 format in morph ape get-admin #1520

Merged
fyrchik merged 1 commit from george.bartolomey/frostfs-node:1-admin-base58 into master 2024-11-25 10:38:06 +00:00

Signed-off-by: George Bartolomey george@bh4.ru

Closes: #1515

Signed-off-by: George Bartolomey <george@bh4.ru> Closes: #1515
george.bartolomey added 1 commit 2024-11-23 14:28:01 +00:00
[#1515] adm: Print address in base58 format in morph ape get-admin
All checks were successful
DCO action / DCO (pull_request) Successful in 2m33s
Tests and linters / Run gofumpt (pull_request) Successful in 2m55s
Vulncheck / Vulncheck (pull_request) Successful in 2m55s
Tests and linters / Staticcheck (pull_request) Successful in 3m2s
Build / Build Components (pull_request) Successful in 3m29s
Pre-commit hooks / Pre-commit (pull_request) Successful in 3m28s
Tests and linters / Lint (pull_request) Successful in 3m49s
Tests and linters / gopls check (pull_request) Successful in 4m7s
Tests and linters / Tests (pull_request) Successful in 4m29s
Tests and linters / Tests with -race (pull_request) Successful in 4m47s
4ea6cbbf61
Signed-off-by: George Bartolomey <george@bh4.ru>
fyrchik requested changes 2024-11-25 06:53:52 +00:00
Dismissed
@ -201,3 +202,3 @@
func setAdmin(cmd *cobra.Command, _ []string) {
s, _ := cmd.Flags().GetString(addrAdminFlag)
addr, err := util.Uint160DecodeStringLE(s)
addrBytes, err := base58.Decode(s)
Owner

Please, look at the address package.
https://github.com/nspcc-dev/neo-go/blob/master/pkg/encoding/address/address.go

We shouldn't use base58 package directly here.

Please, look at the `address` package. https://github.com/nspcc-dev/neo-go/blob/master/pkg/encoding/address/address.go We shouldn't use base58 package directly here.
Owner

Please, don't resolve comments yourself, it is a burden of a reviewer.

Please, don't resolve comments yourself, it is a burden of a reviewer.
fyrchik marked this conversation as resolved
george.bartolomey force-pushed 1-admin-base58 from 4ea6cbbf61 to ceb916ce55 2024-11-25 08:15:08 +00:00 Compare
george.bartolomey requested review from fyrchik 2024-11-25 08:17:52 +00:00
aarifullin approved these changes 2024-11-25 08:30:46 +00:00
aarifullin left a comment
Member

👍

👍
Member

Please update commit message accordingly.

Please update commit message accordingly.
fyrchik approved these changes 2024-11-25 10:37:31 +00:00
fyrchik merged commit 0e5524dac7 into master 2024-11-25 10:38:06 +00:00
george.bartolomey deleted branch 1-admin-base58 2024-11-25 10:41:57 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#1520
No description provided.