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

Clarifying updates #1

Merged
merged 2 commits into from
Dec 12, 2022
Merged

Clarifying updates #1

merged 2 commits into from
Dec 12, 2022

Conversation

setrofim
Copy link
Collaborator

  • Specify that the ear.status tier must be no higher trust than what is
    implied by the trust vector, but could be lower. This allows for the
    possibility that the deployment-specific policy would evaluate the
    evidence to be less trustworthy than the sum of its parts, without
    necessarily overriding the trust level of any individual vector claim
    (e.g. via a blacklist of specific devices).
  • Clarify that ear.raw-evidence should include the signature if present.
  • expand the meaning of "iat" claim name, as it is not immediately
    obvious. It is explained in the linked reference, but expanding it
    here save the reader a click.
  • Change "compute" to "evaluate" in the ear.appraisal-policy-id, as
    "compute" implies a higher level of rigidity than may be applied by
    relying (or some other third) party-provided policies.

- Specify that the ear.status tier must be no higher trust than what is
  implied by the trust vector, but could be lower. This allows for the
  possibility that the deployment-specific policy would evaluate the
  evidence to be less trustworthy than the sum of its parts, without
  necessarily overriding the trust level of any individual vector claim
  (e.g. via a blacklist of specific devices).
- Clarify that ear.raw-evidence should include the signature if present.
- expand the meaning of "iat" claim name, as it is not immediately
  obvious. It _is_ explained in the linked reference, but expanding it
  here save the reader a click.
- Change "compute" to "evaluate" in the ear.appraisal-policy-id, as
  "compute" implies a higher level of rigidity than may be applied by
  relying (or some other third) party-provided policies.

Signed-off-by: setrofim <[email protected]>
@setrofim
Copy link
Collaborator Author

Policy ID calarification

Adding as a comment here, rather than in the commit message as not necessarily related to the EAR spec per se.

The way we currently use policy ID in Veraision is to identify additional evaluation policies. The definition for EAR implies that "policy ID" covers not only what we call a "policy" in Veraison, but also the "scheme". Currently, if no additional policy is specified, and the evidence is evaluated only according to the identified scheme, we do not set the policy ID in the result. Should be setting it to a scheme-defined value instead?

Copy link
Owner

@thomas-fossati thomas-fossati left a comment

Choose a reason for hiding this comment

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

👍

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.

2 participants