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

Add language from resolution about differentiating using cty #314

Merged
merged 4 commits into from
Oct 16, 2024

Conversation

brentzundel
Copy link
Member

@brentzundel brentzundel commented Oct 9, 2024

Some of the language from the WG's resolution is not yet captured in the specification, in particular:

We will note in particular that the cty property is the point of differentiation
for others that may wish to use identical media types. This group intends to
use application/vc and application/vp as the cty values.

This PR adds that language to the specification, and if merged may address issue #313


Preview | Diff

@decentralgabe decentralgabe linked an issue Oct 9, 2024 that may be closed by this pull request
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Copy link
Collaborator

@selfissued selfissued left a comment

Choose a reason for hiding this comment

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

Please apply my suggested changes to tighten the wording.

@iherman iherman self-requested a review October 10, 2024 06:10
index.html Outdated
way to be the means of differentiating this specification's use
of <code>vc+jwt</code> from others that wish to use an identical
media type.
The <code>cty</code> header parameter value can be used to differentiate between secured content of different types when using <code>vc+jwt</code>.
Copy link
Member

Choose a reason for hiding this comment

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

I think this language needs a bit more work (to be applied to all occurences). I do not yet have a suggestion, but I expect to figure it out today.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@decentralgabe
Copy link
Collaborator

editorial approval; open >1 week; changes accepted; merging.

@decentralgabe decentralgabe merged commit 11817d7 into w3c:main Oct 16, 2024
2 checks 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.

"we can both keep using vc+sd-jwt with differing cty values"
7 participants