Re: Review Request 52847: Fixed docker v2 image manifest protobuf definition.

2016-10-13 Thread Jie Yu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52847/#review152626
---


Ship it!




Ship It!

- Jie Yu


On Oct. 13, 2016, 11:25 p.m., Gilbert Song wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52847/
> ---
> 
> (Updated Oct. 13, 2016, 11:25 p.m.)
> 
> 
> Review request for mesos, Artem Harutyunyan, Jie Yu, Kevin Klues, and Timothy 
> Chen.
> 
> 
> Bugs: MESOS-6173
> https://issues.apache.org/jira/browse/MESOS-6173
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Previously, the protobuf definition for docker v2 image manifest
> is not correct. Some proto fields should not be 'required'. E.g.,
> when using the private docker registry from JFrog, the 'kid' (the
> key ID) may not exist in the JWK from the manifest signature.
> 
> Depending on this reference:
> https://tools.ietf.org/html/rfc7517#section-4.5
> 
> 'kid' should be OPTIONAL.
> 
> This patch re-inspects the specification of JWS and JWK, and
> fixed the wrong defined protobuf fields.
> 
> 
> Diffs
> -
> 
>   include/mesos/docker/v2.proto e2c0a131b8429ca086bc5cd09dccfe3e8d2e50ac 
> 
> Diff: https://reviews.apache.org/r/52847/diff/
> 
> 
> Testing
> ---
> 
> N/A.
> 
> 
> Thanks,
> 
> Gilbert Song
> 
>



Review Request 52847: Fixed docker v2 image manifest protobuf definition.

2016-10-13 Thread Gilbert Song

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52847/
---

Review request for mesos, Artem Harutyunyan, Jie Yu, Kevin Klues, and Timothy 
Chen.


Bugs: MESOS-6173
https://issues.apache.org/jira/browse/MESOS-6173


Repository: mesos


Description
---

Previously, the protobuf definition for docker v2 image manifest
is not correct. Some proto fields should not be 'required'. E.g.,
when using the private docker registry from JFrog, the 'kid' (the
key ID) may not exist in the JWK from the manifest signature.

Depending on this reference:
https://tools.ietf.org/html/rfc7517#section-4.5

'kid' should be OPTIONAL.

This patch re-inspects the specification of JWS and JWK, and
fixed the wrong defined protobuf fields.


Diffs
-

  include/mesos/docker/v2.proto e2c0a131b8429ca086bc5cd09dccfe3e8d2e50ac 

Diff: https://reviews.apache.org/r/52847/diff/


Testing
---

N/A.


Thanks,

Gilbert Song