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

Remove status message feature due to lack of implementations. #199

Closed
wants to merge 1 commit into from

Conversation

msporny
Copy link
Member

@msporny msporny commented Feb 23, 2025

This PR removes the status message feature from the specification due to a lack of implementations for the feature:

https://w3c.github.io/vc-bitstring-status-list-test-suite/#Data%20Model:%20BitstringStatusList%20Entry


Preview | Diff

Copy link
Contributor

@David-Chadwick David-Chadwick left a comment

Choose a reason for hiding this comment

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

I think it's a shame to remove this feature, as I can see the utility of it. If no-one has implemented it yet, I suggest this is due to minimalist implementations that want to reduce the cost of implementation, rather than the utility of the feature. Once resources are more abundant I suggest that people will want this feature.
Therefore cannot we make it optional rather than mandatory, and then it does not have to be implemented immediately.

@msporny
Copy link
Member Author

msporny commented Feb 23, 2025

If no-one has implemented it yet, I suggest this is due to minimalist implementations that want to reduce the cost of implementation, rather than the utility of the feature. Once resources are more abundant I suggest that people will want this feature. Therefore cannot we make it optional rather than mandatory, and then it does not have to be implemented immediately.

It was always optional, people just chose to not implement it and now we are out of time. We've given the feature over a year to gather implementations and it failed to do so. A last-minute effort to save the feature is not what we need right now. If people want to implement the feature, there are static CR drafts that include the feature that can be cited as well as a test suite that demonstrates that the feature has been implemented correctly. People can try again in v2.0 if they'd like to add this feature, nothing we're doing here prevents organizations from implementing the feature per the 2nd CR snapshot, which was stable.

I will note that a number of the implementers, ourselves included, think this is a dangerous feature and actively argue against using it with our customers. There are serious privacy issues related with it, both from a individual privacy, and a corporate privacy perspective, that we've found very difficult to implement correctly.

All that to say, this is the path the WG suggested the Editor's take on the call last week and I'm implementing what we had consensus for during the last WG call:

https://www.w3.org/2017/vc/WG/Meetings/Minutes/2025-02-19-vcwg#section1-5

Copy link
Member

@iherman iherman left a comment

Choose a reason for hiding this comment

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

I believe the revision history has to make it clear that this feature has been removed due to the lack of implementation. After all, that is a major change since the CR.

@mkhraisha
Copy link
Contributor

We have an implementation for this feature, so does @brentzundel at TradeVerifyd w3c/vc-bitstring-status-list-test-suite#61

We are passing all tests locally (by bypassing jsonld safe mode) because of missing context terms in the BitstringStatusListEntry we are unable to pass all the test suite test on the current live version.

I have pushed a PR that fixes the test fixtures w3c/vc-bitstring-status-list-test-suite#63 to fix the tests for the StatusMessage feature, however we will not pass it until the context has been fixed.

I have opened an issue under bitstringstatus list ( not sure if that or the Data model repo is the more appropriate one) to outline the missing context terms:
#200

@msporny
Copy link
Member Author

msporny commented Feb 24, 2025

Based on #199 (comment), we have two independent implementations now, closing this PR as unnecessary.

@msporny msporny closed this Feb 24, 2025
@msporny msporny deleted the msporny-remove-messages branch February 24, 2025 23:36
@iherman
Copy link
Member

iherman commented Feb 25, 2025

Based on #199 (comment), we have two independent implementations now, closing this PR as unnecessary.

Yay! I trust they will appear in the implemenation report soon.

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.

5 participants