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

Proof Request "required" field #2261

Open
al-rosenthal opened this issue Nov 8, 2024 · 1 comment
Open

Proof Request "required" field #2261

al-rosenthal opened this issue Nov 8, 2024 · 1 comment
Labels
question Further information is requested

Comments

@al-rosenthal
Copy link
Contributor

Right now, BC wallet mobile app expects schema id or cred def id to be present in proof requests.

Does it make sense for a proof request to be made without either of those ids? Is this something we need to handle?

@al-rosenthal al-rosenthal added the question Further information is requested label Nov 8, 2024
@swcurran
Copy link

swcurran commented Nov 8, 2024

In the AnonCreds spec, there are 8 “restrictions” that are supported, all of which are optional. The details of each are described in the spec here — restrictions. So yes, it makes sense that other restrictions could be used. Whether you want to support them in BC Wallet might be an open question. Certainly the software in BC Wallet can support all forms of restrictions. Restrictions are used when searching the credentials in the wallet for the set that satisfy the presentation request.

Further, since all are optional — there is the case that none are present — that restrictions in the presentation request is an empty array. That is taken to mean that the data is intended to be “self-attested” — provided by the holder directly (e.g. typed into a box) and not “verifiable” at all. While there are some good use cases for that, I wouldn’t think that BC Wallet is likely to support that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants