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

Handle trust pings in the AATH backchannel #1254

Merged

Conversation

gmulhearn
Copy link
Contributor

Close #1253

This adds handling for TrustPing, which solves the issue of connections not entering complete state when VCX is the connection inviter.

Results of ariesvcx-acapy manual runset of RFC0160

behave -D Acme=http://0.0.0.0:9020 -D Bob=http://0.0.0.0:9030 -t @RFC0160 -t ~@wip -t ~@RFC0434 -t ~@RFC0453 -t ~@RFC0211 -t ~@DIDExchangeConnection -t ~@Transport_Ws

Before

all failures.

After

(failures are due to running this with only 2 agents)

Failing scenarios:
  features/0160-connection.feature:52  Inviter Sends invitation for one agent second agent tries after connection
  features/0160-connection.feature:69  Inviter Sends invitation for one agent second agent tries during first share phase

0 features passed, 2 failed, 13 skipped
4 scenarios passed, 2 failed, 154 skipped
28 steps passed, 2 failed, 1353 skipped, 0 undefined

Signed-off-by: George Mulhearn <gmulhearn@anonyome.com>
Signed-off-by: George Mulhearn <gmulhearn@anonyome.com>
Copy link
Contributor

@JamesKEbert JamesKEbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@JamesKEbert JamesKEbert merged commit 08bea95 into hyperledger:main Jul 15, 2024
22 checks passed
lukewli-anonyome pushed a commit to lukewli-anonyome/aries-vcx that referenced this pull request Jul 25, 2024
* handle trust ping

Signed-off-by: George Mulhearn <gmulhearn@anonyome.com>

* clipppy

Signed-off-by: George Mulhearn <gmulhearn@anonyome.com>

---------

Signed-off-by: George Mulhearn <gmulhearn@anonyome.com>
Co-authored-by: George Mulhearn <gmulhearn@anonyome.com>
Signed-off-by: lli <lli@anonyome.com>
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 this pull request may close these issues.

[AATH] Incorrect state reported by AATH Connection Handler when it should have completed
3 participants