-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fix incompatibility/deprecation issue between issuance, wallet and EDC #832
Comments
@jjeroch : Does this summarize the issue we discussed this morning, can you add missing details or forward the issue to people who can? @stephanbcbauer : This is a feature that we should add to the 24/12 release and coordinate between the teams. It is about ensuring that all relevant components support BitStringStatusList and StatusList2021 for now. This enables us to switch at any time convenient in the future. |
I added the milestone 24.12, because we need it on the board. Let's discuss this feature tomorrow in our meeting |
I've added a ticket for the portal team to support the BitString Status List: see #404 |
Just as a small and admittedly detailed comment from the EDC side: yes, we will support BitStringStatusList in future versions (tx-edc probably This is permitted by the spec and still satisfies conformance. There are also two other implementation issues on which all involved parties should have alignment:
|
We should have a look into this during the planning of 25/03. |
Hi @MaximilianHauer, I have created this feature in the past as a consequence of a discussion that was done late in the planning of 24/12. We could never properly discuss the topic with all stakeholders as MIW died away shortly after this item was created. As you can see from Pauls comment, the EDC is somehow prepared with a unilateral implementation which makes some assumption on how the ButString status list is implemented. As the item is labeled with SSI, I co-assign you and kindly ask you to bring this to the SSI expert group for coordination. |
@MaximilianHauer if this feature is still valid, and we want to have it in the planning for 25.03, we should definitely "rerefine" it and also upgrade it to the new feature template |
I see work for the following components:
IIRC the VC 1.1 does not contain BitStringStatusList in the context schema - https://www.w3.org/2018/credentials/v1, therefore, we will need to adapt the creation of the VC, therefore also adapt the issuer component. To be validated. |
Also we need to check whether BDRS is affected by this change since it also validates VP for Directory API access. |
Some hints from Release Management (@ther3sa) and Tractus-X Project Lead (@stephanbcbauer)
|
This ticket is in Portal backlog now. @lgblaumeiser Can you move it to SiG-Release please? |
@Phil91 as discussed could you please provide the input for ssi-credential-issuer and dim-middler-layer |
for the ssi-credential-issuer we need to merge PR #298 and clarify with sap that the solution will work with the existing StatusList2021 and a Statuslist of type BitstringStatusList. if only one of both works than all issued credentials need to be reissued for the newly created status list BitstringStatusList. for the dim-middle-layer the PR #125 needs to be merge |
@ma3u mentioned ⇾ is there a transition phase planned? After evaluation, the dependency labels should be set. |
@lgblaumeiser : Can you please add the following persons as assignes to this ticket: @hkny @evegufy, Marcel Ruland and Ronald Koenig |
somehow cant add @rkxx and @marcelruland |
@hkny are you referring to the shape/contents of the |
Apparently commenting on the issue is sufficient. You should be able to assign me now, and as soon as @rkxx also comments he should be assignable as well. Maybe a committer can add both @rkxx and myself as contributors to the organisation? |
@evegufy, @hkny, @rkxx, and myself just discussed this. This change should not be looked at in isolation but instead be considered a part of the larger update to the VC 2.0 standard. StatusList2021 does define its own context, but BitStringStatusList does not. It is only included in the VC 2.0 context. So to integrate BitStringStatusList with VC 1.1, we would have to define our own context, effectively abandoning standards. There is no motivation to give BitStringStatusList higher priority than any other aspect of the VC 2.0 update. The update to VC 2.0 itself should be postponed for now, as it (and also BitStringStatusList) is still in the recommendation candidate phase. While it is unlikely to go through major changes before reaching recommendation status, there are no significant disadvantages to staying with VC 1.1 until then. |
Follow-up from the SSI EG, since there's new information:
We stick with our recommendation to treat BitStringStatusList as a part of the update to VC 2.0 and to wait with the VC 2.0 update until the standard reaches recommendation status. |
Overview
Explain the topic in 2 sentences
The ecosystem needs to transition from the deprecated StatusList2021 to the current BitStringStatusList format for credential revocation. This requires coordinated changes across multiple components including EDC, Wallet, and ssi-credential-issuer.
What's the benefit?
Technical Implementation
StatusList2021
BitStringStatusList
Data Structure
Status Entry Properties
Common Properties
Unique Features of BitStringStatusList
Performance and Privacy
BitStringStatusList Advantages
What are the Risks/Dependencies?
Migration Implementation Strategy:
Breaking Changes:
Detailed explanation
Current implementation
Proposed improvements
Feature Team
Contributor
Committer
User Stories
Acceptance Criteria
Test Cases
Test Case 1: Dual Format Support
Steps
Expected Result
Architectural Relevance
The following items are ensured after this issue is implemented:
This feature aligns with our current architectural guidelines
The impact on the overall system architecture has been assessed
Potential risks or conflicts with existing architecture has been assessed
Justification: The implementation requires a coordinated breaking change but follows W3C standards and improves overall system security.
Additional information
The text was updated successfully, but these errors were encountered: