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

Finish Connectionless Proof Test Scenario(s) to AATH #90

Closed
nodlesh opened this issue Aug 13, 2020 · 3 comments · Fixed by #474
Closed

Finish Connectionless Proof Test Scenario(s) to AATH #90

nodlesh opened this issue Aug 13, 2020 · 3 comments · Fixed by #474

Comments

@nodlesh
Copy link
Contributor

nodlesh commented Aug 13, 2020

This was attempted in Aug 2020. There is code in the AATH to make this work. In submission related to this issue. It was determined that Acapy at the time did not support the connectionless scenario is played as the prover. This work has been put on hold until Acapy has this fully implemented.

The main Story for this work was #11.

@TimoGlastra
Copy link
Member

I'm curious how connectionless proofs will work in AATH. Will these be separate operations or the same operations, but the connection_id swapped with something else?

And is there anything I can do on the .NET side to leave it "Connectionless Ready"?

@TimoGlastra
Copy link
Member

I'm currently working on integrating connectionless into AATH but only for usage with out of band (the AIP 2 approach). I see there's some foundation made for connectionless using AIP 1.0, but as it seems this has never run.

@nodlesh I don't suspect anyone will implement connectionless anymore for AIP 1.0, so is it okay if I remove this in favour of the AIP 2 implementation (there is some overlap, but it's minimal)?

@nodlesh
Copy link
Contributor Author

nodlesh commented Apr 8, 2022

Yes,I agree that implementing AIP2 over AIP1 is the right way to go here. If we have a requirement for AIP1 with connectionless we will address it at that time.

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

Successfully merging a pull request may close this issue.

2 participants