Add pref to omit "https://" from address bar (pref'd off by default)
Categories
(Firefox :: Address Bar, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox118 | --- | fixed |
People
(Reporter: annevk, Assigned: mseibert)
References
(Blocks 2 open bugs, Regressed 1 open bug)
Details
(Whiteboard: [fxprivacy] )
Attachments
(2 files, 1 obsolete file)
Updated•10 years ago
|
Comment 3•10 years ago
|
||
Comment 5•10 years ago
|
||
Reporter | ||
Comment 7•10 years ago
|
||
Reporter | ||
Updated•9 years ago
|
Updated•9 years ago
|
Comment 8•9 years ago
|
||
Comment 9•9 years ago
|
||
Comment 10•9 years ago
|
||
Comment 11•9 years ago
|
||
Comment 12•9 years ago
|
||
Comment 13•9 years ago
|
||
Reporter | ||
Comment 14•9 years ago
|
||
Comment 15•9 years ago
|
||
Comment 16•9 years ago
|
||
Comment 17•9 years ago
|
||
Updated•8 years ago
|
Comment 19•8 years ago
|
||
Updated•8 years ago
|
Reporter | ||
Comment 20•7 years ago
|
||
Comment 22•7 years ago
|
||
Comment 23•7 years ago
|
||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 24•5 years ago
|
||
This could be implemented for the new dom.security.https_only_mode
(bug 1613063) and for a browser.urlbar.default-to-https pref.
Comment 25•5 years ago
|
||
(In reply to Jan Andre Ikenmeyer [:darkspirit] from comment #24)
This could be implemented for the new
dom.security.https_only_mode
(bug 1613063) and for a browser.urlbar.default-to-https pref.
This bug is about hiding https://
. When I read browser.urlbar.default-to-https
, I thought that it referred to autocompleting to https by default. I didn't find the pref in mozilla-central nor in Bugzilla, so I filed a new bug with that idea at bug 1628831 .
If this feature were to be implemented, I think that the feature would be more easily discoverable if it were to be called browser.urlbar.trimURLs.https
(false = current behavior, true = omit https as requested in this bug).
Comment 26•5 years ago
|
||
browser.urlbar.trimURLs should hide https instead of http if browser.urlbar.default-to-https is true.
dom.security.https_only_mode could imply browser.urlbar.default-to-https.
Updated•3 years ago
|
Comment 31•3 years ago
|
||
browser.urlbar.default-to-https
doesn't exist anymore, and the URL trimming feature is hard coded to remove a single string, http://
Comment 32•3 years ago
|
||
Oh, to be clear though, simply changing the trimURL method to remove both protocols should cause some problems. I think it needs a more careful look. I tested it like 7 or 8 months ago so maybe that's not the case anymore. But at that time it had some side effects with respect to urlbar copy behavior. I think maybe it also affects uriFixup service
Comment 33•3 years ago
|
||
(In reply to aminomancer from comment #31)
browser.urlbar.default-to-https
doesn't exist anymore, and the URL trimming feature is hard coded to remove a single string,http://
It never existed. I suggested it.
https:// should be hidden instead of http.
http:// should be visible and possibly red.
security.insecure_connection_text.enabled should be set to true.
Comment 34•3 years ago
|
||
Oh, I thought you were trying to give advice. Yes I agree 100% with all of that. The idea of hiding the rare dangerous protocol while showing the ubiquitous safe protocol is perfectly backwards. I don't need to know the page is https, I just need to know when it's not https, especially given that like 99.99% of the pages the average user visits are gonna have valid certs nowadays.
Comment 35•2 years ago
|
||
:adw, I'm happy to take this on if we think this is the right strategy. Personally, I feel like this makes sense especially now that other browsers are essentially doing the exact same thing. It's probably less controversial now than eight years ago, especially given the adoption of https.
It seems pretty actionable:
- Turn on the "Not Secure" text in the URL bar on http:// sites by default
- Trim
https://
, don't trimhttp://
It has the potential of making the URL bar a tad cleaner for the vast majority of use cases.
I was thinking we could go further and omit www.
from the value of the urlbar input for https://
addresses, as we'll still have the ability to recover the full real URL via focusing on the urlbar.
Comment 36•2 years ago
|
||
Comment 37•2 years ago
|
||
A major highly visible change like this needs to go through Product review. This isn't something we as engineers can decide to implement by ourselves. Marco wrote an engineering proposal for it, but it didn't make the 2022H2 plan. So if you'd like to push this forward, the next step would be to bring it up at a team meeting or raise it separately with Chris.
Updated•2 years ago
|
Updated•2 years ago
|
Comment 38•2 years ago
|
||
Any updates on this?
Assignee | ||
Comment 39•1 year ago
|
||
Updated•1 year ago
|
Updated•1 year ago
|
Comment 40•1 year ago
|
||
Comment 41•1 year ago
|
||
Backed out for causing bc failures in browser_preferences_usage.js
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | browser/base/content/test/performance/browser_preferences_usage.js | browser.urlbar.trimHttps should not be accessed more than 32 times. - 100 <= 32 - {"filename":"chrome://mochitests/content/browser/browser/base/content/test/performance/browser_preferences_usage.js","name":"checkPrefGetters","sourceId":598,"lineNumber":42,"columnNumber":14,"sourceLine":"","asyncCause":null,"asyncCaller":null,"caller":{"filename":"chrome:
Comment 42•1 year ago
|
||
Comment 43•1 year ago
|
||
bugherder |
Comment 44•1 year ago
|
||
:mseibert could you please consider nominating this for a relnote? https://wiki.mozilla.org/Release_Management/Release_Notes_Nomination
Comment 45•1 year ago
|
||
(In reply to Donal Meehan [:dmeehan] from comment #44)
:mseibert could you please consider nominating this for a relnote? https://wiki.mozilla.org/Release_Management/Release_Notes_Nomination
This is still disabled by default, we'll target 119 at the earliest for shipping this. In any case, let's work on the relnote when we enable this in nightly.
Assignee | ||
Comment 46•1 year ago
|
||
Updated•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Description
•