Device enterprise registration with a certificate.

components/policy/core/common/cloud/cloud_policy_client.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/policy/core/common/cloud/cloud_policy_client.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 35 matching lines @@
 CloudPolicyClient::Observer::~Observer() {}
 
 void CloudPolicyClient::Observer::OnRobotAuthCodesFetched(
     CloudPolicyClient* client) {}
 
 CloudPolicyClient::CloudPolicyClient(
     const std::string& machine_id,
     const std::string& machine_model,
     const std::string& verification_key_hash,
     DeviceManagementService* service,
-    scoped_refptr<net::URLRequestContextGetter> request_context)
+    scoped_refptr<net::URLRequestContextGetter> request_context,
+    SigningService* signing_service)
     : machine_id_(machine_id),
       machine_model_(machine_model),
       verification_key_hash_(verification_key_hash),
       device_mode_(DEVICE_MODE_NOT_SET),
       submit_machine_id_(false),
       public_key_version_(-1),
       public_key_version_valid_(false),
       invalidation_version_(0),
       fetched_invalidation_version_(0),
       service_(service),                  // Can be null for unit tests.
+      signing_service_(signing_service),
       status_(DM_STATUS_SUCCESS),
       request_context_(request_context) {
 }
 
+CloudPolicyClient::CloudPolicyClient(
+    const std::string& machine_id,
+    const std::string& machine_model,
+    const std::string& verification_key_hash,
+    DeviceManagementService* service,
+    scoped_refptr<net::URLRequestContextGetter> request_context) :
+    CloudPolicyClient(machine_id,
+                      machine_model,
+                      verification_key_hash,
+                      service,
+                      request_context,
+                      nullptr /* signing_service */) {
+}
+
 CloudPolicyClient::~CloudPolicyClient() {
   base::STLDeleteValues(&responses_);
 }
 
 void CloudPolicyClient::SetupRegistration(const std::string& dm_token,
                                           const std::string& client_id) {
   DCHECK(!dm_token.empty());
   DCHECK(!client_id.empty());
   DCHECK(!is_registered());
 
   dm_token_ = dm_token;
   client_id_ = client_id;
   request_jobs_.clear();
   policy_fetch_request_job_.reset();
   base::STLDeleteValues(&responses_);
 
   NotifyRegistrationStateChanged();
 }
 
+void CloudPolicyClient::SetClientId(const std::string& client_id) {
+  if (client_id.empty()) {
+    // Generate a new client ID. This is intentionally done on each new
+    // registration request in order to preserve privacy. Reusing IDs would
+    // mean the server could track clients by their registration attempts.
+    client_id_ = base::GenerateGUID();
+  } else {
+    client_id_ = client_id;
+  }
+}
+
 void CloudPolicyClient::Register(em::DeviceRegisterRequest::Type type,
                                  em::DeviceRegisterRequest::Flavor flavor,
                                  const std::string& auth_token,
                                  const std::string& client_id,
                                  const std::string& requisition,
                                  const std::string& current_state_key) {
   DCHECK(service_);
   DCHECK(!auth_token.empty());
   DCHECK(!is_registered());
 
-  if (client_id.empty()) {
-    // Generate a new client ID. This is intentionally done on each new
-    // registration request in order to preserve privacy. Reusing IDs would mean
-    // the server could track clients by their registration attempts.
-    client_id_ = base::GenerateGUID();
-  } else {
-    client_id_ = client_id;
-  }
+  SetClientId(client_id);
 
   policy_fetch_request_job_.reset(
       service_->CreateJob(DeviceManagementRequestJob::TYPE_REGISTRATION,
                           GetRequestContext()));
   policy_fetch_request_job_->SetOAuthToken(auth_token);
   policy_fetch_request_job_->SetClientID(client_id_);
 
   em::DeviceRegisterRequest* request =
       policy_fetch_request_job_->GetRequest()->mutable_register_request();
   if (!client_id.empty())
@@ skipping 10 matching lines @@
   request->set_flavor(flavor);
 
   policy_fetch_request_job_->SetRetryCallback(
       base::Bind(&CloudPolicyClient::OnRetryRegister, base::Unretained(this)));
 
   policy_fetch_request_job_->Start(
       base::Bind(&CloudPolicyClient::OnRegisterCompleted,
                  base::Unretained(this)));
 }
 
