decentralized-identity / jwt-vc-presentation-profile

https://identity.foundation/jwt-vc-presentation-profile/
Apache License 2.0
15 stars 15 forks source link

Add jwk DIDs #90

Closed nklomp closed 11 months ago

nklomp commented 1 year ago

This PR adds did:jwk support, aligning this profile more with the upcoming Dutch Decentralized Identity Profile (no link yet). Adding did:jwk has been discussed in #80 and last meeting. I think the main outstanding questions is, whether did:jwk support should be mandatory or optional (did:web and ion are mandatory, so I added jwk as mandatory as well)

Also has some small editorial changes like spelling errors, and adding myself as contributor

jischr commented 1 year ago

@quartzjer Are there plans to "freeze" the version of did:jwk so we can reference it in the profile?

jischr commented 1 year ago

Waiting for v1.0 to merge this

quartzjer commented 1 year ago

@quartzjer Are there plans to "freeze" the version of did:jwk so we can reference it in the profile?

Good idea, I created a v1.0 release of did:jwk so serve as the official stable reference: https://github.com/quartzjer/did-jwk/tree/v1.0

Sakurann commented 1 year ago

@nklomp we have v.0.0.1 now, so please mark this ready for review and merge :)

nklomp commented 11 months ago

Marked as ready. I do not have permissions to merge