bcgov / bc-wallet-mobile

BC Wallet to hold Verifiable Credentials
Apache License 2.0
60 stars 49 forks source link

Sharing information through the mobile verifier, verifier received "proof request declined" #1972

Closed knguyenBC closed 2 months ago

knguyenBC commented 5 months ago

Description of problem

When receiving a proof request through the mobile verifier, and sharing the information successfully, on the verifier's screen, they receive "Proof request declined, Proof verification failed" screen

Expected behavior

When sharing information and the information was sent successfully, the verifier should receive the information sent.

Steps to reproduce

Screenshots and/or log output

Screenshot_20240503-101719

Environment

Build #: 1692

Android Device Model: Pixel 8

iOS Device Model:

Workaround

Severity

jleach commented 4 months ago

I've tested 1.0.16 to 1.0.16 and things work as expected. Sending a proof works. I think this may be related to 1.0.17 (holder) to 1.0.16 (verifier). This will probably work itself out when we get some updates to fix other verifier issues like 1973. I'll keep the ticket open for retesting but for now there is nothing to be done.

@wadeking98 @bryce-mcmath FYI.

jleach commented 3 months ago

Confirmed this is still an issue with 1.0.17 1744 (tested on iOS 1.0.17 1744 as verifier, Android 1.0.16 as holder)

bryce-mcmath commented 3 months ago

Issue has been debugged and findings are with the Credo team. We may not be able to have backwards compatibility for the verifier this release. Putting in icebox for now

jleach commented 2 months ago

@bryce-mcmath Able to share the root cause? Any update?

bryce-mcmath commented 2 months ago

@jleach unfortunately there is a backwards compatibility issue that causes 1.0.16 and before to not work with 1.0.17 and after. I haven't been able to repro this issue when both wallets are 1.0.17 or above