OpenID4VCI icon indicating copy to clipboard operation
OpenID4VCI copied to clipboard

Credential Issuer Metadata - nested objects & display properties

Open sschulz-t opened this issue 1 year ago • 2 comments

I was wondering how to interpret the definition of Credential Issuer Metadata:

Specifically the sentence

...the most deeply nested value MAY be an object that includes the following parameters defined by this specification:

Does this suggest that we only support display properties at the leaf nodes?

Or is it possible to have display properties at all levels?

E.g. is this a valid definition?

{
   ...
    "claims": {
        "address" : {
            "display" : [
                { "name": "Adresse",  "locale": "de" },
                { "name": "Address", "locale": "en" }
            ],
            "street" : {
                "display" : [
                    { "name": "Strasse", "locale": "de"},
                    { "name": "Street", "locale": "en" }
                ]
            }
        }
    }
}

If this is valid, how would i handle credentials that have a "display" property on their own? e.g.

...
  "property1" : {
     ...
     "display": false,
  }

sschulz-t avatar May 07 '24 08:05 sschulz-t

The problem is being tackled in https://github.com/openid/OpenID4VCI/pull/276 already

paulbastian avatar May 07 '24 12:05 paulbastian

to Paul's point, right now, only display properties at the leaf nodes are supported, so the example you are providing is not defined by the specification right now. The issue was raised in #276, we were discussion solutions, but in the end decided not to introduce this breaking change before VCI ID-1. Please continue this discussion in #276, but would like your feedback how big of a problem it is for you as an implementer the inability to express display properties at all levels for nested claims?

Sakurann avatar May 23 '24 14:05 Sakurann

Thanks, i see. You switched to use path values. I will post a reply to mickrau on Feb 21 and continue there.

sschulz-t avatar May 27 '24 07:05 sschulz-t