Re: [whatwg/fetch] HTTPS upgrades proposal (PR #1655)

@annevk commented on this pull request.



> @@ -5157,8 +5279,6 @@ these steps:
       <p>If one of the following is true
 
       <ul class=brief>
-       <li><p><var>response</var>'s <a for=response>type</a> is "<code>error</code>"

No it does not. That doesn't make sense to me for the cases that fail due to streaming as those are not navigation-related.

It makes sense for _connection_ being failure.

It might make sense for the TLS client certificate dialog? Although maybe that in itself should be a fallback sign?

I also don't see how it makes sense in the canceled case.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1655#discussion_r1390767516
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1655/review/1726775996@github.com>

Received on Monday, 13 November 2023 08:43:59 UTC