-
Notifications
You must be signed in to change notification settings - Fork 112
Add section describing use of media types #1055
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
Merged
Merged
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
3c6b8b5
chore: add section describing use of media types with syntaxes
mprorock b1e0616
correct spelling
mprorock 3bab618
adjust language for clarity
mprorock 7c81bc2
correct line breaks
mprorock 59532a5
Apply suggestions from code review
mprorock 76b1200
correct grammar
mprorock bda384a
caps for a SHOULD
mprorock fee2e8c
chore: open issue #1060 to discuss proof in relation to media type
mprorock File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -3409,6 +3409,41 @@ <h3>Syntactic Sugar</h3> | |||||
</section> | ||||||
|
||||||
</section> | ||||||
<section> | ||||||
<h3>Media Types</h3> | ||||||
|
||||||
<p> | ||||||
Media types as defined in [[RFC6838]] serve a useful purpose with verifiable | ||||||
credentials, especially regarding identification of a particular syntax | ||||||
that is in use with a verifiable credential. | ||||||
</p> | ||||||
<p> | ||||||
Syntaxes SHOULD be identified by a media type, and certain conventions as | ||||||
outlined in this section SHOULD be followed when defining or using media types | ||||||
with verifiable credentials. | ||||||
</p> | ||||||
<p> | ||||||
Any media types associated with the core data model are listed in the section on | ||||||
<a href="#iana-considerations">IANA Considerations</a> for registration with IANA. | ||||||
</p> | ||||||
<p class="issue" data-number="1060"> | ||||||
Presence of the term "verifiable" (or a shortened version) in the media type may | ||||||
or may not imply a proof. Original proposed language is tracked in the issue | ||||||
corresponding to this note. | ||||||
</p> | ||||||
<p> | ||||||
Use of the term `credential` in a media type related to a syntax of verifiable | ||||||
credentials <i>without</i> the corresponding use of `verifiable` as described above <i>does not</i> | ||||||
imply the presence of a proof with the <a href="#dfn-credential">credential</a>. | ||||||
</p> | ||||||
<p> | ||||||
At the time of this writing, there are two media types associated with the core data | ||||||
model: `application/credential+ld+json` and `application/verifiable+credential+ld+json`. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I support this change. |
||||||
Other specifications such as [[VC-JWT]] define additional media types as may be | ||||||
required to allow for proper interpretation of the syntax of verifiable credentials | ||||||
encoded in the syntax described by the respective specification. | ||||||
</p> | ||||||
</section> | ||||||
|
||||||
<section> | ||||||
<h3>Proof Formats</h3> | ||||||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Notably, we haven't actually defined the latter one yet, but this is just an informative thing that can be changed as needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exactly - especially given the discussion on #1034 re
verifiable-credential
vsverifiable+credential
this gives us a good way to get alignedThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@msporny has requested to use
vc+ld+json
elsewhere, which I agree with, but we can leave it like this for now I think.