Skip to content
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

Create a new JSON-LD Context for VCs containing RSS proofs #166

Open
thobson88 opened this issue Dec 21, 2023 · 0 comments
Open

Create a new JSON-LD Context for VCs containing RSS proofs #166

thobson88 opened this issue Dec 21, 2023 · 0 comments

Comments

@thobson88
Copy link
Collaborator

thobson88 commented Dec 21, 2023

To fully support RSS proofs in VCs, the attributes of the RSS proof must be found in the JSON-LD context that is referred to inside the VC.

No such context currently exists (because our implementation of RSS signatures for VC selective disclosure is novel).

In the absence of such a context, our temporary workaround has been to change the expansion policy (that's used e.g. when expanding a VC in the process of signing a VP), setting it to Relaxed rather than Strict. (This was done in this commit on our fork of the SSI repo). Without this workaround, the process of generating a VP from a VC fails.

However this change in the expansion policy causes two test failures in the SSI codebase (albeit in crates for DID methods unrelated to Trustchain).

The steps needed to properly fix this issue are:

  • create a suitable JSON-LD context
  • load that context when handling VCs with RSS proofs (see the related issue #128)
  • undo the temporary workaround by setting the JSON-LD expansion policy in the SSI library back to Strict.
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

No branches or pull requests

1 participant