Closed Bug 1719309 Opened 3 years ago Closed 3 years ago

HTTPS-First test for bad certifactes

Categories

(Core :: DOM: Security, task, P2)

task

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: t.yavor, Assigned: t.yavor)

References

(Blocks 1 open bug)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

When https-first opens a site with bad certificate, we might want to skip to show a warning page.
Instead https-first could downgrade the request to http

Severity: -- → S4
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [domsecurity-active]

Currently https-first acts like that:

  1. Assume we have a website example.com that supports both http and https.
  2. If the initial request is http://example.com https-first is upgrading it, receiving an error and downgrades to http
  3. If the initial request is https://example.com : Since it is an https request which wasn't upgraded by https-first, https-first doesn't downgrade it
  4. Warning page gets displayed.

Probably we would need https-first to interfer https requests, to skip every warning page. But if https-first is interfering, it would have max. two options. One is to accept the bad cert, so we don't have actually an https connection but at top level it would still look like one (? not sure if it is still kind of an https connection... ).
The other option would be to downgrade to http but probably in most cases the http site redirects to the https site, at least that would be reasonable for a website that thinks it supports https.

So a simple downgrade won't work, in most cases.

Flags: needinfo?(ckerschb)
Flags: needinfo?(arthur)

Downgrading to http when encountering a bad certificate error seems correct to us.

Flags: needinfo?(ckerschb)
Flags: needinfo?(arthur)
Pushed by mozilla@christophkerschbaumer.com:
https://hg.mozilla.org/integration/autoland/rev/e207e7944cd2
HTTPS-First test for bad certifactes. r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: