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

chore: update pydid #1915

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Aug 25, 2022

Recent changes to the resolution of sov/indy DIDs caused PyDID to parse
service endpoints that should have been recognized as DIDComm services
as plain services. This broke some logic in accepting invitations from
public DIDs. Pulling in these updates from PyDID resolves the issue.

Signed-off-by: Daniel Bluhm [email protected]

Recent changes to the resolution of sov/indy DIDs caused PyDID to parse
service endpoints that should have been recognized as DIDComm services
as plain services. This broke some logic in accepting invitations from
public DIDs. Pulling in these updates from PyDID resolves the issue.

Signed-off-by: Daniel Bluhm <[email protected]>
@dbluhm
Copy link
Contributor Author

dbluhm commented Aug 25, 2022

The error this fixes:

aries_cloudagent.connections.base_manager.BaseConnectionManagerError: Cannot connect via public DID that has no associated DIDComm services

Caused by these lines:
https://github.com/hyperledger/aries-cloudagent-python/blob/a8101e1fef6004c79a8f0fb10b63138fb0ad140f/aries_cloudagent/connections/base_manager.py#L256-L264

@codecov-commenter
Copy link

Codecov Report

Merging #1915 (1680475) into main (ccf3290) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1915   +/-   ##
=======================================
  Coverage   93.68%   93.68%           
=======================================
  Files         540      540           
  Lines       34187    34187           
=======================================
  Hits        32027    32027           
  Misses       2160     2160           

@andrewwhitehead andrewwhitehead merged commit b12360c into openwallet-foundation:main Aug 26, 2022
@dbluhm dbluhm deleted the chore/update-pydid branch September 17, 2022 14:36
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.

3 participants