Re: [edk2] [PATCH v2 2/2] NetworkPkg: Read HttpTlsCipherList variable and configure it for HTTPS session.

2018-02-11 Thread Wu, Jiaxin
Thanks the comment, I will integrate it when I commit the patch. > -Original Message- > From: Ye, Ting > Sent: Monday, February 12, 2018 11:06 AM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Laszlo Ersek ; Kinney, Michael D > ; Zimmer, Vincent > ; Yao, Jiewen ; Fu, > Siyuan > Subjec

Re: [edk2] [PATCH v2 2/2] NetworkPkg: Read HttpTlsCipherList variable and configure it for HTTPS session.

2018-02-11 Thread Ye, Ting
Hi Jiaxin, In following code, how about use "gEdkiiHttpTlsCipherListGuid" as the variable GUID as to make it consistent with the variable name? + Status = gRT->GetVariable ( + EDKII_HTTP_TLS_CIPHER_LIST_VARIABLE, + &gHttpTlsCipherListGuid, +

[edk2] [PATCH v2 2/2] NetworkPkg: Read HttpTlsCipherList variable and configure it for HTTPS session.

2018-02-10 Thread Jiaxin Wu
v2: * Refine the error handling returned from GetVariable. This patch is to read the HttpTlsCipherList variable and configure it for the later HTTPS session. If the variable is not set by any platform, EFI_NOT_FOUND will be returned from GetVariable service. In such a case, the default CipherList