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

Feature/0.7.4 ssl public did #89

Conversation

cjhowland
Copy link

This PR backports the endpoint attrib structure fix in PR openwallet-foundation#1934. It implements the solution discussed in openwallet-foundation#1928 to resolve an issue with the endpoint attrib structure. We have brought the structure into alignment with the did:sov specification by removing the nested endpoint structure and ensuring that service types profile and linked_domains can be used. This solution is interoperable with AFJ, which also supports endpoint attrib data in the following manner:

{
  "endpoint": "https://example.com",
  "types": ["did-communication", ...],
  "routingKeys": [...],
  "profile": "https://example.com/profile",
  "linked_domains": "https://example.com/linked-domains",
}

…emoved

Signed-off-by: Char Howland <[email protected]>

fix: remove layer of endpoint nesting

Signed-off-by: Char Howland <[email protected]>

fix: endpoint attrib json object with one top level key "endpoint"

Signed-off-by: Char Howland <[email protected]>
Signed-off-by: Char Howland <[email protected]>

fix: flake8 failure

Signed-off-by: Char Howland <[email protected]>

fix: flake8

Signed-off-by: Char Howland <[email protected]>
Copy link
Member

@dbluhm dbluhm left a comment

Choose a reason for hiding this comment

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

Thanks!

@KimEbert42 KimEbert42 merged commit b2463c7 into Indicio-tech:feature/0.7.4-ssl-public-did Sep 30, 2022
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