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

Why vct is not selectively disclosable, but vct#integrity is not? #258

Open
Sakurann opened this issue Oct 7, 2024 · 4 comments
Open

Why vct is not selectively disclosable, but vct#integrity is not? #258

Sakurann opened this issue Oct 7, 2024 · 4 comments
Assignees
Labels
Ready-for-PR Needs PR (typically after resolved discussion)

Comments

@Sakurann
Copy link
Collaborator

Sakurann commented Oct 7, 2024

Would also be good to add rationale why chosen certain claims are not selectively disclosable.

@Sakurann
Copy link
Collaborator Author

Sakurann commented Oct 7, 2024

also what about extends and extends#integrity claims?

@danielfett
Copy link
Member

Right, vct#integrity should be added/mentioned here: https://drafts.oauth.net/oauth-sd-jwt-vc/draft-ietf-oauth-sd-jwt-vc.html#section-3.2.2.2

extends and extends#integrity are not in the credential, so are not affected.

@danielfett danielfett added the Ready-for-PR Needs PR (typically after resolved discussion) label Oct 8, 2024
@bc-pi
Copy link
Collaborator

bc-pi commented Oct 8, 2024

this text is/was about application profiles (like SD-JWT VC) saying what claims the issuer can make selectively disclosable and maybe has text that could serve as inspiration https://www.ietf.org/archive/id/draft-ietf-oauth-selective-disclosure-jwt-12.html#name-selectively-disclosable-val

@bc-pi
Copy link
Collaborator

bc-pi commented Oct 8, 2024

I think I maybe meant for that #258 (comment) to go on #257 but maybe works here too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready-for-PR Needs PR (typically after resolved discussion)
Projects
None yet
Development

No branches or pull requests

3 participants