-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Fallback to HTTP when an upgraded site sees 404 etc #28013
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I think in most cases the connection will just fail. We can detect this by expecting an error page (in a throttle?), check if the We can also add some info/button to the error page to return to |
The above requires |
Verified
|
Brave | 1.51.107 Chromium: 113.0.5672.63 (Official Build) (x86_64) |
---|---|
Revision | 0e1a4471d5ae5bf128b1bd8f4d627c8cbd55f70c-refs/branch-heads/5672@{#912} |
OS | macOS Version 13.4 (Build 22F5049e) |
Steps:
- installed
1.51.107
- launched Brave
- loaded
http://upgradable.arthuredelstein.net/nonexistent.html
- clicked on the Shields icon to note the settings
- confirmed I got an
HTTP 404
error - confirmed I remained on
http://upgradable.arthuredelstein.net/nonexistent.html
- clicked to toggle
Upgrade connections to HTTPS
toOnly connect with HTTPS
- confirmed I remained on
http://upgradable.arthuredelstein.net/nonexistent.html
- confirmed
Continue to site
andGo back
buttons, which worked
example | example | example | example |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Verification PASSED on
Brave | 1.51.107 Chromium: 113.0.5672.63 (Official Build) (64-bit)
-- | --
Revision | 0e1a4471d5ae5bf128b1bd8f4d627c8cbd55f70c-refs/branch-heads/5672@{#912}
OS | Windows 10 Version 21H2 (Build 19044.2846)
Steps:
- installed
1.51.107
- launched Brave
- loaded
http://upgradable.arthuredelstein.net/nonexistent.html
- clicked on the Shields icon to note the settings
- confirmed I got an
HTTP 404
error - confirmed I remained on
http://upgradable.arthuredelstein.net/nonexistent.html
- clicked to toggle
Upgrade connections to HTTPS
toOnly connect with HTTPS
- confirmed I remained on
http://upgradable.arthuredelstein.net/nonexistent.html
- confirmed
Continue to site
andGo back
buttons, which worked
example | example | example | example |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
Verification passed on
Brave | 1.51.107 Chromium: 113.0.5672.63 (Official Build) (64-bit) |
---|---|
Revision | 0e1a4471d5ae5bf128b1bd8f4d627c8cbd55f70c-refs/branch-heads/5672@{#912} |
OS | Ubuntu 18.04 LTS |
![]() |
![]() |
---|
Verification PASSED on
Went through the STR/Cases outlined via #28013 (comment) and ensured everything was working as expected:
Verification PASSED on
Went through the STR/Cases outlined via #28013 (comment) and ensured everything was working as expected:
|
As an improvement to #27141, can we detect when a site that has been upgraded to HTTPS from HTTP shows a top-level error code such as
404
? In that case, we should fall back to HTTP again.For QA testing:
We will direct the browser to visit a non-existent page over HTTP. The browser will attempt HTTPS behind the scenes, and when it gets a
404
code, it should automatically fall back to HTTP.404
error and useshttp://
in the address bar.404
error and useshttps://
in the address bar.(This change landed in 1.52.60)
The text was updated successfully, but these errors were encountered: