Closed
Bug 38008
Opened 25 years ago
Closed 23 years ago
Basic authentication dialog should display more information.
Categories
(Core :: Networking: HTTP, defect, P3)
Core
Networking: HTTP
Tracking
()
VERIFIED
FIXED
Future
People
(Reporter: bugzilla, Assigned: darin.moz)
References
()
Details
Attachments
(4 files)
The dialog that comes up for a password protected site is wrong. The following
is wrong:
- first of all it's ugly...:)
- "Confirm password"? I haven't even entered the password once.
- The website that asks for the username/password should be included into the
dialog.
Reporter | ||
Comment 1•25 years ago
|
||
Comment 2•25 years ago
|
||
marking rfe, changing plat/os to all.. although i agree completely.
it would be nice to have a lock icon (ala PSM) to see
if the site is over SSL or not..
Severity: normal → enhancement
OS: Windows 2000 → All
Hardware: PC → All
Summary: Prompt for password protected sites is wrong → [rfe] Prompt for password protected sites is wrong
Reporter | ||
Comment 3•25 years ago
|
||
Currently is also not able to localize it. Perhaps a XUL dialog would be better?
Comment 4•25 years ago
|
||
You should really consider splitting off all these bugs into separate reports.
Comment 6•25 years ago
|
||
Reassign to owner of Networking component.
Assignee: norris → gagan
Component: Security: General → Networking
QA Contact: junruh → tever
Comment 8•25 years ago
|
||
UI text... reassigning to myself
Assignee: ben → verah
Target Milestone: --- → Future
First of all: this is a regression. It has worked correctly for
more than one month. (I must identify myself to a Squid proxy server,
so I know).
It has to do with:
element.setAttribute("hidden", "true");
(line 201 in chrome/global/content/commonDialog.js).
If I change that line to:
element.setAttribute("sytle","visibility: collapse;");
the second element (first password) is eminently visible, but
at least, if I use the third field, I can identify myself properly.
Hope this helps.
Comment 10•25 years ago
|
||
This seems to be a duplicate of 27784.(and a whole buch of others)
Lengthy discussion there
Comment 11•25 years ago
|
||
nope. read the comment from Mathew Thomas in bug 27784. that is linux-only,
and for a _completely_ inaccessable password dialog. this is an RFE not covered
by that bug.
Comment 12•25 years ago
|
||
han.holl: this bug is not about the password field not working (bug 27784), it is
about the basic authentication dialog needing better design in general. And this
bug shouldn't be fixed before that one, because the authentication dialog is
acting as the primary test-case for that bug.
So, let's assume that IE's dialog (as attached to this report) is better than the
current Mozilla one -- not too much of a contentious claim, really. But what
could still be improved from IE's dialog?
* The title refers only to a `network password', when you also have to enter a
user name.
* The whole dialog has too much empty padding, and the horizontal and vertical
padding is inconsistent.
* It probably shouldn't use the word `please' -- you shouldn't pretend that the
computer actually cares about whether you decide to authenticate or not.
* It says `type your user name and password', which is inaccurate if you are
pasting or entering your data by some method other than typing.
* If either the server name or the realm name were long (which they might well
be), they would fall off the edge of the dialog. They should be in non-editable
text fields, so they can be scrolled to see the entire name if they get too
long.
* The `Realm' field is conceptually more important than the `Site' field, so
`Realm' should come first.
* The `Site' field is poorly named, since server != `site'.
* The `Realm' field isn't aligned properly (either horizontally or vertically)
with the rest of the dialog.
* The `User Name' and `Realm' labels should have colons, since they're not part
of sentences.
* In `User Name', `name' doesn't need to be capitalized.
* `Save this password in your password list' is more technical and less
results-explaining than it needs to be.
* The dialog doesn't give an option to remember the password just for the current
session.
So, removing all those problems, we get:
+---------------------------------------------------+
|[] Log in :::::::::::::::::::::::::::::::::::::::::|
+---------------------------------------------------+
| ,.__ Enter your user name and password for |
| `'"" realm [Site Administration ] |
| at server [www.gamal.dk ] |
| |
| _User name: [| ] |
| _Password: [ ] |
| |
| [ ] Remember these for when I log in again |
| [ ] Remember just for this session |
| |
| ( Help ) ( Cancel ) (( Log in )) | <-- platform-specific order
+---------------------------------------------------+
The checkboxes would be mutually exclusive -- you could select zero or one of
them, but as soon as you tried to check both of them the first one you had
checked would become unchecked. This isn't ideal -- three radio buttons would be
the technically correct solution -- but I think the simplicity would outweigh the
slightly strange behavior in this case.
Anyone see any further room for improvement in the dialog design?
Summary: [rfe] Prompt for password protected sites is wrong → [rfe] Basic authentication dialog needs better layout
Comment 13•25 years ago
|
||
Feh. Here's the design again, without Bugzilla's extra line breaks:
+---------------------------------------------------+
|[] Log in :::::::::::::::::::::::::::::::::::::::::|
+---------------------------------------------------+
| ,.__ Enter your user name and password for |
| `'"" realm [Site Administration ] |
| at server [www.gamal.dk ] |
| |
| _User name: [| ] |
| _Password: [ ] |
| |
| [ ] Remember these for when I log in again |
| [ ] Remember just for this session |
| |
| ( Help ) ( Cancel ) (( Log in )) | <-- platform-specific order
+---------------------------------------------------+
Comment 14•25 years ago
|
||
See also bug 38933. Since this dialog, and the dialog required for that bug,
carry out equivalent functions (doing authentication), we might want the first
sentences of the dialogs to be roughly the same.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 15•24 years ago
|
||
*** Bug 51691 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 16•24 years ago
|
||
Changed URL since old one didn't work anymore...
I've also filed bug 51696 to be able to access the "realm" and "server" info in
the commonDialog.xul file.
Reporter | ||
Comment 17•24 years ago
|
||
Comment 18•24 years ago
|
||
Comment 19•24 years ago
|
||
Design tweaks:
* added realm to title bar (and dependency on bug 51696)
* made server/realm field labels follow Mac/Windows capitalization/punctuation
convention
* changed to `Remember this password', because it's clearer, and remembering the
password implies remembering the user name to use it with anyway
* added security info (and dependency on bug 38019)
* removed `Remember just for this session', because Mozilla always does (duh).
+--------------------------------------------------------+
|[] Log in to ITS :::::::::::::::::::::::::::::::::::::::|
+--------------------------------------------------------+
| ,.__ Enter your user name and password for |
| `'"" Server: [cantsb.canterbury.ac.nz ] |
| Realm: [ITS ] |
| __________________________________ |
| _User name: [|_________________________________] |
| _Password: [__________________________________] |
| [_] _Remember this password |
| ,. |
| [] This login is not using a secure connection. |
| |
| [?] ( Cancel ) (( Log in )) |
+--------------------------------------------------------+
Comment 20•24 years ago
|
||
I really prefer the way Netscape 4.x does it, more than anything. Also, I
believe that a non-editable textbox with the server and the realm is the *worst*
thing to UI-wise, as users may wonder whether or not they're supposed to fill
in, or at least edit, the fields, and be worried when it doesn't work
(non-editable). Besides, I've never met a realm that didn't fit in 4.x, but we
can use trunication ("Big Bad Long Realm Of The...") if it grows too long (you
may say that this may hide important information, but a very long realm is never
necessary). Also, I don't believe the title should change at all. If it had to
change at all, I'd rather that it used "Log In To [site]" rather than "Log In
To [realm]". However, I'd still rather use "Authentication Required". Based on
all this, I copied and pasted your dialog into Notepad and edited it (used
Windows style button arrangements) for a fake entry with a realm of "Better
Browser", without the quotes, and a site of www.mozilla.org:
+--------------------------------------------------------+
| Authentication Required::::::::::::::::::::::::::::::|
+--------------------------------------------------------+
| ,.__ Enter your username and password for "Better |
| `'"" Browser" at www.mozilla.org: |
| |
| __________________________________ |
| _User name: [|_________________________________] |
| _Password: [__________________________________] |
| [_] _Remember this password |
| ,. |
| [] Remember these for when I log in again |
| |
| [?] (( OK )) ( Cancel ) |
+--------------------------------------------------------+
Reporter | ||
Comment 21•24 years ago
|
||
A lot of the text is inside c++ code...:
http://lxr.mozilla.org/mozilla/source/netwerk/protocol/http/src/nsHTTPChannel.cp
p#2049
bad, bad, bad...
Comment 22•24 years ago
|
||
[Reassigning to component owner]
Assignee: verah → neeti
Status: ASSIGNED → NEW
Comment 23•24 years ago
|
||
Adding to meta authentication bug
Comment 24•24 years ago
|
||
*** Bug 77369 has been marked as a duplicate of this bug. ***
Comment 25•23 years ago
|
||
Netscape 4 used to has realm and host name in the auth dialog => 4xp keyword.
AFAIK, 4xp's are considered bugs, not RFE's
Comment 26•23 years ago
|
||
I filed a similar bug for NNTP authentication dialog - bug #85637
Comment 27•23 years ago
|
||
+mozilla 1.0, basic design seems to have reached enough consensus where this
should be doable...
Keywords: mozilla1.0
Comment 28•23 years ago
|
||
Is this fixed (except for bug 38019)? I see realm and host information in basic
auth dialog on BuildID 20010806000
Reporter | ||
Comment 29•23 years ago
|
||
actually this bug is kind of fixed since this bug is about "more info". We
currently show all the info the other browsers show.
Perhaps a new bug should be opened that has to do better better UI for the
dialog.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 30•23 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•