[MESOS-4294] Protobuf parse should support parsing JSON object containing JSON Null.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.27.0
• Component/s: stout
• Labels: mesosphere
• Sprint: Mesosphere Sprint 26
• Story Points: 1
• Assignee: Gilbert Song
• Reporter: Gilbert Song
• Shepherd: Benjamin Mahler
• Created: 05/Jan/16 19:48
• Updated: 26/Nov/18 12:22
• Resolved: 26/Nov/18 12:22

Description

(This bug was exposed by MESOS-4184, when serializing docker v1 image manifest as protobuf).

Currently protobuf::parse returns failures when parsing any JSON containing JSON::Null. If we have any protobuf field set as `JSON::Null`, any other non-repeated field cannot capture their value. For example, assuming we have a protobuf message:

message Nested {
  optional string str = 1;
  repeated string json_null = 2;
}

If there exists any field containing JSON::Null, like below:

      {
        \"str\": \"message\",
        \"json_null\": null
      }

When we do protobuf::parse, it would return the following failure:

Failure parse: Not expecting a JSON null

Issue Links

• blocks: MESOS-4005 Support workdir runtime configuration from image