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

OIDC4VCI Draft 13 with vc format = jwt_vc_json, credential request is not correct #2796

Closed
ThierryThevenet opened this issue Jul 17, 2024 · 0 comments
Assignees
Labels

Comments

@ThierryThevenet
Copy link
Member

ThierryThevenet commented Jul 17, 2024

when Draft 13 and vc format is jwt_vc_json or ldp_vc

wallet must send a credential request like

{
   "format": "jwt_vc_json",
   "credential_definition": {
      "type": [
         "VerifiableCredential",
         "UniversityDegreeCredential"
      ],
      "credentialSubject": {
         "given_name": {},
         "family_name": {},
         "degree": {}
      }
   },
   "proof": {
      "proof_type": "jwt",
      "jwt":"eyJraWQiOiJkaWQ6ZXhhbXBsZTplYmZlYjFmNzEyZWJjNmYxYzI3NmUxMmVjMjEva2V5cy8
      xIiwiYWxnIjoiRVMyNTYiLCJ0eXAiOiJKV1QifQ.eyJpc3MiOiJzNkJoZFJrcXQzIiwiYXVkIjoiaHR
      0cHM6Ly9zZXJ2ZXIuZXhhbXBsZS5jb20iLCJpYXQiOiIyMDE4LTA5LTE0VDIxOjE5OjEwWiIsIm5vbm
      NlIjoidFppZ25zbkZicCJ9.ewdkIkPV50iOeBUqMXCC_aZKPxgihac0aW9EkL1nOzM"
   }
}

In credential_definition object only type is required and credentialSubject is optional and limited to the claims without anything in side.

To make it simple, wallet provides in the request only type and remove credentialSubject as it is optional

example

{
   "format": "jwt_vc_json",
   "credential_definition": {
      "type": [
         "VerifiableCredential",
         "UniversityDegreeCredential"
      ]
   },
   "proof": {
      "proof_type": "jwt",
      "jwt":"eyJraWQiOiJkaWQ6ZXhhbXBsZTplYmZlYjFmNzEyZWJjNmYxYzI3NmUxMmVjMjEva2V5cy8
      xIiwiYWxnIjoiRVMyNTYiLCJ0eXAiOiJKV1QifQ.eyJpc3MiOiJzNkJoZFJrcXQzIiwiYXVkIjoiaHR
      0cHM6Ly9zZXJ2ZXIuZXhhbXBsZS5jb20iLCJpYXQiOiIyMDE4LTA5LTE0VDIxOjE5OjEwWiIsIm5vbm
      NlIjoidFppZ25zbkZicCJ9.ewdkIkPV50iOeBUqMXCC_aZKPxgihac0aW9EkL1nOzM"
   }
}

For info, I have seen the attribute order and display, etc in the credential request of the OIDC4VCI test 7

see below what is sent by wallet :

