Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

section 4 Wallet Conformance Criteria: description of DID methods #31

Closed
bluesteens opened this issue Mar 24, 2023 · 11 comments · Fixed by #53
Closed

section 4 Wallet Conformance Criteria: description of DID methods #31

bluesteens opened this issue Mar 24, 2023 · 11 comments · Fixed by #53
Assignees
Labels
documentation Improvements or additions to documentation Steering - FYI Technical or internal change. Not necessary for Steering to review.

Comments

@bluesteens
Copy link
Member

bluesteens commented Mar 24, 2023

4.1.2 Verifiable Data Registry & Endorsed DID Methods
a. This sentence should be removed, as it offers a loophole for others to challenge or complicate existing OCI architecture.

A verifiable data registry might also be used for other cryptographically-verifiable data structures such as verifiable credentials.

b. The last column in the table (OCI Credentialing Actor) should be removed, as OCI should simply state which DID methods are endorsed without imposing the roles for which they are used.

c. OCI should define what is meant by:

Digital Wallet Providers SHALL implement and support all DID methods

"Support" can simply mean being able to resolve DIDs but also to offer DID creation for usage. I'd removed the word "implement".
It is the solution provider's decision, which DID methods to offer to users for creation/usage. OCI can only really demand DID resolution of all methods to ensure interop.

d. OCI may want to consider setting out broad expectations on the qualities for DID methods to justify the recommended ones, e.g.

  • DID document integrity
  • DID document history
  • always available registry

03/31 update: following review, this ticket has the same objective as ticket #32 - thus it is recommended to discuss them together

@bluesteens bluesteens added the documentation Improvements or additions to documentation label Mar 24, 2023
@bluesteens
Copy link
Member Author

bluesteens commented Mar 24, 2023

loosely related to issue: Open-Credentialing-Initiative/Credential-Issuer-Conformance-Criteria#12

  • moved comment after ticket was split

@bluesteens
Copy link
Member Author

bluesteens commented Mar 24, 2023

  • Is Issue appropriate for OCI Architecture
  • Create Steering-Level summary of request
  • Assign Size
  • Assign Priority
  • Assign Label (if needed)
  • OCI affected Artifacts Identified
  • Assign Triage - Artifact Version Target (v x.x.x Milestone)
  • Assign Triage - Interop Profile Version Target (v x.x.x Milestone)
  • Create sub-project (if needed)

Affected Parties (help determine Sunrise/Sunset):

  • Trading Partners
  • Issuers
  • Wallet Solutions
  • PI Verification Solutions

@bluesteens
Copy link
Member Author

bluesteens commented Mar 24, 2023

fix of CI 6.1 could be done alongside issue #15
ref relates to ticket #32

@rceleste125
Copy link

The Trusted Issuer List is on Etherum and is a list of Trusted DIDs. Do the DIDs need to be ETH DIDs?

@bluesteens
Copy link
Member Author

bluesteens commented Mar 30, 2023

separate ticket into 2 based problem areas

@lleifermann
Copy link
Contributor

The Trusted Issuer List is on Etherum and is a list of Trusted DIDs. Do the DIDs need to be ETH DIDs?

The trusted issuer list in its state right now stores the list of trusted dids as strings. So all even though the list resides on Ethereum it supports all types of dids (and other identifiers).

@bluesteens
Copy link
Member Author

atm the Digital-Wallet-Conformance-Criteria talk about did:web security but not did:ethr >> consider adding a section

@alexcolganLD
Copy link
Contributor

A. I read this as general information, but OK with removal as it might lead to confusion.
B. Agree, this seems like an outdated placeholder.
C. How about the following: "To ensure resolution of DID methods, Digital Wallet Providers SHALL support the resolving of all DID methods endorsed by OCI, and SHALL implement at least one DID method."
D. 160+ DID methods exist or are under development, suggest we leave this one alone as a change management process question.

@rceleste125
Copy link

rceleste125 commented Apr 24, 2023

Steering Summary:

  1. Remove the limitation for issuers to be limited to use DID:Web method only (allow all OCI supported methods)
  2. Define what "Support" means for a Wallet.

bluesteens added a commit that referenced this issue May 24, 2023
addresses tix:
#31
#32
main changes in 4.1.2, 4.1.4
@bluesteens
Copy link
Member Author

@bluesteens
Copy link
Member Author

13.7.23: amend - "Digital Wallet Providers SHALL support the resolving of all DID methods endorsed by OCI, and SHALL implement at least one such DID method, i.e. create and maintain the resulting DIDs."

@rceleste125 rceleste125 added the Steering - FYI Technical or internal change. Not necessary for Steering to review. label Jul 17, 2023
bluesteens added a commit that referenced this issue Aug 10, 2023
* description of DID methods

addresses tix:
#31
#32
main changes in 4.1.2, 4.1.4

* further tweaks re DID methods

* 4.1. overview table updated

* minor fixes

* IF002 - reference removed

ref had wrong section number and does not seem needed here

* Update index.html

---------

Co-authored-by: Philipp Bolte <philipp.bolte@spherity.com>
bluesteens added a commit that referenced this issue Sep 26, 2023
* description of DID methods

addresses tix:
#31
#32
main changes in 4.1.2, 4.1.4

* further tweaks re DID methods

* 4.1. overview table updated

* minor fixes

* IF002 - reference removed

ref had wrong section number and does not seem needed here

* Update index.html

* feat: update DID method considerations

Signed-off-by: Philipp Bolte <philipp.bolte@spherity.com>

* feat: support signature for ethereum keys

Signed-off-by: Philipp Bolte <philipp.bolte@spherity.com>

* h2 to h3 formatting

 h2 to   <h3>Securing did:ethr</h3>

---------

Signed-off-by: Philipp Bolte <philipp.bolte@spherity.com>
Co-authored-by: Chris <34170038+bluesteens@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation Steering - FYI Technical or internal change. Not necessary for Steering to review.
Projects
Status: Done - Published
Development

Successfully merging a pull request may close this issue.

4 participants