Support nested policy protobufs in policy test server |
||||||
Issue description
Currently it doesn't nested message if they are not repeated. For example:
message Inner {
repeated int array = 1;
optional string value = 2;
}
message Middle {
optional Inner inner = 1;
}
message Outer {
optional Middle middle = 1;
}
While such structure is not possible for user policies, for device policies the structure could be such nested.
,
Jul 17 2017
,
Aug 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/70e69d2fbfa306af4ed5b7ce254dd98fe3edebb5 commit 70e69d2fbfa306af4ed5b7ce254dd98fe3edebb5 Author: Daria Yakovleva <yakovleva@google.com> Date: Mon Aug 07 13:36:14 2017 Upgrade policy_testserver.py Now policy test server supports nested protos and enum types. It is needed for a forthcoming policy "OffHours" which has nested proto type. For example: message Inner { repeated int array = 1; optional string value = 2; } message Middle { optional Inner inner = 1; } message Outer { optional Middle middle = 1; } Bug: 743147 Change-Id: If337312868b73709d248390c60a3553ba4898e99 Reviewed-on: https://chromium-review.googlesource.com/576090 Commit-Queue: Daria Iakovleva <yakovleva@google.com> Reviewed-by: Maksim Ivanov <emaxx@chromium.org> Reviewed-by: Sergey Poromov <poromov@chromium.org> Cr-Commit-Position: refs/heads/master@{#492309} [modify] https://crrev.com/70e69d2fbfa306af4ed5b7ce254dd98fe3edebb5/chrome/browser/policy/test/policy_testserver.py
,
Oct 19 2017
,
Jan 22 2018
,
Jan 23 2018
,
Mar 5 2018
Verified fixed. The policy works correctly (see attached screenshot with the policy example). Chrome OS: 10323.47.0 Chrome: 65.0.3325.110 Device: Kevin |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by poromov@chromium.org
, Jul 14 2017