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

Outdated TerseBitstringStatusListEntry to BitstringStatusListEntry conversion #19

Open
timothee-haudebourg opened this issue Oct 25, 2024 · 2 comments

Comments

@timothee-haudebourg
Copy link

The new definition of BitstringStatusListEntry includes the statusSize and statusMessages, that cannot be derived from TerseBitstringStatusListEntry.

@msporny
Copy link
Contributor

msporny commented Oct 25, 2024

Yes, statusSize and statusMessages isn't supported in TerseBitstringStatusListEntry because it would bloat the size of the vc-barcode to the point where the VC wouldn't be very compact. We had to make some trade-offs with TerseBitstringStatusListEntry and this was one of them.

Does that create an issue for your use cases, @timothee-haudebourg ?

@timothee-haudebourg
Copy link
Author

It does create an issue because in the newest version of Bitstring Status List, statusSize and statusMessages are stored in BitstringStatusListEntry. Here the BitstringStatusListEntry is supposed to be derived from TerseBitstringStatusListEntry, which does not support those properties.

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

No branches or pull requests

2 participants