ockam Unify error handling, node req/res handling, and output

Unify error handling, node req/res handling, and output

Open adrianbenavides opened this issue 3 years ago • 2 comments

[x] Develop a set of helpers to simplify commands logic (PR)
[ ] Apply to configuration commands
[ ] Apply to enroll commands
[x] Apply to forwarder commands
[ ] Apply to identity commands
[ ] Apply to message commands
[ ] Apply to node commands
[x] Apply to project CRUD commands
[ ] Apply to project enroller/member/credential commands
[x] Apply to secure_channel commands
[ ] Apply to secure_channel_listener commands
[ ] Apply to service commands
[x] Apply to space commands (PR)
[ ] Apply to tcp commands
[ ] Apply to vault commands
[ ] Create async runtime at the top level and remove the node_rpc + rpc functions from all commands and just call the run_impl method

Aug 16 '22 06:08 adrianbenavides

@mrinalwadhwa I would like to apply the new request handling pattern to identity commands if that's ok?

Sep 10 '22 22:09 RajivTS

@RajivTS that would be great! Let us know if you have any questions.

Sep 10 '22 22:09 mrinalwadhwa

@RajivTS I've created separated issues with reduced scope for the identity commands, in case you want to check them out:

I'm closing this issue for now.

Oct 04 '22 09:10 adrianbenavides