-
Notifications
You must be signed in to change notification settings - Fork 81
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
[AATH] Incorrect state reported by AATH Connection Handler when it should have completed #1253
Comments
acapy (and agents other than vcx) are sending an trust ping message after receiving the connection response from VCX. then the VCX backchannel does not handle this trustping, it silently ignores it; leaving VCX is the state of "request" (when really it should be in atleast the "response" state). in the case of ariesvcx-ariesvcx, it does NOT send a trustping, it sends an ACK, which the VCX backchannel CAN handle and will bump the connection state from request->complete. So the problem of incorrect state does not manifest. Perhaps just adding support for Trustping is an easier fix here. |
Some background. The RFC 0160 Connections protocol completion was left without a confirmation of the connection (bad design), and the “recommended” solution was to do a trust ping immediately after setup to confirm the connection. As such, that is what ACA-Py and other frameworks have done, so yes, implementing Trust Ping is the easiest fix. The protocol issue was “fixed” in RFC 0023 DID Exchange, but alas, 0160 was already in stone. A pain. |
The AATH results for the
./manage runset ariesvcx-acapy
runset is choking due to the aries-vcx connection handler reporting that it is still in the "request" state, after it has sent a response and received a trusttping... This tanks the results:compared to
./manage runset acapy-ariesvcx
which does ok:We need to fix the connection state that VCX reports when it is the Connection 0160 Inviter/Responder and receives a trustping
The text was updated successfully, but these errors were encountered: