ockam icon indicating copy to clipboard operation
ockam copied to clipboard

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

Open adrianbenavides opened this issue 2 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

adrianbenavides avatar Aug 16 '22 06:08 adrianbenavides

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

RajivTS avatar Sep 10 '22 22:09 RajivTS

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

mrinalwadhwa avatar 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:

  • https://github.com/build-trust/ockam/issues/3562
  • https://github.com/build-trust/ockam/issues/3561

I'm closing this issue for now.

adrianbenavides avatar Oct 04 '22 09:10 adrianbenavides