Re: 回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-04-11 Thread Maciej Rabeda
Patchset merged: https://github.com/tianocore/edk2/pull/2756 On 8 kwi 2022 11:07, Rabeda, Maciej wrote: Alright. Reviewed-by: Maciej Rabeda Merge soon. On 7 kwi 2022 14:55, Oliver Steffen wrote: On Thu, Apr 7, 2022 at 2:46 PM Rabeda, Maciej wrote: 1. In HttpResponseWorker():   if (A

Re: 回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-04-08 Thread Maciej Rabeda
Alright. Reviewed-by: Maciej Rabeda Merge soon. On 7 kwi 2022 14:55, Oliver Steffen wrote: On Thu, Apr 7, 2022 at 2:46 PM Rabeda, Maciej wrote: 1. In HttpResponseWorker(): if (AsciiStrnCmp (HttpHeaders, "HTTP/1.0", AsciiStrLen ("HTTP/1.0")) == 0) { DEBUG ((DEBUG_VERBOSE, "HT

Re: 回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-04-07 Thread Oliver Steffen
On Thu, Apr 7, 2022 at 2:46 PM Rabeda, Maciej wrote: > 1. In HttpResponseWorker(): > > if (AsciiStrnCmp (HttpHeaders, "HTTP/1.0", AsciiStrLen > ("HTTP/1.0")) == 0) { >DEBUG ((DEBUG_VERBOSE, "HTTP: Server version is 1.0. Setting > Connection close.\n")); >HttpInstance->Connect

Re: 回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-04-07 Thread Maciej Rabeda
Hi Gerd, Sorry, I can spare very little time for reviews these days... I am alright with the patch with two cosmetic changes. 1. In HttpResponseWorker():     if (AsciiStrnCmp (HttpHeaders, "HTTP/1.0", AsciiStrLen ("HTTP/1.0")) == 0) {   DEBUG ((DEBUG_VERBOSE, "HTTP: Server version is 1.0.

Re: 回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-04-07 Thread Gerd Hoffmann
On Wed, Mar 23, 2022 at 09:19:09AM +0800, gaoliming wrote: > Maciej, Jiaxin and Siyuan: > Can you help review this fix? Ping. Anything blocking the merge of these bugfixes? thanks, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online

回复: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-03-22 Thread gaoliming
redhat.com; Oliver Steffen > > 主题: [edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two) > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2720 > > This set of patches fixes booting from HTTP/1.0 servers. > It also improves the interaction with HTTP/1.1 servers by recog

[edk2-devel] [PATCH v3 0/5] Http Fixes (Take Two)

2022-03-22 Thread Oliver Steffen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2720 This set of patches fixes booting from HTTP/1.0 servers. It also improves the interaction with HTTP/1.1 servers by recognizing the 'Connection: close' header field, which fixes a problem with servers that close the connection after a 404-err

Re: [edk2-devel] [PATCH v3 0/5] Http Fixes

2022-03-16 Thread Gerd Hoffmann
On Tue, Mar 15, 2022 at 02:42:24PM +0100, Oliver Steffen wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2720 > > This set of patches fixes booting from HTTP/1.0 servers. > It also improves the interaction with HTTP/1.1 servers by recognizing > the 'Connection: close' header field, wh

[edk2-devel] [PATCH v3 0/5] Http Fixes

2022-03-15 Thread Oliver Steffen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2720 This set of patches fixes booting from HTTP/1.0 servers. It also improves the interaction with HTTP/1.1 servers by recognizing the 'Connection: close' header field, which fixes a problem with servers that close the connection after a 404-err