{
    "proof": {
        "proof_type": "jwt",
        "jwt": "eyJhbGciOiJFUzI1NiIsInR5cCI6Im9wZW5pZDR2Y2ktcHJvb2Yrand0Iiwia2lkIjoiZGlkOmp3azpleUpqY25ZaU9pSlFMVEkxTmlJc0ltdDBlU0k2SWtWRElpd2llQ0k2SW5kNGNWVkViVlo1YVdJeWJHNVRaUzFIVGtGd05qQmFTR2x3YWtFMllXSjJTVlJ6U0VweVpWRk1OR3NpTENKNUlqb2lkMjlQWTFGWGVXVnJPRGhFVHpSeVltTTRiMUpNUld4dE56VkJhVlpWZWtZd2MzZHJkRU5xTW0xSVRTSjkjMCJ9.eyJpc3MiOiJkaWQ6andrOmV5SmpjbllpT2lKUUxUSTFOaUlzSW10MGVTSTZJa1ZESWl3aWVDSTZJbmQ0Y1ZWRWJWWjVhV0l5Ykc1VFpTMUhUa0Z3TmpCYVNHbHdha0UyWVdKMlNWUnpTRXB5WlZGTU5Hc2lMQ0o1SWpvaWQyOVBZMUZYZVdWck9EaEVUelJ5WW1NNGIxSk1SV3h0TnpWQmFWWlZla1l3YzNkcmRFTnFNbTFJVFNKOSIsImlhdCI6MTcyMTI0NjA4MywiYXVkIjoiaHR0cDovLzE5Mi4xNjguMC4yMDozMDAwL2lzc3Vlci9yYWFteGVwcWV4Iiwibm9uY2UiOiI3YThmNjZlMC00NDc2LTExZWYtYWE4MS0xN2Y2ODNiZmZlODEifQ.mvtV5t42y57g4XIydAA3U6MWBah3_JPepi-WQSIfaTCrx0-abXb7_80yffq4hlwk75nRstT8Ste5dn9wTMGgeQ"
    },
    "format": "jwt_vc_json",
    "credential_definition": {
        "credentialSubject": {
            "dateIssued": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Issuance date"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "D\u00e9livr\u00e9 le"
                    }
                ],
                "mandatory": true
            },
            "dateOfBirth": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Date of birth"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "N\u00e9(e) le"
                    }
                ],
                "mandatory": true
            },
            "email": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Email"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "Email"
                    }
                ],
                "mandatory": true
            },
            "familyName": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Family name"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "Nom"
                    }
                ],
                "mandatory": true
            },
            "firstName": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "First name"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "Pr\u00e9nom(s)"
                    }
                ],
                "mandatory": true
            },
            "gender": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Gender"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "Sexe"
                    }
                ],
                "mandatory": true
            },
            "issuing_country": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Issuing country"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "D\u00e9livr\u00e9 par"
                    }
                ],
                "mandatory": true
            },
            "phone_number": {
                "display": [
                    {
                        "locale": "en-US",
                        "name": "Phone number"
                    },
                    {
                        "locale": "fr-FR",
                        "name": "T\u00e9l\u00e9phone"
                    }
                ],
                "mandatory": true
            }
        },
        "order": [
            "firstName",
            "familyName",
            "dateOfBirth",
            "gender",
            "dateIssued",
            "issuing_country",
            "email",
            "phone_number"
        ],
        "type": [
            "VerifiableCredential",
            "VerifiableId"
        ]
    }
}
@ThierryThevenet ThierryThevenet changed the title Draft 13, credential request with jwt_vc_json Draft 13, credential request with jwt_vc_json is not correct Jul 17, 2024
@ThierryThevenet ThierryThevenet changed the title Draft 13, credential request with jwt_vc_json is not correct OIDC4VCI Draft 13 with vc format = jwt_vc_json, credential request is not correct Jul 17, 2024
hawkbee1 added a commit that referenced this issue Jul 30, 2024
* fix: Update account name instantly #2667

* linter update

* feat: Add green color for card valid status #2764

* color adjustment

* fix: Update display issue of sd selections #2777

* feat: Update ebsi logo to make light theme compatible #2775

* feat: Use english for oidc4vc screen #2696

* version update to 2.9.1+470

* fix: Avoid double scan to get and present #2668

* fix: Force user to delete account if wallet is suspended #2759

* version update to 2.9.2+471

* fix: Close app if wallet attestation is revoked #2759

* refactor: Remove linkedIn credential contents #2683

* refactor: Remove reward check and display #2682

* feat: Manage blockchain options from portal #2780

* version update to 2.9.3+472

* feat: Use verification method from didkit for EVM #2637 #2638

* fix:Update status display #2783

* refactor: Add place to add image #2771

* version update to 2.9.4+473

* fix: Fetch uri for image but not working and indication of deleted message #2771

* refactor: Update sub value for cypto account ownership card #2609

* feat: Add all type of VCs #2779

* feat: Support all types presentaton wth VCFormat auto #2779

* verson update to 2.9.5+474

* feat: Use engish lang for profile list for all lang #2786

* refactor: Rename dutch into diipv2point1

* feat: Rename owf baseline profile into diipv3 #2784

* feat: Update logo for diip profiles #2785

* feat: Show all the issuer in discover for auto vc format #2787

* Update test

* feat: No jwt test for redirect_uri scheme and handle sd selection for auto #2790

* version update to 2.9.6+475

* feat: Disble wallet attestation card when dev mode is off #2789

* fix: Solve presentation issue for Test 13 when vc format is auto #2791

* fix: Solve json path issue for string numerals #2792

* remove old comment

* version update: 2.9.7+476

* fix: Solve uncheck issue #2792

* refactor: Make font bold for sd-jwt display #2797

* fix: Change the text color so that it is readable in light mode #2795

