Skip to content

Conversation

PatStLouis
Copy link

I'd like to suggest a re-alignment with the vc-api approach for the credential offer attachment. I'm not convinced the suggested change provides sufficient improvements and the rational aligns with the direction of the vc-api. The Credential Offer Exceptions section is in line with the recent discussions on the vc-api calls.

This change seems to get rid of a dedicated options field and put all options at the root layer of the request.

The JSON-LD attachment dates from 2021 and the vc-api specification has changed since then.

I have a few questions:

  • Why include the data model version in the options? Could it not be derived from the context? In which scenario would an array be beneficial?
  • Why include a binding_required bool? Could it not be assumed as required if there is a binding_method present? Is there a scenario where the binding isn't required but a method is provided?

@TimoGlastra

Patrick St-Louis added 2 commits April 22, 2024 11:54
Signed-off-by: Patrick St-Louis <patrick.st-louis@idlab.org>
Signed-off-by: Patrick St-Louis <patrick.st-louis@idlab.org>
@PatStLouis PatStLouis changed the title Credential offer proposal RFC 0809 - VCDI Credential offer request alternate proposal Apr 22, 2024
@TelegramSam
Copy link
Contributor

Discussed 2024. Hard to merge given the changes, but could be useful in a note on the RFC.

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.

2 participants