chore: remove presentation decoding #50
Conversation
Signed-off-by: Timo Glastra <[email protected]>
|
Signed-off-by: Timo Glastra <[email protected]>
|
|
||
| export const zVpTokenDcql = z.record(z.union([z.string(), z.record(z.any())]), { | ||
| message: | ||
| 'dcql vp_token must be an object with keys referencing the dcql credential query id, and values the encoded (string or object) presentation', |
There was a problem hiding this comment.
Is there any chance to get support for upcoming draft 25?
The main change for DCQL is that there can be multiple presentations for a credential query id, e.g.
{
"credential_query_id_1": [....]
}
There was a problem hiding this comment.
Yeah should be quite simple to support. We'd need to make the needed changes in the DCQL library as well.
Let's wait for 25 to be published, and then we can add support for it
Remove the decoding from this library, this needs to be handled by the user of this library.
I did return expectedNonce, to indicate which nonce should be verified against. I also added a todo related to transaction data hashes (see #49)