* feat: Implement display data for dummy cards #2799

* feat: Update credential request for draft 13 - jwt_vc_json and ldp_vc #2796

* version update: 2.9.8+477

* Revert "version update: 2.9.8+477"

This reverts commit 6e8265c.

* Revert "version update: 2.9.8+477"

This reverts commit 6e8265c.

* refactor: Update jwt decode test

* refactor: Update oidc4vc test

* feat: Consider backgroun_image parameter in display external issuer #2800

* refactor: Update logic based on limi_disclosure #2801

* version update: 2.9.9+478

* refactor: Update logic based on limi_disclosure #2801

* version update: 2.9.10+479

* refactor: Try to display image #2771

* fix: Fix image and add photo viewer #2771

* refactor: Optimise code to load image smoothly side by side #2771

* feat: Display advanced security option in enterprise #2584

* refactor: Add specific error message when required address is not present in account list #2659

* feat: Remove crypto account #2065

* feat: Update error responses #2573

* feat: Update UI for remove account and bug fix #2065

* feat: Fix EBSI issue - pathNested format provided #2805

* refactor: Remove long error message for now #2573

* version update: 2.9.11+480

* refactor: Remove long error message for now #2573

* refactor: Display advanced security options #2584

* refactor: Bug fix and optimise code

* refactor: Update tezart package #2804

* some update and package update

* version update to 2.9.12+481

* feat: Update walletconnect #2806

* feat: Remove trust framework #2817

* feat: Improve design #2816

* fix: Show wallet attestation card #2810

* version update: 2.10.0+482

* feat: Add app name in the wallet attestation request #2808

* refactor: Bug fix - show dialog

* feat: Update developer data for OIDC4VCI #2811

* feat: Can add only one line in button

* version update to 2.10.1+483

---------

Co-authored-by: hawkbee1 <googandads@gmail.com>
hawkbee1 added a commit that referenced this issue Aug 6, 2024
* fix: Update account name instantly #2667

* linter update

* feat: Add green color for card valid status #2764

* color adjustment

* fix: Avoid double scan to get and present #2668

* fix: Fetch uri for image but not working and indication of deleted message #2771

* feat: Add all type of VCs #2779

* feat: Support all types presentaton wth VCFormat auto #2779

* feat: Update credential request for draft 13 - jwt_vc_json and ldp_vc #2796

* version update: 2.9.8+477

* Revert "version update: 2.9.8+477"

This reverts commit 6e8265c.

* refactor: Update oidc4vc test

* refactor: Try to display image #2771

* fix: Fix image and add photo viewer #2771

* refactor: Optimise code to load image smoothly side by side #2771

* feat: Display advanced security option in enterprise #2584

* feat: Update error responses #2573

* refactor: Remove long error message for now #2573

* refactor: Display advanced security options #2584

* refactor: Bug fix and optimise code

* some update and package update

* refactor: Bug fix - show dialog

* feat: Update PIN Code input placement #2822

* feat: Disble software license page #2822

* fix: Solve margin issue #2822

* feat: Add message in the developer info dialog #2822

* feat: Add message in the developer info dialog #2822

* feat: Make credential overly content color to white #2822

* feat: Make network title bold #2822

* feat: Update warning text #2822

* feat: Improve redability by moving default part to the end of sentence #2822

* feat: Change color of copy and share icons #2822

* feat: Simplify recovery info safety dialog #2822

* feat: Update enterprise display data in about section #2822

* feat: Remove extra padding in blockchain accounts #2822

* feat: Only wallet ready string is shown #2822

* feat: Dosplay more text in terms #2822

* feat: Change background text field #2822

* feat: Some bug fix

* feat: Support base64 with extra prefix in string #2819

* feat: Support base64 with extra prefix in string #2819

* feat: Remove black opacity from the credential image

* feat: Make description title bold #2822

* version update:  2.10.3+485

* feat: Add network Etherlink basics #2820

---------

Co-authored-by: hawkbee1 <googandads@gmail.com>
hawkbee1 added a commit that referenced this issue Aug 15, 2024
* fix: Update account name instantly #2667

* linter update

* feat: Add green color for card valid status #2764

* color adjustment

* fix: Avoid double scan to get and present #2668

* fix: Fetch uri for image but not working and indication of deleted message #2771

* feat: Add all type of VCs #2779

