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

Switch from Product to Target Environment #9

Merged

Conversation

aj-stein-nist
Copy link
Contributor

I do find the explanation of Product approachable, but it is not a formally defined term in RFC9334 and is disjoint with the way RATS WG formally describes the entity and related roles. I would think we might want to consider this phrasing.

Note, I am drafting an email with other information about re-attestation and supportability re email, the next PR will propose other rewrites separate of Product->TE, but this is a specific change I thought worth reviewing separate of that.

@aj-stein-nist aj-stein-nist requested a review from KME July 19, 2024 16:03
@aj-stein-nist aj-stein-nist self-assigned this Jul 19, 2024
@KME
Copy link
Owner

KME commented Jul 19, 2024

OK, I think PRODUCT may have been added by someone else as it was not a term I would have put in caps. My points, and the ones I think are more materially important was in the rest of the text I proposed. I'll include here and can propose the changes directly so they reflect as from me.
Current:

The remote attestation framework shall include provisions within the
system and attestation authority to allow for Product modification.

Over its lifecycle, the Product may experience modification due to:
maintenance, failures, upgrades, expansion, moves, etc..

The customer can chose to:

  • Run remote attestation after product modification, or

  • Not take action and remain un-protected

In the case of Re-Attestation:

  • framework needs to invalidate previous TPM PCR values and tokens,

  • framework needs to collect new measurements,

  • framework needs to maintain history or allow for history to be
    logged to enable change traceability attestation, and

  • framework needs to notify that the previous attestation has been
    invalidated

New:

The posture assessment process shall include provisions within the
system and attestation authority to allow for Product modification.

Over its lifecycle, the Product may experience modification due to:
maintenance, failures, upgrades, expansion, moves, changes in desired assurance levels, etc..

The administrator can choose to:

  • Update the posture assurance policy (may be local) and re-assess posture with a local attestation process, summarizing with a remote attestation to the new policy or level, or

  • Run remote attestation after product modification as an external validation, or

  • Not take action and remain un-protected

In the case of Re-Attestation:

  • framework needs to invalidate previous TPM PCR values and tokens,

Moriarty, et al. Expires 4 January 2025 [Page 6]
Internet-Draft RPASCA July 2024

  • framework needs to collect new measurements,

  • framework needs to maintain history or allow for history to be
    logged to enable change traceability attestation, and

  • framework needs to notify that the previous attestation has been
    invalidated

@aj-stein-nist
Copy link
Contributor Author

OK, I think PRODUCT may have been added by someone else as it was not a term I would have put in caps. My points, and the ones I think are more materially important was in the rest of the text I proposed.

Absolutely, I am about to send an email reply with how I would say we integrate those changes but use the WG terminology and arch concepts, if you/all of us agree. I just did things in a certain order. Email will point to #10, which I just opened to continue that discussion. Email reply coming in a minute!

@KME
Copy link
Owner

KME commented Jul 19, 2024 via email

@aj-stein-nist aj-stein-nist merged commit 7423579 into KME:main Jul 19, 2024
1 check passed
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