-
Notifications
You must be signed in to change notification settings - Fork 2
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
Proposal: use did:jwk for testing dids and vc-jwt in the vc-jwt-test-suite #10
Comments
The VC-data-model spec was very explicit about DIDs not being a dependency of VCs. Can the test suite be written in a way which nonetheless does not depend on a correct implementation of did:jwk? |
@bellebaum yes, assuming The point of using The problem is not explaining sufficiently how you get verification material... I propose we cover that concretely for DIDs, URLs, and anything else. |
Our initial implementation tried to avoid using did's altogether. The Issuer uses an X.509 PKC of type domain validated, so that the verifier can make a TLS connection to the domain and pick up the X.509 PKC from there (e.g. in PEM format). The iss is set to the domain name. Registering an IANA well-known for this page would take the verifier straight to the right web page. |
To be clear, I am not proposing the test suite only cover DIDs... I am proposing the test suite cover ALL legal "issuer" identifier formats. I am strongly opposed to "verify with this public key" without describing "how to get this public key". |
I have nothing against using did:jwk - it should be an option that would have the most utility to the testers while being the least controversial. just to document... a way to avoid using DIDs would be, for the issuer provide a certificate (how to get this key becomes just use this key) and for the Holder, in the VC pass the pubKey by value using |
@Sakurann we can add tests like you suggest after we follow up on w3c/vc-jose-cose#111 |
I started work on a test suite here: https://github.com/transmute-industries/vc-jwt-test-suite |
The text was updated successfully, but these errors were encountered: