Clicking Get Certificate causes the Add Security Exception window to expand indefinitely

VERIFIED FIXED in Firefox 60

Status

()

defect
P1
normal
VERIFIED FIXED
2 years ago
7 months ago

People

(Reporter: joemaffei, Assigned: keeler)

Tracking

(Depends on 1 bug, {regression})

55 Branch
mozilla61
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox57 wontfix, firefox58 wontfix, firefox59 wontfix, firefox60 verified, firefox61 verified)

Details

(Whiteboard: [psm-assigned], )

Attachments

(2 attachments)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.94 Safari/537.36

Steps to reproduce:

- Go to a URL that triggers the "Your connection is not secure" (like http://expired.badssl.com)
- click Advanced, then Add Exception
- every time you click the Get Certificate button the Add Security Exception window gets bigger -- and it stays that way even after closing/reopening the browser!


Actual results:

The Add Security Exception window gets bigger every time you click the Get Certificate button.


Expected results:

The Add Security Exception window should not get bigger every time you click the Get Certificate button.
Version: 57 Branch → Trunk
I cannot reproduce this issue in Fx57, 58 and Nightly 59 on Win10 1709. However, I recently encountered the height of the window was exceeded the screen height and had to remove xulstore.json for fix it.
Component: Untriaged → Security
I can reproduce this issue by clicking "View" button (opening "Certificate Viewer") before clicking "Get Certificate" button.
This issue can be reproduced based on comment 2. This issue will eventually make the dialog unusable.
Status: UNCONFIRMED → NEW
Has STR: --- → yes
Ever confirmed: true
Keywords: regression
Version: Trunk → 55 Branch
Regression range:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=167ed7d3545b6b8ff0c8fee6593a505bd181b08b&tochange=67e5ea975d53131939ef6d399ff1558fd9475c0f
Blocks: 1347859
Has Regression Range: --- → yes
Component: Security → Security: PSM
Product: Firefox → Core
Priority: -- → P5
Whiteboard: [psm-backlog]
Duplicate of this bug: 1428437
Duplicate of this bug: 1443262
This has picked up a couple dupes now. Is this something we may want to bump up higher than P5?
This will make it better. Still not perfect, but at least the dialog will be usable.
Assignee: nobody → dkeeler
Flags: needinfo?(dkeeler)
Priority: P5 → P1
Whiteboard: [psm-backlog] → [psm-assigned]
Comment on attachment 8963346 [details]
bug 1424311 - don't persist the certificate exception dialog's size so it doesn't grow unboundedly

https://reviewboard.mozilla.org/r/232248/#review237710

This seems reasonable to me, but tbh I'm not super familiar with how this part of XUL works. I suppose we can try it out on Nightly though.
Attachment #8963346 - Flags: review?(jjones) → review+
Thanks for the review.
Pushed by dkeeler@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/02213fd3a9c8
don't persist the certificate exception dialog's size so it doesn't grow unboundedly r=jcj
https://hg.mozilla.org/mozilla-central/rev/02213fd3a9c8
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Thanks for fixing this, David! Please nominate this for Beta approval when you get a chance so it makes the next ESR.
Flags: needinfo?(dkeeler)
Comment on attachment 8963346 [details]
bug 1424311 - don't persist the certificate exception dialog's size so it doesn't grow unboundedly

Approval Request Comment
[Feature/Bug causing the regression]: bug 1347859
[User impact if declined]: add certificate exception dialog can become unusable eventually
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: no
[Needs manual test from QE? If yes, steps to reproduce]:  yes:

1. visit https://expired.badssl.com
2. click "advanced"
3. click "add exception"
4. click "view"
5. close the dialog that comes up (the certificate viewer)
6. click "get certificate" repeatedly
Step 6 should not cause the dialog to keep growing in size

[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: limited change, doesn't affect core functionality of this dialog, just the display of it
[String changes made/needed]: none
Flags: needinfo?(dkeeler)
Attachment #8963346 - Flags: approval-mozilla-beta?
Comment on attachment 8963346 [details]
bug 1424311 - don't persist the certificate exception dialog's size so it doesn't grow unboundedly

Approved for 60.0b9, thanks.
Attachment #8963346 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
I successfully reproduced the issue on Nightly 59.0a1 (2017-12-08) under Windows 10 (x64) using the STR from Comment 0 and Comment 2.

The issue is no longer reproducible on Firefox 60.0b9 and latest Nightly 61.0a1 (2018-04-03) under Windows 10 (x64). I also checked on Ubuntu 16.04 (x64) and macOS 10.12 and everything is working as expected.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Duplicate of this bug: 1441304
The update means that the dialog no longer grows larger, but it does not reset the dialog to the original size and does not allow the user to manually resize. This means that if you are already afflicted with this problem then this update does not fix it. It only prevents it from getting worse or impacting new users. Running 61.01 x64 Win10.

I've followed the instructions on #1389547 which through trial and error I worked out you have to quit Mozilla before you make the change to xulstore.json :

chrome://pippki/content/exceptionDialog.xul":{"exceptiondialog":{"screenX":"132","screenY":"8","width":"955","height":"200"}}

But it would be helpful if a sane exceptiondialog height in newversion(x) or resetting it back to the default if <> default was implemented.
Depends on: 1521034
You need to log in before you can comment on or make changes to this bug.