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

Making predicateTypes consistent with predicate names #371

Open
marcelamelara opened this issue Jul 3, 2024 · 2 comments
Open

Making predicateTypes consistent with predicate names #371

marcelamelara opened this issue Jul 3, 2024 · 2 comments
Assignees
Labels
bug Something isn't working predicates

Comments

@marcelamelara
Copy link
Contributor

marcelamelara commented Jul 3, 2024

We recently introduced the notion of the predicate name, which we use in a couple ways: as a hint in the envelope mediaType, and in the predicateType URI for predicates in the in-toto/attestation namespace (see step 4).

Most predicates in the in-toto/attestation namespace already follow this convention for the predicateTypes, but we have two that were defined before we introduced this convention and don't:

How should we resolve these two cases?

My recommendation for SCAI is to remove the "attribute-report" piece since it somehow implies that there may be other subtypes of SCAI, and we don't currently support predicate subtypes. This may break existing tooling (mostly in-toto/scai-demos), and we may need to bump the predicate version number.

My suggestion for vuln may be to update the predicate name to vulns.md since that won't break current tools.

Any other thoughts? If this looks good, I'll open a PR to make the relevant changes.

@marcelamelara
Copy link
Contributor Author

Update: We may probably also move the expected naming convention to the predicate template in spec/predicates/templates.

@adityasaky
Copy link
Member

My suggestion for vuln may be to update the predicate name to vulns.md since that won't break current tools.

This seems reasonable to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working predicates
Projects
None yet
Development

No branches or pull requests

2 participants