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

[proposal] Implement DID Provider for did:dht #1374

Open
nickreynolds opened this issue Apr 9, 2024 · 5 comments · May be fixed by #1412
Open

[proposal] Implement DID Provider for did:dht #1374

nickreynolds opened this issue Apr 9, 2024 · 5 comments · May be fixed by #1412
Assignees
Labels
did-manager enhancement New feature or request good first issue Good for newcomers pinned don't close this just for being stale

Comments

@nickreynolds
Copy link
Contributor

Is your feature request related to a problem? Please describe.
did:dht is a new DID Method created by TBD and the spec is essentially finalized: https://did-dht.com/

TBD has already implemented the DID Method in web5.js and it would be great to get a provider in Veramo for better interoperability

Describe the solution you'd like
Implement a new Veramo plugin which includes a provider for did:dht

Additional context
https://developer.tbd.website/blog/did-dht/

@nickreynolds nickreynolds added enhancement New feature or request good first issue Good for newcomers did-manager labels Apr 9, 2024
Copy link

stale bot commented Jun 16, 2024

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.

@stale stale bot added the wontfix This will not be worked on label Jun 16, 2024
@mirceanis mirceanis added the pinned don't close this just for being stale label Jun 18, 2024
@stale stale bot removed the wontfix This will not be worked on label Jun 18, 2024
@walnut07 walnut07 linked a pull request Aug 27, 2024 that will close this issue
6 tasks
@walnut07
Copy link

walnut07 commented Aug 27, 2024

Hi, do you mind if I work on this issue? The draft PR has an empty package only for now, but I'm intended to implement these methods there: createIdentifier, addKey, removeKey, and deleteIdentifier

@mirceanis
Copy link
Member

Hi, do you mind if I work on this issue? The draft PR has an empty package only for now, but I'm intended to implement these methods there: createIdentifier, addKey, removeKey, and deleteIdentifier

By all means!
Contributions are very welcome.

For broader usability we require that all packages published from this monorepo work in all 3 target environments: nodejs, web(react), and mobile(react-native).
Veramo is a community led project. All maintenance is voluntary. So, to keep maintenance costs as low as possible, it's best to add plenty of unit tests to new code.

If you think your work will meet these constraints, please add it as a draft PR and mark it as ready for review when ready. Otherwise, you can also publish an independent package, and then you can link to it in Awesome.md

@walnut07
Copy link

Thank you so much, and I'm sorry for my delayed response. I forked the repository and I'm planning to add implementation there for now, and I might try pushing my code to this current repo if I think I can meet the conditions!

@walnut07
Copy link

Hi, I'm just sharing with you that I decided to work on my own repo and make it as a third-party plugin: https://github.com/walnut07/did-dht, as I struggled with some configuration stuff in the mono repo

It's almost empty currently but once it's done I will list it on the Awesome.md!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
did-manager enhancement New feature or request good first issue Good for newcomers pinned don't close this just for being stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants