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

Add cryptosuites to issuer config #128

Closed
dlongley opened this issue Jan 31, 2024 · 5 comments
Closed

Add cryptosuites to issuer config #128

dlongley opened this issue Jan 31, 2024 · 5 comments
Assignees

Comments

@dlongley
Copy link
Member

cryptosuites should be an array of cryptosuite options objects that will be used by the issuer. It should look like this:

cryptosuites: [{cryptosuite: 'cryptosuite name (same values as the old suiteName), options: {mandatoryPointers}}, {...}]

Support for suiteName should remain for backwards compatibility, but only one of cryptosuites or suiteName should be used.

@dlongley
Copy link
Member Author

dlongley commented Feb 2, 2024

See #131 as well.

@aljones15
Copy link
Contributor

aljones15 commented Feb 2, 2024

Work on cryptosuites as an issuer config option was started here: https://github.com/digitalbazaar/bedrock-vc-issuer/tree/add-issuer-provision-option-cryptosuites

@dlongley
Copy link
Member Author

I don't know why I wrote that we'd want this field to be an array, there's just one cryptosuite per issuer instance, so we can simplify that bit.

@dlongley dlongley self-assigned this Apr 15, 2024
@dlongley
Copy link
Member Author

This work is in progress now -- existing work is out of date and we need a slightly different direction. Also, the reason I wrote that we wanted an array was to support proof sets, so we do want it to function that way.

@dlongley
Copy link
Member Author

This was done in #142, closing.

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

2 participants