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

update secp256k1-2019 w3id.org URLs to be versioned and consistent #525

Merged
merged 3 commits into from
Aug 28, 2023

Conversation

bnewbold
Copy link
Contributor

@bnewbold bnewbold commented Aug 3, 2023

I'm not sure if a small PR like this is a helpful way to contribute; I hope so! Don't think this gets anywhere near a "substantive contributions to specifications" but can do paperwork if needed.

The primary motivation here is that URL https://w3id.org/security/suites/secp256k1-2019 is valid, but does not point to an actual JSON-LD document, just a landing page which links to the v1 document. Most of the other URLs in the registry document point directly to the JSON-LD document.

For the secp256k1recovery-2020 URL, the href already has /v1, but the display URL does not. Seems like they should be consistent like the other URLs.

The background context which led me here was the https://didlint.ownyourdata.eu/validate tool wanting @context URLs to match what is in the spec registry document (which for secp256k1-2019 means no trailing /v1), while the @context should point directly to a JSON-LD doc, resulting in a contradiction.

Hypothetically it might be most correct to list multiple URLs in the "JSON-LD" columns in this document, both versioned and not.


Preview | Diff

index.html Show resolved Hide resolved
@bnewbold
Copy link
Contributor Author

Gentle bump on this; can it be merged?

It looks like @mprorock (intentionally not mentioning) is no longer an editor in this repo, so won't be reviewing.

@OR13 OR13 merged commit 87fdc56 into w3c:main Aug 28, 2023
1 check passed
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.

4 participants