Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Trust anchors and pivots 00005 #21

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

Conversation

MikeCamel
Copy link
Contributor

@MikeCamel MikeCamel commented Mar 16, 2020

Was incorrectly named (@axelsimon came in with a 00004 before I submitted this).
First draft.

Copy link
Contributor

@mbestavros mbestavros left a comment

Choose a reason for hiding this comment

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

One tiny spelling correction. Otherwise, this makes a lot of sense to me. Good to codify these definitions.

rfc#00005-trust-pivot-introduction Outdated Show resolved Hide resolved
…st-pivot-introduction

00005 had already been taken. :-(
Copy link
Contributor

@npmccallum npmccallum left a comment

Choose a reason for hiding this comment

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

### Trust pivot
A trust pivot is a component which allows a trust relationship
from one entity to another to be transferred, or added, to another
entity. The validity of the pivot assumes the existence of one or more trust anchors.
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be possible to get an example here? The CA example above made the description more clear. But after reading this I am still not clear on how the trust pivot would work.

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've provided a couple. Please let me know what you think.

Copy link
Contributor

Choose a reason for hiding this comment

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

@MikeCamel Thanks! That works well.

Added examples of trust pivots
Clarified trust pivot definition
lkatalin
lkatalin previously approved these changes Mar 30, 2020
Copy link
Contributor

@lkatalin lkatalin left a comment

Choose a reason for hiding this comment

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

My comments have been addressed.

@@ -0,0 +1,111 @@
# 00006 Trust anchors and pivots

Choose a reason for hiding this comment

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

The RFC number here is different than the RFC number in the title of this PR.

rfc#00006-trust-pivot-introduction.md Outdated Show resolved Hide resolved
rfc#00006-trust-pivot-introduction.md Show resolved Hide resolved
Comment on lines +48 to +49
relationship to a trust anchor is assumed - based on the endorsing
authority - rather than derived. One example of a trust anchor

Choose a reason for hiding this comment

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

Let's drop the hyphens and add the word "it" to the end of the sentence, so:

"is assumed based on the endorsing authority rather than derived from it."

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm having trouble putting it together @connorkuehl, could you use the built-in suggestion tool to suggest a diff/patch? (<ctrl+g> apparently)

rfc#00006-trust-pivot-introduction.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@axelsimon axelsimon left a comment

Choose a reason for hiding this comment

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

Lots of interesting ideas and food for thought. I think i'm not the only one for whom more non-technical examples would be helpful, to better grasp the difference between anchors and pivots. I think adding a section that more clearly defines how these two articulate would be very helpful!

Comment on lines +48 to +49
relationship to a trust anchor is assumed - based on the endorsing
authority - rather than derived. One example of a trust anchor
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm having trouble putting it together @connorkuehl, could you use the built-in suggestion tool to suggest a diff/patch? (<ctrl+g> apparently)

rfc#00006-trust-pivot-introduction.md Outdated Show resolved Hide resolved
rfc#00006-trust-pivot-introduction.md Outdated Show resolved Hide resolved
Copy link
Contributor

@lkatalin lkatalin left a comment

Choose a reason for hiding this comment

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

LGTM

Edit: Assuming the commits will be squashed.

Copy link
Contributor

@mbestavros mbestavros left a comment

Choose a reason for hiding this comment

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

Content looks good, but it needs a rebase and organizational changes (numbering, according to #22).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants