-
Notifications
You must be signed in to change notification settings - Fork 34
Open
Labels
bugSomething isn't workingSomething isn't working
Description
What version of the package are you using?
- Veraison: c46900d
- ratsd: veraison/ratsd@0851e95 (latest commit as an issue)
Does this issue reproduce with the latest release?
Yes
Issue
The Veraison sevsnp scheme scheme expects the following media types:
services/scheme/sevsnp/scheme.go
Lines 10 to 11 in c46900d
| EvidenceMediaTypeTSMCbor = "application/vnd.veraison.tsm-report+cbor" | |
| EvidenceMediaTypeTSMJson = "application/vnd.veraison.configfs-tsm+json" |
While ratsd uses those two types both with cbor and json encoding in different places:
- https://github.com/veraison/ratsd/blob/1ac35b2e2184baa6c18baf41531aa7caa31051eb/tokens/tsm-report.go#L13-L14
- https://github.com/veraison/ratsd/blob/1ac35b2e2184baa6c18baf41531aa7caa31051eb/tokens/tsm-report.go#L13-L14
In the IANA registry there is only:
vnd.veraison.tsm-report+cbor: https://www.iana.org/assignments/media-types/application/vnd.veraison.tsm-report+cborvnd.veraison.tsm-report+json: https://www.iana.org/assignments/media-types/application/vnd.veraison.tsm-report+json
Is there a reason for application/vnd.veraison.configfs-tsm to exist or can we standardize Veraison and ratsd on vnd.veraison.tsm-report?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working