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

Release/v0.2.0 #57

Merged
merged 8 commits into from
Nov 12, 2024
Merged

Release/v0.2.0 #57

merged 8 commits into from
Nov 12, 2024

Conversation

stormer78
Copy link
Contributor

  • Updating dependency crate versions
  • Code cleanup on warnings
  • Implement local and network features on SDK
  • Added to did:key the ability to populate keyAgreement
  • Added WASM support
  • Added HTTP GET resolution
    • GET /did/v1/resolve/did
  • Configuration option to enable HTTP or WebSocket routes

was not being populated by SSI crate

Added Local and Network feature to sdk
- a precursor to getting DID resolver being WASM ready
  GET /did/v1/resolve/<did>
for i in 0..2 {
assert!(keys_multibase.contains(&method_ids[i][1..].to_string()));
}
method_ids.iter().take(2).for_each(|id| {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good point, noted!

Copy link
Collaborator

@anton-iskryzhytskyi anton-iskryzhytskyi left a comment

Choose a reason for hiding this comment

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

LGTM

YoussefAWasfy
YoussefAWasfy previously approved these changes Nov 5, 2024
Copy link
Collaborator

@YoussefAWasfy YoussefAWasfy left a comment

Choose a reason for hiding this comment

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

LGTM

* Updating crates
* Changed how keys are resolved in did:peer from multicodec to JsonWebTokens
@stormer78 stormer78 merged commit 5cc3063 into main Nov 12, 2024
9 of 12 checks passed
@stormer78 stormer78 deleted the release/v0.2.0 branch November 12, 2024 20:30
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.

3 participants