-
Notifications
You must be signed in to change notification settings - Fork 133
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
DIDs: alsoKnownAs property #260
Comments
Thanks for flagging this. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Still an open issue that should be worked on IMO |
Yes, it is. We're waiting for the DID spec to become a bit more stable before committing to this. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
The
alsoKnownAs
property has been added to the DID spec recently.It's basically a string array of some identifiers (URIs) by which the DID subject can also be identified. (see the corresponding section in w3c did-core)
I was wondering whether it is planned to include that in daf-ethr-did too?
Possible implementation
Could be done building upon the
setAttribute
method of the ethr-did-registry, e.g.did/aka
+ hex encoded string.Similar to the other methods in daf-ethr-dids identity-provider (beta-branch) it would just be something like
async addAlsoKnownAs({ identity, service, options }: { identity: IIdentity; alsoKnownAs: string; options?: any }, context: IContext): Promise<any>
What's your opinion on that?
(Should I also mention that in
ethr-did-resolver
somehow?)Do you have an estimation when the beta-branch will be merged? (Thinking about whether it makes sense to include that in the current version or just go for beta-branch).
EDIT: maybe someone could remove the bug label and add something like feature or feature-request...
The text was updated successfully, but these errors were encountered: