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

Fixes and interop testing #768

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Jul 15, 2024

This PR adds interop testing against Credo and Sphereon's OID4VCI client library. There were also some fixes to achieve interop:

  • Issuer ID recorded in OID4VCI exchange record.
  • Fill in the iss and credentialSubject.issuer fields using record Issuer ID instead of the verification method ID.

Heads up @timbl-ont @weiiv, there might be some minor conflicts with the work you've been doing.

mepeltier and others added 2 commits July 15, 2024 17:26
Signed-off-by: Micah Peltier <micah6_8@yahoo.com>
Against Credo and Sphereon libraries.

Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
@dbluhm dbluhm force-pushed the fix/sphereon-interop branch from d2e63af to 067a0a3 Compare July 15, 2024 21:27
@dbluhm dbluhm requested a review from jamshale July 15, 2024 21:28
dbluhm added 2 commits July 15, 2024 17:37
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Copy link
Contributor

@jamshale jamshale left a comment

Choose a reason for hiding this comment

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

I will approve. Just wasn't sure if some of the console logs should be there or not, or if they were used for debugging?

oid4vci/integration/credo/index.ts Show resolved Hide resolved
oid4vci/integration/sphereon/index.ts Show resolved Hide resolved
oid4vci/pyproject.toml Outdated Show resolved Hide resolved
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
@dbluhm dbluhm merged commit 084088c into openwallet-foundation:main Jul 16, 2024
3 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.

3 participants