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

support attached COSE payloads and presentations #33

Merged
merged 3 commits into from
Jul 2, 2024
Merged

Conversation

decentralgabe
Copy link
Collaborator

Fix #31

Also added support for rendering presentation examples, I've only enabled this for vc-jose-cose work for now.

There is also #30 which I believe should follow this once media types settle down.

Copy link
Member

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

Media types in this appear to be incorrect (as of this writing), as many have multiple plus signs, which have again been declared invalid by IETF.

index.html Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@decentralgabe
Copy link
Collaborator Author

@TallTed yes we're in an awkward place here since no media types have (yet) been approved by the IETF

I should update this PR (will shortly) to at least align with vc-jose-cose as of this PR.

@decentralgabe
Copy link
Collaborator Author

nudge @msporny hope we can get this in and then adjust the media types when things settle

@msporny msporny merged commit e5dbc7c into main Jul 2, 2024
1 check passed
@msporny msporny deleted the attached-payloads branch July 2, 2024 22:02
@msporny
Copy link
Member

msporny commented Jul 2, 2024

Merged and released v3.3.0 -- can you please check to make sure it does what you expect it to, @decentralgabe? I tested it locally on the test document you provided and it seemed to work for me.

@decentralgabe
Copy link
Collaborator Author

lgtm thanks @msporny

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.

+cose examples look wrong (diagnostic mode broken?)
3 participants