Re: [RFC PATCH] ArmPkg: add driver to add distro installer HTTP boot options

[Leif Lindholm <leif.lindholm@linaro.org>]

On Tue, Oct 24, 2017 at 03:32:51PM +0200, Laszlo Ersek wrote:
> > Convenience. And given that I only noticed yesterday that the boot
> > fails on HTTP redirects, of somewhat restricted value.
> > 
> > Is that an official policy decision, or just a restriction of the
> > implementation?
> 
> Hm, I'm unsure; the following seems to imply that HttpBootDxe intends to handle redirects:
> 
>         if (HttpBootIsHttpRedirectStatusCode (HttpMessage->Data.Response->StatusCode)) {
>           //
>           // Server indicates the resource has been redirected to a different URL
>           // according to the section 6.4 of RFC7231 and the RFC 7538.
>           // Display the redirect information on the screen.
>           //
>           HttpHeader = HttpFindHeader (
>                  HttpMessage->HeaderCount,
>                  HttpMessage->Headers,
>                  HTTP_HEADER_LOCATION
>                  );
>           if (HttpHeader != NULL) {
>             Print (L"\n  HTTP ERROR: Resource Redirected.\n  New Location: %a\n", HttpHeader->FieldValue);
>           }
> 
> Can you perhaps capture a packet trace and discuss it with Siyuan
> and Jiaxin? Perhaps the server returns a status code that is not
> handled by HttpBootDxe right now.

Oh, it's explicitly printed as a 302 (Found), detected by
HttpBootDxe/HttpBootSupport.c:HttpBootPrintErrorMessage().

See https://www.mail-archive.com/edk2-devel@lists.01.org/msg27819.html

But yes, I should have cc:d Siyuan and Jiaxin (and have now done so).
Can you comment?

Regards,

Leif