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

content: draft: reliable -> authenticatable and auditable #1143

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

TomHennen
Copy link
Contributor

@TomHennen TomHennen commented Sep 20, 2024

Clarify what type of data source level 3 provides to policy enforcement tools.

'authenticatable and auditable' is more easily understood by the community than 'reliable'. It's also less prone to misinterpretation than 'verifiable' which might sound like a much more thorough process has been done.

fixes #1137

Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for slsa ready!

Name Link
🔨 Latest commit cfd174b
🔍 Latest deploy log https://app.netlify.com/sites/slsa/deploys/670fdc4e32460a0008a1002c
😎 Deploy Preview https://deploy-preview-1143--slsa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@TomHennen TomHennen changed the title reliable -> verifiable content: draft: reliable -> verifiable Sep 20, 2024
@TomHennen
Copy link
Contributor Author

Need one more maintainer to approve. Maybe @arewm can help.

@@ -183,7 +183,7 @@ Intended for:
Organizations that want strong guarantees and auditability of their change management processes.

Benefits:
Provides reliable information to policy enforcement tools.
Provides verifiable information to policy enforcement tools.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we chose the word "reliable" specifically to avoid "verifiable."
I suspect these will never be fully verifiable -- the data required to verify contributor info for instance is enormous and probably PII.

I think the best we can do is make the claims "authentic," IE: we know who made the claims.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm looking for @marcelamelara's thoughts here before I do anything else with this PR.

I also think we're never going to find any terminology that is exactly right here. There are different levels of reliability, verifiability, and authenticity depending on how deep you want to go.

Copy link
Contributor

Choose a reason for hiding this comment

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

My main concern with using "reliable" is that it's a subjective term, much how "trustworthy" is in the eye of the auditor/verifier. I do think "authentic" could work, as might "auditable". @zachariahcox is right that the info may not ever be fully verifiable, but my view is that we're aiming to enable auditing (if not verification) of the process.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I like 'auditable'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Went with "authenticatable and auditable"

@marcelamelara marcelamelara self-requested a review October 11, 2024 19:04
@TomHennen TomHennen changed the title content: draft: reliable -> verifiable content: draft: reliable -> authenticatable and auditable Oct 14, 2024
Copy link
Contributor

@marcelamelara marcelamelara left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the revisions @TomHennen !

Signed-off-by: Tom Hennen <tomhennen@google.com>
Copy link
Member

@mlieberman85 mlieberman85 left a comment

Choose a reason for hiding this comment

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

LGTM

@TomHennen TomHennen merged commit 2a1e32b into slsa-framework:main Oct 16, 2024
6 checks passed
TomHennen added a commit to TomHennen/slsa that referenced this pull request Oct 16, 2024
Updating name of Source Level 3 to make it more clear by
removing the somewhat ambiguous 'Source Provenance' and
including the language from PR slsa-framework#1143 instead.

Signed-off-by: Tom Hennen <tomhennen@google.com>
TomHennen added a commit to TomHennen/slsa that referenced this pull request Oct 23, 2024
Updating name of Source Level 3 to make it more clear by
removing the somewhat ambiguous 'Source Provenance' and
including the language from PR slsa-framework#1143 instead.

Signed-off-by: Tom Hennen <tomhennen@google.com>
TomHennen added a commit that referenced this pull request Oct 23, 2024
Updating name of Source Level 3 to make it more clear by removing the
somewhat ambiguous 'Source Provenance' and including the language from
PR #1143 instead.

fixes #1112

---------

Signed-off-by: Tom Hennen <tomhennen@google.com>
Signed-off-by: Tom Hennen <TomHennen@users.noreply.github.com>
Co-authored-by: Aditya Sirish <8928778+adityasaky@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Revisit 'reliable' language for L3
6 participants