+void CloudPolicyClient::RegisterWithCertificate(
+    em::DeviceRegisterRequest::Type type,
+    em::DeviceRegisterRequest::Flavor flavor,
+    const std::string& pem_certificate_chain,
+    const std::string& client_id,
+    const std::string& requisition,
+    const std::string& current_state_key) {
+  DCHECK(service_);
+  DCHECK(!pem_certificate_chain.empty());
+  DCHECK(!is_registered());
+
+  if (!signing_service_) {

[pastarmovj, 2016/08/22 15:09:43]

Can you explain why is this not a DCHECK or CHECK even? Shouldn't it be the
caller that needs to know beforehand that he can do cert based registration or
not?

[The one and only Dr. Crash, 2016/08/22 16:00:01]

I started with a DCHECK, but thought that since there typically is a backup
enrollment method, return an error that can be recovered from is friendlier.

This being said I am fine either way and if a DCHECK fits better with the rest
of the code I'll revert to it.

[pastarmovj, 2016/08/23 08:46:02]

+Daren, what do you think?

+    LOG(ERROR) << "Cryptographic misconfiguration. Cannot sign request.";
+    em::DeviceManagementResponse response;
+    OnRegisterCompleted(DM_STATUS_CANNOT_SIGN_REQUEST, 0, response);
+    return;
+  }
+
+  SetClientId(client_id);
+
+  em::CertificateBasedDeviceRegistrationData data;
+  data.set_certificate_type(em::CertificateBasedDeviceRegistrationData::
+      ENTERPRISE_ENROLLMENT_CERTIFICATE);
+  data.set_device_certificate(pem_certificate_chain);
+
+  em::DeviceRegisterRequest* request = data.mutable_device_register_request();
+  if (!client_id.empty())
+    request->set_reregister(true);
+  request->set_type(type);
+  if (!machine_id_.empty())
+    request->set_machine_id(machine_id_);
+  if (!machine_model_.empty())
+    request->set_machine_model(machine_model_);
+  if (!requisition.empty())
+    request->set_requisition(requisition);
+  if (!current_state_key.empty())
+    request->set_server_backed_state_key(current_state_key);
+  request->set_flavor(flavor);
+
+  signing_service_->SignData(data.SerializeAsString(),
+      base::Bind(&CloudPolicyClient::OnRegisterWithCertificateRequestSigned,
+                 base::Unretained(this)));
+}
+
+void CloudPolicyClient::OnRegisterWithCertificateRequestSigned(bool success,
+    em::SignedData signed_data) {
+  if (success) {

[pastarmovj, 2016/08/22 15:09:43]

nit: maybe reverse this like if (!success) { error; return; } ...
this way the bulk of the code will not be indented :)

[The one and only Dr. Crash, 2016/08/22 16:00:01]

Done.

+    policy_fetch_request_job_.reset(
+        service_->CreateJob(
+            DeviceManagementRequestJob::TYPE_CERT_BASED_REGISTRATION,
+            GetRequestContext()));
+    policy_fetch_request_job_->SetClientID(client_id_);
+    em::SignedData* signed_request = policy_fetch_request_job_->GetRequest()->
+        mutable_cert_based_register_request()->mutable_signed_request();
+    signed_request->set_data(signed_data.data());
+    signed_request->set_signature(signed_data.signature());
+    signed_request->set_extra_data_bytes(signed_data.extra_data_bytes());
+    policy_fetch_request_job_->SetRetryCallback(
+        base::Bind(&CloudPolicyClient::OnRetryRegister,
+                   base::Unretained(this)));
+    policy_fetch_request_job_->Start(
+        base::Bind(&CloudPolicyClient::OnRegisterCompleted,
+                   base::Unretained(this)));
+    return;
+  }
+  em::DeviceManagementResponse response;
+  OnRegisterCompleted(DM_STATUS_CANNOT_SIGN_REQUEST, 0, response);
+}
+
 void CloudPolicyClient::SetInvalidationInfo(int64_t version,
                                             const std::string& payload) {
   invalidation_version_ = version;
   invalidation_payload_ = payload;
 }
 
 void CloudPolicyClient::FetchPolicy() {
   CHECK(is_registered());
   CHECK(!types_to_fetch_.empty());
 
@@ skipping 569 matching lines @@
 
 void CloudPolicyClient::NotifyRobotAuthCodesFetched() {
   FOR_EACH_OBSERVER(Observer, observers_, OnRobotAuthCodesFetched(this));
 }
 
 void CloudPolicyClient::NotifyClientError() {
   FOR_EACH_OBSERVER(Observer, observers_, OnClientError(this));
 }
 
 }  // namespace policy