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

Improve description for Figure 1 #300

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

aj-stein-nist
Copy link
Collaborator

Closes #268.

@aj-stein-nist aj-stein-nist self-assigned this Aug 7, 2024
@aj-stein-nist aj-stein-nist linked an issue Aug 7, 2024 that may be closed by this pull request
{{fig-concept-relationship}} illustrates entities and processes that comprise a Transparency Service independent of any one use case.
There are three main entities and their associated processes in SCITT:

* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
Copy link
Collaborator

@SteveLasker SteveLasker Aug 13, 2024

Choose a reason for hiding this comment

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

Suggested change
* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
* Issuers that use their credentials to create and register Signed Statements about Artifacts

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I would prefer my edit. I specifically used those phrases to align with the different sections of the diagram because the request was to explain and reference it in the text, removing some of the words and terms misses the requirement.

If not, I would have been less wordy, believe you me. 😄

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see you're trying to encompass Issers make a Statement about an Artifact, use their credentials to create a "Signed Statement", then register them. It just reads longer than needed. The sentence prior was trying to define the three roles. In the call, there was resistance to redefining roles as entities.

I've updated the suggestion a bit, which acknowledges a statement becomes a signed statement, with minimal text.

There are three main entities and their associated processes in SCITT:

* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
* Transparency Services that register Signed Statements and Transparent Statements and in doing produce Receipts
Copy link
Collaborator

@SteveLasker SteveLasker Aug 13, 2024

Choose a reason for hiding this comment

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

Removed the reference to Transparent Statements as those are assembled on the client

Suggested change
* Transparency Services that register Signed Statements and Transparent Statements and in doing produce Receipts
* Transparency Services that evaluate Signed Statements against Registration Policies, producing Receipts upon successful Registration

Copy link
Collaborator

Choose a reason for hiding this comment

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

I disagree with this loss of capabilities. A ledger should be able to take a transparent statement from another ledger and handle it as well.

Copy link
Collaborator

Choose a reason for hiding this comment

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

While accurate, a Signed Statement can be a previously Signed Statement with an attached receipt (Transparent Statement), it's not what the diagram currently shows.

The diagram shows a single signed statement being placed on one, two, or more Transparency Services. Each TS produces a receipt, which can be stapled into a single Signed Statement with two receipts.

The current text, nor the diagram cover the scenario of a Transparent Statement being re-registered. That could be a separate PR if we want to load the diagram with more lines.

The other suggested change describes the Signed Statement must be first evaluated against a registration policy before it's successfully registered.

There are three main entities and their associated processes in SCITT:

* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
* Transparency Services that register Signed Statements and Transparent Statements and in doing produce Receipts
Copy link
Collaborator

Choose a reason for hiding this comment

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

I disagree with this loss of capabilities. A ledger should be able to take a transparent statement from another ledger and handle it as well.

@SteveLasker
Copy link
Collaborator

Editors call notes:

  • Moving above the diagram is good clarity
  • Changing from defining the roles to the overall diagram caused some confusion.

If the PR can maintain the roles, and add diagram clarity, that would be preferred.

Copy link
Collaborator

@SteveLasker SteveLasker left a comment

Choose a reason for hiding this comment

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

Address the editors meeting feedback with some proposals to

  • Move the roles definition above the diagram
  • Reflect the current diagram capabilities

@@ -326,6 +326,16 @@ The SCITT architecture consists of a very loose federation of Transparency Servi
In order to accommodate as many Transparency Service implementations as possible, this document only specifies the format of Signed Statements (which must be used by all Issuers) and a very thin wrapper format for Receipts, which specifies the Transparency Service identity and the agility parameters for the Signed Inclusion Proofs.
Most of the details of the Receipt's contents are specified in the COSE Signed Merkle Tree Proof document {{-COMETRE}}.

{{fig-concept-relationship}} illustrates entities and processes that comprise a Transparency Service independent of any one use case.
There are three main entities and their associated processes in SCITT:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
There are three main entities and their associated processes in SCITT:
This section describes the three main roles and associated processes in SCITT:

{{fig-concept-relationship}} illustrates entities and processes that comprise a Transparency Service independent of any one use case.
There are three main entities and their associated processes in SCITT:

* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
Copy link
Collaborator

Choose a reason for hiding this comment

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

I see you're trying to encompass Issers make a Statement about an Artifact, use their credentials to create a "Signed Statement", then register them. It just reads longer than needed. The sentence prior was trying to define the three roles. In the call, there was resistance to redefining roles as entities.

I've updated the suggestion a bit, which acknowledges a statement becomes a signed statement, with minimal text.

There are three main entities and their associated processes in SCITT:

* Issuers that use Statements about Artifacts and their credentials to create Signed Statements
* Transparency Services that register Signed Statements and Transparent Statements and in doing produce Receipts
Copy link
Collaborator

Choose a reason for hiding this comment

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

While accurate, a Signed Statement can be a previously Signed Statement with an attached receipt (Transparent Statement), it's not what the diagram currently shows.

The diagram shows a single signed statement being placed on one, two, or more Transparency Services. Each TS produces a receipt, which can be stapled into a single Signed Statement with two receipts.

The current text, nor the diagram cover the scenario of a Transparent Statement being re-registered. That could be a separate PR if we want to load the diagram with more lines.

The other suggested change describes the Signed Statement must be first evaluated against a registration policy before it's successfully registered.

Copy link
Collaborator

@SteveLasker SteveLasker 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.

Figure 1 needs a description
3 participants