Skip to content

New SD-JWT Encoding #174

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

Merged
merged 29 commits into from
Nov 23, 2022
Merged

New SD-JWT Encoding #174

merged 29 commits into from
Nov 23, 2022

Conversation

danielfett
Copy link
Member

@danielfett danielfett commented Nov 21, 2022

Link to editor's copy: https://drafts.oauth.net/oauth-selective-disclosure-jwt/danielfett/new-sd-jwt-encoding/draft-ietf-oauth-selective-disclosure-jwt.html

  • Disclosures are now delivered not as a JWT but as separate base64url-encoded JSON objects.
  • In the SD-JWT, hash digests are collected under a _sd claim per level.
  • Terms "II-Disclosures" and "HS-Disclosures" are replaced with "Disclosures".
  • Holder Binding is now separate from delivering the Disclosures and implemented, if required, with a separate JWT.
  • Examples updated and modified to properly explain the specifics of the new SD-JWT format.
  • Examples are now pulled in from the examples directory, not inlined.
  • Updated and automated the W3C VC example.
  • Added examples with multibyte characters to show that the specification and demo code work well with UTF-8.

@Sakurann I updated the W3C example since I had to adapt it to the new format. Also made some updates together with @tlodderstedt, please check.

@Sakurann I added an example with a Japanese address that was thankfully AI-generated by Github Copilot. I have no clue if it makes sense or not, please take a look!

@danielfett danielfett marked this pull request as ready for review November 21, 2022 18:26
@danielfett danielfett requested a review from Sakurann as a code owner November 21, 2022 18:26
Copy link
Contributor

@tlodderstedt tlodderstedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SD-JWT-VC example looks good to me.

"sub": "6c5c0a49-b589-431d-bae7-219122a9ec2c",
"given_name": "太郎",
"family_name": "山田",
"email": "\"unusual email address\"@日本.com",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"email": "\"unusual email address\"@日本.com",
"email": "\"unusual email address\"@nihon.com",

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we replace it with another domain with multibyte characters? 日本.example.com?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no one uses such domains in Japan though..?

@Sakurann
Copy link
Collaborator

I think updated SD-JWT-VC is clean and nice. we should probably clarify in the text that it is not "compliant" with vc-data-model v1.1 and is "future-looking" in the light of v2.0 conversations.

the Japanese address is legit and correct and is pretty close from where I lived. I suspect it is an address of a tokyo tower, but have not checked. tho "(@日本.com)" looked weird so I changed to nihon.com?

Copy link
Collaborator

@Sakurann Sakurann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, Daniel!!

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

Successfully merging this pull request may close these issues.

3 participants