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

fix: look up conn record by invite msg id instead of key #2891

Merged

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Apr 13, 2024

Fixes #2890.

When reusing DIDs in invitations, the invitation key is no longer a unique value by which a connection record can be discovered. To workaround this, this change causes connection records to be looked up by invitation message id (request message's pthid) instead of by invitation key.

dbluhm added 3 commits April 13, 2024 14:53
Fixes openwallet-foundation#2890.

When reusing DIDs in invitations, the invitation key is no longer a
unique value by which a connection record can be discovered. To
workaround this, this change causes connection records to be looked up
by invitation message id (request message's pthid) instead of by
invitation key.

Signed-off-by: Daniel Bluhm <[email protected]>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@swcurran swcurran merged commit 8df6b3e into openwallet-foundation:main Apr 15, 2024
8 checks passed
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.

Reusing peer DID for invitation resulting in DID Ex failure
3 participants