From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.100; helo=mga07.intel.com; envelope-from=siyuan.fu@intel.com; receiver=edk2-devel@lists.01.org Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 93A0A202E6186 for ; Tue, 24 Oct 2017 19:14:57 -0700 (PDT) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga105.jf.intel.com with ESMTP; 24 Oct 2017 19:18:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,430,1503385200"; d="scan'208";a="167207459" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by fmsmga006.fm.intel.com with ESMTP; 24 Oct 2017 19:18:41 -0700 Received: from fmsmsx158.amr.corp.intel.com (10.18.116.75) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.319.2; Tue, 24 Oct 2017 19:18:41 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx158.amr.corp.intel.com (10.18.116.75) with Microsoft SMTP Server (TLS) id 14.3.319.2; Tue, 24 Oct 2017 19:18:41 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.213]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.93]) with mapi id 14.03.0319.002; Wed, 25 Oct 2017 10:18:39 +0800 From: "Fu, Siyuan" To: Leif Lindholm , Laszlo Ersek CC: "edk2-devel@lists.01.org" , "daniel.thompson@linaro.org" , Ard Biesheuvel , "Wu, Jiaxin" Thread-Topic: [edk2] [RFC PATCH] ArmPkg: add driver to add distro installer HTTP boot options Thread-Index: AQHTSm4Xz3mUBq2AK0mUqm5DSWCLo6LyZ0MAgAAHFgCAAA/cgIAAE/EAgAFG54A= Date: Wed, 25 Oct 2017 02:18:38 +0000 Message-ID: References: <20171021131049.23844-1-ard.biesheuvel@linaro.org> <0f440c7d-f590-57c1-9f34-f9c8a54c81d6@redhat.com> <20171024123605.mpcyaemq55s6v7e2@bivouac.eciton.net> <20171024144414.rlwaygmqsirgovlh@bivouac.eciton.net> In-Reply-To: <20171024144414.rlwaygmqsirgovlh@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_IC x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZDMxOWIwMTgtNmM2Yi00MmZiLTkwNWEtYmQ4OGQwMDE3NjA1IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjIuNS4xOCIsIlRydXN0ZWRMYWJlbEhhc2giOiJoZTQ2Q2JSc0s3U2NhanhnSFdMR2Q4NEhiamJ5eFE3Y1YrRlJtbFhPS3JxQmpHRWpvYldBcHVrUVNYSW1Ra25uIn0= dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [RFC PATCH] ArmPkg: add driver to add distro installer HTTP boot options X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 25 Oct 2017 02:14:57 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi, Leif and Laszlo The HTTP restriction is actually not supported now. If server replies a HTT= P redirect response, the current HTTP boot driver will only print new URL a= ddress on the screen then abort the HTTP boot. It won't attempt to download= the image from the new address. BestRegards Fu Siyuan > -----Original Message----- > From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > Sent: Tuesday, October 24, 2017 10:44 PM > To: Laszlo Ersek > Cc: edk2-devel@lists.01.org; daniel.thompson@linaro.org; Ard Biesheuvel > ; Fu, Siyuan ; Wu, Jiaxin > > Subject: Re: [edk2] [RFC PATCH] ArmPkg: add driver to add distro installe= r > HTTP boot options >=20 > 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 t= o > 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 =3D HttpFindHeader ( > > HttpMessage->HeaderCount, > > HttpMessage->Headers, > > HTTP_HEADER_LOCATION > > ); > > if (HttpHeader !=3D 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. >=20 > Oh, it's explicitly printed as a 302 (Found), detected by > HttpBootDxe/HttpBootSupport.c:HttpBootPrintErrorMessage(). >=20 > See https://www.mail-archive.com/edk2-devel@lists.01.org/msg27819.html >=20 > But yes, I should have cc:d Siyuan and Jiaxin (and have now done so). > Can you comment? >=20 > Regards, >=20 > Leif