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

purpose should be mandatory #459

Open
npdoty opened this issue Nov 15, 2023 · 3 comments
Open

purpose should be mandatory #459

npdoty opened this issue Nov 15, 2023 · 3 comments
Assignees

Comments

@npdoty
Copy link

npdoty commented Nov 15, 2023

It's never appropriate to request a credential without explaining for what purpose it will be used. This property of a Presentation Definition should, as a result, be mandatory.

Per-field purpose strings might make more sense as optional (if the requester can't be more specific in explaining each field's specific purpose rather than just the top-level purpose).

If the expectation is that purpose (and other important contextual information, like retention, deletion, secondary use, registration and auditing of the use, etc.) will be presented elsewhere, then the spec should instead be defining pointers to that external or out-of-band explanation, so that the wallet/holder can present confirmation of it to the user at the time that a decision is made.

@rado0x54
Copy link
Contributor

The intend of the spec was to provide a simple method to transport a text-only purpose with the presentation definition. It does not expect to be exclusive (e.g. the purpose could be transported out-of-band of PE, or that all credentials meet the threshold of requiring a purpose). Therefore we would not want to require is as part of the spec.

We like the idea of binding an out-of-band purpose to a presentation definition and then remove the simple "purpose" field from the spec. This can be addressed in a version 3.0.

@rado0x54
Copy link
Contributor

@npdoty We will close this ticket and recreate an out-of-band purpose binding ticket.

@rado0x54 rado0x54 self-assigned this Nov 30, 2023
@npdoty
Copy link
Author

npdoty commented Dec 1, 2023

Just tag me or drop a link when you have an out-of-band binding issue. I do think that's a more promising long-term approach.

I'm concerned in the meantime though that optional or ill-defined purpose specification mechanisms will lead to what has often been the status quo: users being asked (harassed) without context, hoping that they'll just give in and accept in order to get on with what they're doing.

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