-
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
Should vc http api use PE Spec for query format? #174
Comments
I think that should be --
|
I think this is related to w3c-ccg/vp-request-spec#7 |
This is also related to the see: https://github.com/trustbloc/wallet/blob/main/test/mock/adapter/templates/webWallet.html#L567-L643 But Trustbloc has wrapped it in the following way:
|
The group discussed this on the 2022-04-05 telecon, the only company that the group knows of having interest in implementing PE in the VC-API is SecureKey. @mavarley does SecureKey plan to do work in this area soon? If not, should we close this issue? The group discussed if we should have a top-level primitive that is a PE in VC API -- at least two people were against the concept since we 1) don't have any implementers that have committed to PE at the top level, and 2) we have a mechanism (VPR) that can slot in arbitrary query languages, so we /can/ support PE in the future via extension points. For this reason, suggesting that this issue is closed. Marking as 7 day close unless there are objections. |
Apologies for missing the call. There are a couple implications to the above:
SecureKey would accept the above - but it does mean implementors wanting to build extensions need to implement VPR, and then their protocol within (which again, I personally see no issues with). I think this approach also allows for "safe extension", as opposed to a wild-west approach. So how PE, DIDComm, etc.. fit within a VPR flow will be left to the organization that wants to bring that forward 'first'. If VC API was expected to be used as the API/Resource endpoint in a OIDC4SSI flow the complications are likely more severe - as the userinfo or credential endpoint would suddenly be incompatible! @tplooker do we need an issue for the above? Not PE specific, but rather OIDC-CP/ OIDC4SSI ? |
Block is looking to implement common methods of exchange and will be using PE within them all, so you can count us as an implementing party. |
Trace folks have been tracking this PEx / WACI-DIDComm ticket here: w3c-ccg/traceability-interop#17 currently no plans to implement, we are waiting for a perma url for didcomm. |
We discussed this on the 2022-05-17 telecon. There was concern about more protocols being more difficult to demonstrate interoperability on. PRs are welcome to explore the concept. |
The group discussed this on 2023-12-05: @dlongley mentioned that there are OID4 implementations that use VC API Exchanges that use PE, but there isn't a need to document that since it's just an implementation of OID4. @jrhender noted that he's using PeX with VC API, but could add something, but doesn't fit necessarily since VC-API is somewhat agnostic to query language and definitions. Maybe language to state that it's agnostic might be the point of a PR. @brianorwhatever noted that this comes up in workflows, which aren't fleshed out in the spec at present. @dlongley noted that there are now 3 different protocols that have been implemented over VC API Exchanges, one of them uses VPR, if you wanted to use PE you can embed it in a VPR, OID4 uses PE, and then there is some DIDComm stuff that could use PE. Other than talking about it in that way, it's not directly related to VC API. We discussed just closing this on the call. There were no objections to just closing (since PE is being used already in a subset of implementations). We might add language in the future to say that's explicitly OK, but implementers seem to be using PE w/ VC API today (among other exchange/query languages). |
Consider the following example:
Holder A
wants to present credentials toHolder B
Holder A
would tellHolder B
"IntentToProvideDataToHolderB"Holder B
would lookup "IntentToProvideDataToHolderB", and reply with the required credential types....@mavarley @troyronda can you provide an example of this flow in PE spec for comparison?
The text was updated successfully, but these errors were encountered: