Skip to content

fix(directory): presentCredential offers only credentials your agent presents as the member - #376

Merged
stormer78 merged 1 commit into
mainfrom
fix/present-member-bound-only
Oct 26, 2026
Merged

stormer78 merged 1 commit into
mainfrom
fix/present-member-bound-only

Conversation

@stormer78

Copy link
Copy Markdown
Contributor

The community accepts a verified field only from a presentation its member holds, so a credential bound to another key (an SD-JWT with a separate holder did:key) was offered, chosen, presented and then refused. The wallet now asks the agent only for credentials whose subject is the member's DID (vault/credentials/query subjectDid, trust-tasks-tf #884).

Needs a VTA that serves subjectDid: an older one refuses the query as malformed, so this ships after the VTI release that carries it.

Draft until trustoverip/dtgwg-trust-tasks-tf#884 merges and a VTI release serves subjectDid (VTI PR to follow). No @openvtc/trust-tasks bump needed to build; the core test checks the payload carries the filter and that it alone is refused before sending.

…presents as the member

The community accepts a verified field only from a presentation its member holds, so a credential bound to another key (an SD-JWT with a separate holder did:key) was offered, chosen, presented and then refused. The wallet now asks the agent only for credentials whose subject is the member's DID (vault/credentials/query subjectDid, trust-tasks-tf #884).

Needs a VTA that serves subjectDid: an older one refuses the query as malformed, so this ships after the VTI release that carries it.

Signed-off-by: Glenn Gore <glenn.g@affinidi.com>
@stormer78
stormer78 marked this pull request as ready for review October 26, 2026 01:36
@stormer78
stormer78 merged commit ed73916 into main Oct 26, 2026
4 checks passed
@stormer78
stormer78 deleted the fix/present-member-bound-only branch October 26, 2026 01:36
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.

1 participant