Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-12 Thread Laszlo Ersek
On 02/11/18 03:33, Wu, Jiaxin wrote: > Hi Laszlo, > > Besides the compatibility consideration, we'd better *not* put > CipherList and CaCertificate into one variable. I didn't suggest to put them in the same variable -- I meant to put them in separate variables, just the two variables should

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-10 Thread Wu, Jiaxin
Hi Laszlo, Besides the compatibility consideration, we'd better *not* put CipherList and CaCertificate into one variable. In the future, we prefer to manage the CaCertificate with other cert configuration items together (e.g. HostPublicCert, HostPrivateCert, etc ) rather than the parameters

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-09 Thread Laszlo Ersek
On 02/09/18 06:22, Fu, Siyuan wrote: > Hi, Jiaxin > > I think we can remove the "TlsCipherList.h" to another name like > "HttpTlsCipherListVariable.h" to highlight that the variable is only > used for HTTP configuration. And also the variable name and GUID > name. If we are renaming

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-08 Thread Wu, Jiaxin
e, Ting <ting...@intel.com>; Laszlo Ersek > <ler...@redhat.com>; Yao, Jiewen <jiewen@intel.com> > Subject: Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to > configure TLS CipherList. > > Hi, Jiaxin > > I think we can remove the "TlsCipherList.h&qu

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-08 Thread Li, Ruth
t.com>; Yao, Jiewen <jiewen....@intel.com> Subject: Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList. Hi, Jiaxin I think we can remove the "TlsCipherList.h" to another name like "HttpTlsCipherListVariable.h" to highlight that the

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-08 Thread Wu, Jiaxin
Thanks the comment, I will refine the series patch. > -Original Message- > From: Fu, Siyuan > Sent: Friday, February 9, 2018 1:23 PM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Laszlo Ersek ; Kinney, Michael D > ;

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-08 Thread Fu, Siyuan
Hi, Jiaxin I think we can remove the "TlsCipherList.h" to another name like "HttpTlsCipherListVariable.h" to highlight that the variable is only used for HTTP configuration. And also the variable name and GUID name. Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Friday,