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

Bug with Present Proof flow in BC Wallet #2117

Open
mathieuglaude opened this issue Aug 8, 2024 · 7 comments
Open

Bug with Present Proof flow in BC Wallet #2117

mathieuglaude opened this issue Aug 8, 2024 · 7 comments
Milestone

Comments

@mathieuglaude
Copy link

Description of problem

  • We have encountered an issue with the verification flow in BC Wallet.
  • When the DID who created the Schema ID and the DID who created the Cred Defn ID are different, the BC Wallet is throws an error.
  • For example, if the SchemaID is FkPAh5o3s9PSmb8g9cFnQg:2:Person:1.0 and cred defn ID isFkPAh5o3s9PSmb8g9cFnQg:3:CL:726654:Person, the verification process runs properly.
  • Whereas if the Schema ID is BDQvbqKN9CVBVn1UryetKL:2:digital_business_card:1.0.0 and Cred defn ID is FkPAh5o3s9PSmb8g9cFnQg:3:CL:60640:Digital Business Card, the BC Wallet fails to provide the proof presentation.
  • These Schema IDs and CredDef IDs are created using BC Traction and are in BCovrin Test Ledger.
  • I've attached a screenshot for reference.
  • This is preventing us from running Present Proofs with a customer. Big roadblock for us right now.

Screenshots and/or log output

image

Build #:

version 1.0.18 Build (1801)

Severity

  • [ ] High
@cvarjao
Copy link
Member

cvarjao commented Aug 8, 2024

@mathieuglaude , do you have an example of a proof request we can use to replicate this issue?

@mathieuglaude
Copy link
Author

@cvarjao here you go:

{
"requested_attributes": [
{
"names": [
"identifier",
"role",
"family_name",
"cra_business_number",
"business_type",
"credential_id",
"business_name",
"given_names",
"company_status"
],
"restrictions": [
{
"cred_def_id": "FkPAh5o3s9PSmb8g9cFnQg:3:CL:60640:Digital Business Card",
"issuer_did": "FkPAh5o3s9PSmb8g9cFnQg"
}
]
}
],
"requested_predicates": []
}

@cvarjao
Copy link
Member

cvarjao commented Aug 8, 2024

Thanks. I will create a test scenario to replicate that and I will get back to you

@cvarjao
Copy link
Member

cvarjao commented Aug 9, 2024

@mathieuglaude , was that on iOS or Android? or both?

@cvarjao
Copy link
Member

cvarjao commented Aug 9, 2024

I was able reproduce. The next version of BC Wallet will come with Credo 0.5.9. BC Wallet 1.0.18 was with Credo 0.5.3. I can't pin point exactly what was fixed, but it works with the beta builds (1.0.19). Would you like access to beta build to validate?

@cvarjao cvarjao added this to the Release 1.0.19 milestone Aug 13, 2024
@cvarjao
Copy link
Member

cvarjao commented Aug 13, 2024

@mathieuglaude , not sure if you have seen my prior comment

@mathieuglaude
Copy link
Author

Hi @cvarjao - yes that would be a good idea if possible so that we can test internally.

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

No branches or pull requests

2 participants