- From: meacer <notifications@github.com>
- Date: Thu, 14 Dec 2023 11:45:22 -0800
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 14 December 2023 19:45:28 UTC
@meacer commented on this pull request. > + <li> + <p>Optionally, run <a>upgrade an HTTP request</a> algorithm on <var>request</var>. + + <p class=note>HTTPS upgrading only applies to requests with <a>HTTP(S) scheme</a>s, but it's done + in <a>main fetch</a> instead of <a>HTTP fetch</a> to ensure that + <a>upgrade a mixed content <var>request</var> to a potentially trustworthy URL, if appropriate</a> + step runs next and applies to the upgraded request. + Thanks! 1. I'm adding a WPT for the HTTPS Upgrades & referrer interaction at https://github.com/web-platform-tests/wpt/pull/43676 2. I opened issue 1727 for the HSTS & referrer interaction. Since it's a preexisting issue, I'm not sure if it should be a blocker for this PR though. 3. I removed the note about mixed content autoupgrading from the upgrade step. 4. Happy holidays! -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/fetch/pull/1655#discussion_r1427193765 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/fetch/pull/1655/review/1782589636@github.com>
Received on Thursday, 14 December 2023 19:45:28 UTC