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

did:ion and DID operations #255

Merged
merged 10 commits into from
Mar 2, 2022
Merged

did:ion and DID operations #255

merged 10 commits into from
Mar 2, 2022

Conversation

clehner
Copy link
Contributor

@clehner clehner commented Jan 21, 2022

Integrate Sidetree/ION implementation and DID operations from spruceid/ssi#379.

  • Add did:ion method/resolver
  • Add generate-secp256k1-key command
  • Add create command
  • Submit operations to Sidetree/ION API
  • Add update command
  • Add recover command
  • Add deactivate command
  • Update readme to document new commands

Example Usage

# Generate initial keys
$ didkit generate-secp256k1-key > update0.jwk
$ didkit generate-secp256k1-key > recovery0.jwk
$ didkit generate-secp256k1-key > signing0.jwk

# Create DID
$ did=$(didkit did-create ion -r recovery0.jwk -u update0.jwk -s signing0.jwk)

# Resolve long-form DID
$ DID_ION_API_URL=http://localhost:3000/ didkit did-resolve "$did"

@clehner clehner marked this pull request as ready for review February 15, 2022 21:44
@clehner clehner requested a review from sbihel February 16, 2022 21:21
@sbihel
Copy link
Member

sbihel commented Feb 24, 2022

Do you think you will move to subcommands before merging?

@clehner
Copy link
Contributor Author

clehner commented Mar 1, 2022

@sbihel I don't think so, unless there is a straightforward way to do it

@clehner clehner mentioned this pull request Mar 1, 2022
@clehner
Copy link
Contributor Author

clehner commented Mar 1, 2022

Opened #266 about the CLI

@clehner clehner merged commit a6332d5 into main Mar 2, 2022
@clehner clehner deleted the feat/did-ion branch March 2, 2022 21:12
@clehner
Copy link
Contributor Author

clehner commented Mar 2, 2022

Rebased and merged.
Docker image publish failed because ssi@main updated to v0.4 while didkit is not yet ready for ssi v0.4: https://github.com/spruceid/didkit/runs/5398417705?check_suite_focus=true
spruceid/ssi#401 should have been merged after this PR for that workflow to have succeeded. This will be fixed when didkit is updated to use ssi v0.4 (#269).

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