* feat: Support all types presentaton wth VCFormat auto #2779

* feat: Update credential request for draft 13 - jwt_vc_json and ldp_vc #2796

* version update: 2.9.8+477

* Revert "version update: 2.9.8+477"

This reverts commit 6e8265c.

* refactor: Update oidc4vc test

* refactor: Try to display image #2771

* fix: Fix image and add photo viewer #2771

* refactor: Optimise code to load image smoothly side by side #2771

* feat: Display advanced security option in enterprise #2584

* feat: Update error responses #2573

* refactor: Remove long error message for now #2573

* refactor: Display advanced security options #2584

* refactor: Bug fix and optimise code

* some update and package update

* feat: Some bug fix

* feat: Add some etherlink functionlities #2820

* version update: 2.10.4+486

* feat: Add empty etherlink when no tokens are available

* version update : 2.10.5+487

* feat: Add etherlink balance and transfer

* version update: 2.10.6+488

* refactor: Copy clipboard data correctly #2830

* refactor: Copy clipboard data correctly #2830

* refactor: Create crypto account VC for etherlink #2833

* feat: Set timeout of 10seconds for statulist http get #2824

* feat: Enable advanced settings for non-enterprise profile #2828

* refactor: Correct the backend provider url #2818

* feat: Display some vc based on crypto account did issuer #2680

* version update: 2.10.7+489

* fix: Advanced security option #2828

* feat: Fix wallet configuration failing issue #2835

* fix: Show crypto account for respective wallet address only #2833

* version update:  2.10.8+490

---------

Co-authored-by: Bibash Shrestha <bibashshrestha@gmail.com>
hawkbee1 added a commit that referenced this issue Aug 21, 2024
* fix: Update account name instantly #2667

* linter update

* feat: Add green color for card valid status #2764

* color adjustment

* fix: Avoid double scan to get and present #2668

* fix: Fetch uri for image but not working and indication of deleted message #2771

* feat: Add all type of VCs #2779

* feat: Support all types presentaton wth VCFormat auto #2779

* feat: Update credential request for draft 13 - jwt_vc_json and ldp_vc #2796

* version update: 2.9.8+477

* Revert "version update: 2.9.8+477"

This reverts commit 6e8265c.

* refactor: Update oidc4vc test

* refactor: Try to display image #2771

* fix: Fix image and add photo viewer #2771

* refactor: Optimise code to load image smoothly side by side #2771

* feat: Display advanced security option in enterprise #2584

* feat: Update error responses #2573

* refactor: Remove long error message for now #2573

* refactor: Display advanced security options #2584

* refactor: Bug fix and optimise code

* some update and package update

* feat: Some bug fix

* feat: Add some etherlink functionlities #2820

* version update: 2.10.4+486

* feat: Add empty etherlink when no tokens are available

* version update : 2.10.5+487

* feat: Add etherlink balance and transfer

* version update: 2.10.6+488

* refactor: Copy clipboard data correctly #2830

* refactor: Copy clipboard data correctly #2830

* refactor: Create crypto account VC for etherlink #2833

* feat: Set timeout of 10seconds for statulist http get #2824

* feat: Enable advanced settings for non-enterprise profile #2828

* refactor: Correct the backend provider url #2818

* feat: Display some vc based on crypto account did issuer #2680

* version update: 2.10.7+489

* fix: Advanced security option #2828

* feat: Fix wallet configuration failing issue #2835

* fix: Show crypto account for respective wallet address only #2833

* version update:  2.10.8+490

* fix: Solve deeplink issue #2836

* feat: Fix button of dialog #2816

* version update: 2.10.9+491

* feat: Change icon color for Add account section #2842

* feat: Improve screen design #2815

* feat: Dialog bug fix

* feat: Add cancel button in sd-jwt selection screen #2843

* feat: Hide disabled button when limited disclosure is required #2844

* feat: Change the issuer call for vertifiable ID for profile DIIP V3.0 in discover #2837

* version update: 2.10.11+493

* refactor: Remove unnecessary codes

* feat: Add empty tokens for etherlink #2840

* feat: Add empty tokens for tezos #2840

* feat: Add empty tokens for tezos #2840

* feat: Add empty tokens for evms #2840

* display nested json on one level - no display

---------

Co-authored-by: Bibash Shrestha <bibashshrestha@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants