-
Notifications
You must be signed in to change notification settings - Fork 47
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
Document that endpoint instances have specific behaviour by default, but can have options #419
Comments
Some outline text that might be helpful here:
|
Some notes on this issue as I work on the related PR (#426):
I think there is text in the workflows section of the spec relates to this: "The request body can also include configuration options to enable the exchange to be executed using additional protocols beyond VC API."
I think this might be at odds with the current VC API architecture as the current architecture expects that issuer and verifier services are called from the respective coordinators. I think that the above text implies that issuer and verifier coordinators should rather interface with the workflow services and the workflow services should interface with the issuer and verifier coordinators. |
The group discussed this on 2024-09-10:
The specification needs to elaborate that endpoint instances are configured in a ways that perform a concrete set of steps when called. Implementers can add
options
to the call to change the default behavior of the endpoint. A particular workflow can be configured to have access to multiple issuer instances and can choose which one to use based on options provided to protocol used in an exchange w/ a particular exchange client aka digital wallet. Each protocol that is used on trust boundary expresses information in different way, up to workflow to make sure that it's configured against is appropriately expressed when client makes a choice. Workflow maps into config on what it can do. Keep configurations simple and allow configs to be composed according to a business use case.A PR needs to be raised that accomplishes the goals above.
The text was updated successfully, but these errors were encountered: