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

Introduce VC spec #3

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Introduce VC spec #3

wants to merge 13 commits into from

Conversation

lucemans
Copy link
Contributor

@lucemans lucemans commented Sep 3, 2024

Introduces the verifiable Credential Specification.

This specification allows for W3C Verifiable Credentials to be stored inside ENS profiles

PS; This PR is also used for testing the ensip flow

@lucemans lucemans force-pushed the ensip/vc branch 9 times, most recently from 87d1a02 to 29c5e29 Compare September 4, 2024 01:51
Copy link

cloudflare-workers-and-pages bot commented Sep 4, 2024

Deploying ensips with  Cloudflare Pages  Cloudflare Pages

Latest commit: 212f74a
Status: ✅  Deploy successful!
Preview URL: https://fd9166e8.ensips.pages.dev
Branch Preview URL: https://ensip-vc.ensips.pages.dev

View logs

@lucemans lucemans force-pushed the ensip/vc branch 9 times, most recently from 05f1d5a to f9bb90e Compare September 4, 2024 04:59
@lucemans lucemans force-pushed the ensip/vc branch 2 times, most recently from 9e3e84b to dae0ddd Compare September 23, 2024 09:46
"type": ["VerifiableCredential"],
"issuanceDate": "2024-05-21T00:00:00Z",
"credentialSubject": {
"id": "did:ens:luc.eth", // the subject's ens name
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it might be useful to add the address here as well, as the ens owner can change and render the credential invalid. so it might look something like that:

"did:ens:luc.eth#0x225f137127d9067788314bc7fcc1f36746a3c3B5"


Note the requirement for `id` to be `did:ens:luc.eth` in order to prevent credential spoofing.

```json
Copy link

@Ghadi8 Ghadi8 Jan 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wouldn't the issuer and Proof be needed here as well?

We also worked on a fix for the ens did resolver library and are currently using it at JustaName. We will open a PR with the fix, and potentially publish the lib if needed...

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.

2 participants