-
Notifications
You must be signed in to change notification settings - Fork 619
ADCSAdministration PowerShell reference update #2630
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
ADCSAdministration PowerShell reference update #2630
Conversation
|
Docs Build status updates of commit 611ce2f: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
|
||
| ### [Confirm-CAEndorsementKeyInfo](./Confirm-CAEndorsementKeyInfo.md) | ||
| ### [Confirm-CAAttestationIdentityKeyInfo](Confirm-CAAttestationIdentityKeyInfo.md) | ||
| Checks whether the local certification authority trusts secure hardware for identity attestation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you use CA here to be inline with all other descriptions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use "identity key attestation" everywhere instead of "identity attestation". So th e description should be "Checks whether the local CA trusts secure hardware for identity key attestation"
|
|
||
| ### [Confirm-CAEndorsementKeyInfo](./Confirm-CAEndorsementKeyInfo.md) | ||
| ### [Confirm-CAAttestationIdentityKeyInfo](Confirm-CAAttestationIdentityKeyInfo.md) | ||
| Checks whether the local certification authority trusts secure hardware for identity attestation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use "identity key attestation" everywhere instead of "identity attestation". So th e description should be "Checks whether the local CA trusts secure hardware for identity key attestation"
| @@ -0,0 +1,115 @@ | |||
| --- | |||
| description: Checks whether the local certification authority trusts secure hardware for identity attestation. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be updated to: "Checks whether the local CA trusts secure hardware for identity key attestation"
| # Confirm-CAAttestationIdentityKeyInfo | ||
|
|
||
| ## SYNOPSIS | ||
| Checks whether the local certification authority trusts secure hardware for identity attestation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Checks whether the local CA trusts secure hardware for identity key attestation"
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
| The **Confirm-CAAttestationIdentityKeyInfo** cmdlet checks whether the local certification authority (CA) trusts secure hardware, such as a Trusted Platform Module (TPM), for identity attestation. The Attestation Identity Key replaces the Endorsement Key as an identity for the TPM. An Attestation Identity Key is permanently embedded in the security hardware. The public portion of the key helps to recognize genuine security hardware. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"identity attestation" -> "identity key attestation"
| ## DESCRIPTION | ||
| The **Confirm-CAAttestationIdentityKeyInfo** cmdlet checks whether the local certification authority (CA) trusts secure hardware, such as a Trusted Platform Module (TPM), for identity attestation. The Attestation Identity Key replaces the Endorsement Key as an identity for the TPM. An Attestation Identity Key is permanently embedded in the security hardware. The public portion of the key helps to recognize genuine security hardware. | ||
|
|
||
| This cmdlet verifies whether the public certificate connects through a certificate chain to an anchor that the CA trusts. Specify an X509 certificate by using the **Certificate** parameter. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update it to - "This cmdlet verifies whether the Attestation Identity Key (AIK) public certificate ..."
|
|
||
| This cmdlet verifies whether the public certificate connects through a certificate chain to an anchor that the CA trusts. Specify an X509 certificate by using the **Certificate** parameter. | ||
|
|
||
| This cmdlet checks whether the attestation identity public key exists as a file in a folder configured at the local CA for key attestation. Specify the public key by using the **PublicKeyHash** parameter. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use capital letters for attestation identity - "... Attestation Identity Key (AIK) public key exists ..."
…ershell-docs into adcsadministration-v-tishe
|
Docs Build status updates of commit d290a25: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thanks.
|
Hi @tiburd - I got SME approval for this one. Ready for review. |
|
@tiburd It looks like we have three approvals and are ready for merge. |
|
Docs Build status updates of commit f83e85b: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
1871153
One new cmdlet. Other changes are Input and Output.