search

openid / OpenID4VP

57 stars 20 forks source link

Remove the whole 'Support for Federations/Trust Schemes' section #282

Closed bc-pi closed 1 month ago

bc-pi commented 1 month ago

As an alternative fix to the one in #280, which is an alternative fix to the one in #279

Some more on issues with this section: https://github.com/openid/OpenID4VP/pull/274#discussion_r1790491130 https://github.com/openid/OpenID4VP/pull/274#discussion_r1790466696

And I'm pretty sure the claims parameter with = { "vp_token": { "presentation_definition": { ... isn't a thing anymore

as noted by @jogu in #280, if this stuff is needed and can be written down in a consumable and accurate way, maybe the emergent federation-wallet spec would be a more appropriate place.

jogu commented 1 month ago

6 approvals, open a week, discussed on last week's WG calls - merging!