Cert exception UI in captive portal error page looks broken and doesn't work
Categories
(Firefox :: Security, defect, P1)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox75 | --- | verified |
People
(Reporter: nhnt11, Assigned: prathiksha)
References
(Depends on 1 open bug)
Details
Attachments
(2 files)
[edited]
The "advanced" section in the captive portal error page currently looks like it's missing information, and the "accept risk and continue" button doesn't seem to do anything.
|
Reporter | |
Updated•4 years ago
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 1•4 years ago
|
||
|
|
Assignee | |
Comment 2•4 years ago
|
||
The patch fixes -
- Working of the "Go Back" and "Accept and Continue" buttons on the captive portal Page
- Fills the advance section with relevant data and it doesn't look broken anymore
TO DO (filing follow-ups) -
- Fix the layout of the captive portal page
- Change the return button text from "Go Back (recommended)" to "Go Back"
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
|
Assignee | |
Updated•4 years ago
|
Pushed by prathikshaprasadsuman@gmail.com: https://hg.mozilla.org/integration/autoland/rev/006c560913f2 Add relevant info to the advanced section of the captive portal page and make the buttons work. r=johannh
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
|
|
||
Comment 6•4 years ago
|
||
Great work! Prathiksha, thank you for landing the patch quicly!
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 7•4 years ago
|
||
A node script to test the captive portal.
|
|
Assignee | |
Comment 8•4 years ago
|
||
Adding steps to reproduce -
a) Download and run the captive.js node script (see attachment) to start a captive portal server
b) Change the captivedetect.canonicalURL pref in about:config to "http://localhost:8080"
c) Start the browser and go to "expired.badssl.com"
d) Click on the "Advanced..." button to open the advanced panel
Actual result: "Go Back(Recommended)" and "Accept Risk and Continue" buttons in the advanced panel of the error page do not work. There are no strings in the advanced panel that provide advanced technical error details and the section looks broken.
Expected results:
- The"Go Back(Recommended)" button should navigate the user to their home page
- The "Accept Risk and Continue" button should add a certificate exception and take the user to expired.badssl.com
- There should be relevant error information in the advanced panel
|
||
Comment 9•4 years ago
|
||
I was able to reproduce this issue with the steps provided in comment 8, using an affected Nightly build 73.0a1 from Jan.
This issue is verified fixed on latest Beta 75.0b7 running Windows 10 x64, mac 10.14 and Ubuntu 18.04 x64.
Description
•