New issue
Advanced search Search tips

Issue 822814 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Support --attestation_server in attestation_client like we do for cryptohome

Project Member Reported by drcrash@chromium.org, Mar 16 2018

Issue description

With a similar default of the --va_server flag to the value of the --attestation_server one.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 28 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/fb446791eecb02b9b3fefd9ea9eecbc142df73b9

commit fb446791eecb02b9b3fefd9ea9eecbc142df73b9
Author: Yves Arrouye <drcrash@google.com>
Date: Sat Apr 28 01:05:28 2018

attestation: add support for the test CA

Also change the default certificate type to be a user certificate
to be consistent with cryptohome.

BUG= chromium:822814 
TEST=manual

Change-Id: Ib30f291510549b871c9b04e64a2a7aeeb48eb56a
Reviewed-on: https://chromium-review.googlesource.com/967723
Commit-Ready: Yves Arrouye <drcrash@chromium.org>
Tested-by: Yves Arrouye <drcrash@chromium.org>
Reviewed-by: Andrey Pronin <apronin@chromium.org>

[modify] https://crrev.com/fb446791eecb02b9b3fefd9ea9eecbc142df73b9/attestation/client/main.cc

Status: verified (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, May 31 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/system_api/+/06602ccbabdbc1c4825cec2736f8a8df20fe21ad

commit 06602ccbabdbc1c4825cec2736f8a8df20fe21ad
Author: Yves Arrouye <drcrash@google.com>
Date: Thu May 31 23:22:50 2018

cryptohome: add per-PCA enrollment status

This changes allow us to include information about the new entities
introduced by CL:746146, namely identities and identity
certificates, into the TPM status message.

BUG= chromium:822814 
TEST=manual and tests for CL:746146

Change-Id: I4e6fec94bb5d7b9390ee9289568eeb62bd85b86f
Reviewed-on: https://chromium-review.googlesource.com/982820
Commit-Ready: Yves Arrouye <drcrash@chromium.org>
Tested-by: Yves Arrouye <drcrash@chromium.org>
Reviewed-by: Andrey Pronin <apronin@chromium.org>

[modify] https://crrev.com/06602ccbabdbc1c4825cec2736f8a8df20fe21ad/dbus/cryptohome/rpc.proto

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/98a53d25ec9f858d459e087216b36ea0f00b0d67

commit 98a53d25ec9f858d459e087216b36ea0f00b0d67
Author: Yves Arrouye <drcrash@google.com>
Date: Sat Jun 09 09:19:27 2018

attestation: support CA type in creating cert requests

While there is no server implementation this brings the attestion_client
in line with cryptohome and lays the foundation for the CA type to be
used when adding support for multiple identities in attestationd.

BUG= chromium:822814 
TEST=manual

Change-Id: I5b305272a8da6d3bc63f4efed8c4788ee8653800
Reviewed-on: https://chromium-review.googlesource.com/985796
Commit-Ready: Yves Arrouye <drcrash@chromium.org>
Tested-by: Yves Arrouye <drcrash@chromium.org>
Reviewed-by: Andrey Pronin <apronin@chromium.org>

[modify] https://crrev.com/98a53d25ec9f858d459e087216b36ea0f00b0d67/attestation/common/interface.proto
[modify] https://crrev.com/98a53d25ec9f858d459e087216b36ea0f00b0d67/attestation/client/main.cc

Cc: drcrash@chromium.org
 Issue 826866  has been merged into this issue.

Sign in to add a comment