Skip to content
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

Remove notion of connection_handle from Aries state machines #111

Closed
Patrik-Stas opened this issue Sep 24, 2020 · 1 comment
Closed

Remove notion of connection_handle from Aries state machines #111

Patrik-Stas opened this issue Sep 24, 2020 · 1 comment

Comments

@Patrik-Stas
Copy link
Contributor

  • Code in V3 directory should not contain notion of connection_handle as that's something what belongs to higher layers of abstractions.
  • Connection handle is being used typically to: send message (maybe more)
  • Instead of connection handle, different minimal abstractions necessary to performs these operations will have to be passed - perhaps closures capable of sending message to the right place, some trait object or maybe connection state machine?
  • Side effect will be that connection handle won't be part of serialized credential/proof state machines.
  • This will be important step toward extracting state aries protocol state machines to independent crate.
@Patrik-Stas
Copy link
Contributor Author

Implemented by:

#256
#272
#273

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant