No pref to turn off insecure-submit-from-secure warning dialog
Categories
(Core Graveyard :: Security: UI, enhancement)
Tracking
(firefox-esr78 fixed, firefox80 fixed)
People
(Reporter: forums, Assigned: mkaply)
References
()
Details
(Whiteboard: [good next bug])
Attachments
(4 files)
Comment 1•17 years ago
|
||
Comment 3•17 years ago
|
||
Comment 5•16 years ago
|
||
Comment 6•16 years ago
|
||
Comment 7•16 years ago
|
||
Comment 8•16 years ago
|
||
Comment 9•16 years ago
|
||
Reporter | ||
Comment 10•16 years ago
|
||
Comment 11•16 years ago
|
||
Reporter | ||
Comment 12•16 years ago
|
||
Comment 13•16 years ago
|
||
Reporter | ||
Comment 14•16 years ago
|
||
Comment 15•16 years ago
|
||
Comment 16•16 years ago
|
||
Reporter | ||
Comment 17•16 years ago
|
||
Comment 18•16 years ago
|
||
Comment 19•16 years ago
|
||
Comment 20•16 years ago
|
||
Comment 22•16 years ago
|
||
Comment 23•16 years ago
|
||
Comment 24•16 years ago
|
||
Comment 25•16 years ago
|
||
Comment 26•16 years ago
|
||
Comment 27•16 years ago
|
||
Comment 28•16 years ago
|
||
Comment 29•15 years ago
|
||
Comment 31•15 years ago
|
||
Comment 32•14 years ago
|
||
Comment 34•14 years ago
|
||
Comment 35•13 years ago
|
||
Comment 36•12 years ago
|
||
Comment 37•12 years ago
|
||
Updated•11 years ago
|
Updated•11 years ago
|
![]() |
||
Updated•9 years ago
|
Updated•8 years ago
|
Comment 38•7 years ago
|
||
Comment hidden (metoo) |
Comment hidden (metoo) |
Assignee | ||
Comment 41•5 years ago
|
||
I'm going to add a preference for this. The world is different now, most sites are https and the people that need this, still need this.
It will be in about:config only.
Comment 42•5 years ago
|
||
The world is also different now in that there's a much stronger need to avoid insecure sites, and there are more security threats caused by XSRF, and specifically things like posting forms from the web to intranet services. Is there some way that, rather than the preference disabling the prompt entirely, it could list either specific sites or specific ranges of addresses to allow submits from/to, to address the specific use case people have for it?
Assignee | ||
Comment 43•5 years ago
|
||
Comment 44•5 years ago
|
||
Back when https was only used by important sites like banks a full modal dialog block made a lot of sense. That's not the internet of today. If a lot of people were hitting this modal dialog it might be worth the effort to craft a better warning that didn't block submission. Chrome's "degraded lock" that treats it like a mixed image is too subtle; maybe something in the style of the warning we show for passwords on insecure pages would work.
In practice this pref is currently useful mostly for intranet sites. Anyone with a public site will still need to make their site work for the vast majority of Firefox users who won't flip this pref. The users who do flip the pref will be at increased risk on public sites without a warning, but can somewhat rely on "herd immunity" of site authors not doing it if it's going to annoy most Firefox users. Would be nice to have some telemetry on how many people flip this pref because if a lot of people are doing it then it definitely becomes worth the effort to develop a better warning.
I was initially concerned that the patch moved the pref check and bail-out to the top of the check, because then we can't even post a warning to the web console. But on further reflection who would such a warning serve? Users aren't going to look, especially if the pages are "working". Developers are most likely using a Firefox with default values and won't need the warning because they'll see the modal block.
Comment 45•5 years ago
|
||
Filed an enhancement bug 1654046 on changing the modal block to an in-page warning by default. We'll see if anyone wants to run with that.
Comment 46•5 years ago
|
||
Comment 47•5 years ago
|
||
bugherder |
Assignee | ||
Comment 48•5 years ago
|
||
Comment on attachment 9164187 [details]
Bug 436200 - Add a pref to turn off warning submitting secure to insecure. r?dveditz,pbz
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Enterprise specific usecase.
- User impact if declined: No impact to users, admins unable to turn off prompt.
- Fix Landed on Version: 80
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Automated tests.
- String or UUID changes made by this patch:
Comment 49•5 years ago
|
||
Comment on attachment 9164187 [details]
Bug 436200 - Add a pref to turn off warning submitting secure to insecure. r?dveditz,pbz
approved for 78.2esr
Comment 50•5 years ago
|
||
bugherder uplift |
Description
•