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

VC schema: clarify optionality of parameters #9

Open
8 of 31 tasks
bluesteens opened this issue Jun 2, 2023 · 2 comments · May be fixed by #11
Open
8 of 31 tasks

VC schema: clarify optionality of parameters #9

bluesteens opened this issue Jun 2, 2023 · 2 comments · May be fixed by #11
Assignees
Labels
documentation Improvements or additions to documentation Roadmap The Issue represents a feature that will be part of the OCI Roadmap. Steering - FYI Technical or internal change. Not necessary for Steering to review.

Comments

@bluesteens
Copy link
Member

bluesteens commented Jun 2, 2023

Steering summary:

The current OCI Credential Schema leaves room for interpretation in its statement of Verifiable Presentation metadata requirements. Suggest to rework the spec to introduce clarity.


Detailed Description:

The OCI Credential Schema spec 7.12 Verifiable Presentation of DSCSA-ATP-Credential lists several metadata fields and references the original W3C VC Data Model spec.

Issue:

OCI does not explicitly state whether the listed items are mandatory or optional. The W3C spec states or may be interpreted as stating that at least some of the items may be optional. Wallet implementers may use different interpretations, which could hamper interoperability through differing implementations.

Suggestion:

Clarify in the OCI spec which parameters are mandatory/optional.

Note:

Look at Credential (7.1) and Credential Presentation (7.12) Metadata.
related ticket: Open-Credentialing-Initiative/Digital-Wallet-Conformance-Criteria#58


Triage:

  • Is Issue appropriate for OCI Architecture
  • 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

Affected OCI Artifact

  • Schema Document
  • Identity Schema
  • ATP Schema
  • Issuer Conformance Criteria
  • Wallet Conformance Criteria
  • VRS Solution Conformance Criteria
  • Wallet API Specification
  • Governance Document
  • Conformance Program
  • OCI Website
  • Internal Process

Change Category (Guides Steering Review)

- Steering/Industry Review

  • Business-Level (May affect business operations)
  • OCI Governance, Policy or website feature

- Steering/Industry Notification

  • Technical-Level (Does not affect business operations)
  • OCI Internal Process or Infrastructure

Communication

  • Website
  • Newsletter
  • email:
  • Other:
@bluesteens bluesteens converted this from a draft issue Jun 2, 2023
@bluesteens bluesteens added the documentation Improvements or additions to documentation label Jun 2, 2023
@alexcolganLD alexcolganLD moved this from 🔖 Initial Triage to P&A Triage Review & Consensus in OCI Change Management Jun 7, 2023
@bluesteens
Copy link
Member Author

also consider specifying the exact W3C doc version in the confo criteria

@bluesteens
Copy link
Member Author

also consider whether para 7.1 needs to have requirements stated more explicitly: https://open-credentialing-initiative.github.io/schemas/specification/v1.0.0/#dscsa-atp-credential-metadata

@rceleste125 rceleste125 moved this from P&A Triage Review & Consensus to Steering Business Review in OCI Change Management Jun 8, 2023
@bluesteens bluesteens moved this from Steering Business Review to 📋 Ready for work in OCI Change Management Jun 19, 2023
@rceleste125 rceleste125 added the Steering - FYI Technical or internal change. Not necessary for Steering to review. label Jul 17, 2023
@bluesteens bluesteens linked a pull request Aug 16, 2023 that will close this issue
@bluesteens bluesteens moved this from 📋 Ready for work to 🏗 In progress in OCI Change Management Aug 16, 2023
@bluesteens bluesteens moved this from 🏗 In progress to 👀 P&A Completed Work Approval in OCI Change Management Aug 16, 2023
@bluesteens bluesteens moved this from 👀 P&A Completed Work Approval to 🏗 In progress in OCI Change Management Aug 16, 2023
@bluesteens bluesteens added the On Hold The issue is planned to start later in the year, or is pending input. Do not include in reviews. label Sep 13, 2023
@bluesteens bluesteens added Roadmap The Issue represents a feature that will be part of the OCI Roadmap. and removed On Hold The issue is planned to start later in the year, or is pending input. Do not include in reviews. labels Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Roadmap The Issue represents a feature that will be part of the OCI Roadmap. Steering - FYI Technical or internal change. Not necessary for Steering to review.
Projects
Status: 🏗 In progress
Development

Successfully merging a pull request may close this issue.

5 participants