Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(722)

Unified Diff: components/policy/proto/device_management_backend.proto

Issue 879233003: Initial RemoteCommandService (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remote-commands
Patch Set: more fixes Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/policy/proto/device_management_backend.proto
diff --git a/components/policy/proto/device_management_backend.proto b/components/policy/proto/device_management_backend.proto
index c908dc9b754fd8beea64b6b1ade7fd6952b1291d..1ae0512c6313fc29cfe4cae0bb5532b0086f550f 100644
--- a/components/policy/proto/device_management_backend.proto
+++ b/components/policy/proto/device_management_backend.proto
@@ -454,6 +454,21 @@ message RemoteCommandResult {
optional bytes payload = 4;
}
+message DeviceRemoteCommandRequest {
+ // The command ID of the last command received from the server until
+ // now. Omitted if no commands have been received yet.
+ optional int64 last_command_unique_id = 1;
+
+ // The execution results of previously fetched commands.
+ // The client should send back a command result whenever possible.
+ repeated RemoteCommandResult command_results = 2;
+}
+
+message DeviceRemoteCommandResponse {
+ // The queue of pending commands.
+ repeated RemoteCommand commands = 1;
+}
+
// Request from device to server for reading policies.
message DevicePolicyRequest {
// The policy fetch request. If this field exists, the request must
@@ -934,6 +949,7 @@ message CheckDevicePairingResponse {
// * register
// * status
// * unregister
+// * remote_commands
//
// * devicetype: MUST BE "1" for Android or "2" for Chrome OS.
// * apptype: MUST BE Android or Chrome.
@@ -943,7 +959,7 @@ message CheckDevicePairingResponse {
// * For register and ping requests
// Authorization: GoogleLogin auth=<auth cookie for Mobile Sync>
//
-// * For unregister, policy, status, and cert_upload requests
+// * For unregister, policy, status, cert_upload and remot commands requests
bartfab (slow) 2015/03/10 16:09:21 Nit: s/remote/remote/
binjin 2015/03/12 11:03:16 Done.
// Authorization: GoogleDMToken token=<dm token from register>
//
// * The Authorization header isn't used for enterprise_check
@@ -963,6 +979,7 @@ message CheckDevicePairingResponse {
// register: register_request
// status: device_status_report_request or session_status_report_request
// unregister: unregister_request
+// remote_commands: remote_command_request
//
//
message DeviceManagementRequest {
@@ -999,6 +1016,9 @@ message DeviceManagementRequest {
// Check if two devices are paired.
optional CheckDevicePairingRequest check_device_pairing_request = 12;
+
+ // Remote command fetching.
+ optional DeviceRemoteCommandRequest remote_command_request = 13;
}
// Response from server to device.
@@ -1053,4 +1073,7 @@ message DeviceManagementResponse {
// Response to check device pairing request.
optional CheckDevicePairingResponse check_device_pairing_response = 13;
+
+ // Response to remote command request.
+ optional DeviceRemoteCommandResponse remote_command_response = 14;
}

Powered by Google App Engine
This is Rietveld 408576698