New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Admin] GetIdentity command #2191
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2191 +/- ##
==========================================
- Coverage 57.56% 57.48% -0.09%
==========================================
Files 642 643 +1
Lines 38117 38154 +37
==========================================
- Hits 21943 21933 -10
- Misses 13382 13430 +48
+ Partials 2792 2791 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -1113,6 +1113,8 @@ func (fnb *FlowNodeBuilder) RegisterDefaultAdminCommands() { | |||
return storageCommands.NewReadResultsCommand(config.State, config.Storage.Results) | |||
}).AdminCommand("read-seals", func(config *NodeConfig) commands.AdminCommand { | |||
return storageCommands.NewReadSealsCommand(config.State, config.Storage.Seals, config.Storage.Index) | |||
}).AdminCommand("get-identity", func(config *NodeConfig) commands.AdminCommand { | |||
return common.NewGetIdentityCommand(config.IdentityProvider) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to add example curl
command lines to the README.md
[1] file like the other commands :-)
[1] https://github.com/onflow/flow-go/tree/cd1449feabdaa2be1ebe1c5ccb51eb96f1649b80/admin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving assuming my request for example command lines for the README is okay :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What’s the use case for this command?
Identity is dynamic for each block, but the command doesn’t take block ID.
Is it for identities only in root block? But if so, they can just open the bootstrap file to look up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just left some minor comments.
bors merge |
new command to get identity info by flow ID or libp2p peer ID.