Closed Bug 239241 Opened 20 years ago Closed 20 years ago

(seamonkey port) option to show/display saved passwords

Categories

(Toolkit :: Password Manager, enhancement, P3)

enhancement

Tracking

()

VERIFIED FIXED
mozilla1.7.4

People

(Reporter: jmd, Assigned: steffen.wilberg)

References

Details

(Keywords: fixed-aviary1.0)

Attachments

(1 file)

Port fix from bug 78754.

At least when Mozilla used simple Base64 encoding, my simple perl script could
unobscure the contents of the password file. Now the new obscuring code is too
annoying for me to figure out to update my script, and I can't get at my
passwords at all. Incredibly frustrating.

Here is "mozpwview", for anyone with passwords still Base64'd and needing to
access them:

use MIME::Base64;
while (<>) {
        if (s/^~//) {
                chomp;
                print decode_base64($_) . "\n";
        } else {
                print;
        }
}
Before we add this, we should offer a simple way to specify a master password.
Adding dependency to bug 218694.
Severity: normal → enhancement
Depends on: 218694
OS: Linux → All
Hardware: PC → All
Summary: (seamonkey port) Password manager should have mode to display saved password → (seamonkey port) option to show/display saved passwords
We should go ahead and do this since it's just porting a fix from seamonkey.
Flags: blocking1.0+
Priority: -- → P3
(In reply to comment #2)
> We should go ahead and do this since it's just porting a fix from seamonkey.

(In reply to comment #2)
> We should go ahead and do this since it's just porting a fix from seamonkey.

One request, if it is not too much of a job to accomplish, would be to
incorporate the ability to print out a copy of the password list and/or save as
a text file so that it can be taken with you...there are occasions when having a
printed copy of my site passwords would be very useful when accessing the web
from a machine other than my usual one.

Thanks
Attached patch patchSplinter Review
Assignee: bryner → steffen.wilberg
Status: NEW → ASSIGNED
Comment on attachment 151396 [details] [diff] [review]
patch

This is basically a port of attachment 143220 [details] [diff] [review], which is the "updated final
patch" for bug 78754 for Seamonkey. I also included the whitespace changes.

I inserted the button labeling code in Startup() above "// load password
manager items" because LoadSignons() throws an exception if you cancel the
master password prompt upon opening the password manager. We've got to label
the buttons before that. I'll leave catching the exception to another bug.

I removed <script src="chrome://global/content/strres.js"/> from
passwordManager.xul since we don't use srGetStrBundle().

I had to rework ConfirmShowPasswords() because we don't use the pref
"wallet.crypto". I was able to simplify the function a bit. The code works fine
with and without a master password being set, I only hope I got all the
comments right.
Attachment #151396 - Flags: review?(bryner)
Steffen Wilberg , why don't include this code into FireFox nightly builds to
test it ?

I like so much this option in Mozilla 1.7 and is very useful , this is the
reason i still use the Mozilla suite instead of FireFox.
Juan, my patch needs to be reviewed first.
Bitrot (locale files move) is near...
Requesting blocking1.0RC1 because of localization impact.
Flags: blocking-aviary1.0RC1?
Attachment #151396 - Flags: review?(bryner) → review+
I checked that in 2004-07-16 09:15. My first checkin ever!
Trunk checkin will have to wait until tinderboxen look greener there.
Flags: blocking-aviary1.0RC1?
Keywords: fixed-aviary1.0
Target Milestone: --- → Firefox1.0beta
*** Bug 251450 has been marked as a duplicate of this bug. ***
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7) Gecko/20040716
Firefox/0.9.1+

WFM
Trunk checkin 2004-07-19 06:37.

The only modification I had to make was in passwordManager.js:
-    } else if (column=="passwordCol") {
+    } else if (column.id=="passwordCol") {

-> fixed.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
(In reply to comment #13)
> Trunk checkin 2004-07-19 06:37.
> 
> The only modification I had to make was in passwordManager.js:
> -    } else if (column=="passwordCol") {
> +    } else if (column.id=="passwordCol") {
> 
> -> fixed.

First off, thanks for your work on this.

Question:  Is this incorporated in the FF 20040719 trunk build for OS X?  When I
go to Firefox>Preferences>Privacy>Saved Passwords>View Saved Passwords>Show
Passwords>Yes I still get an empty/blank window even though I know that there
are saved passwords (I logged in to post this question using that build and the
user ID and password appeared automatically).  I looked in my user.js file and
did not see any language other than some entries I have previously made to
include URL autofill and to enable the use of my default email client when
clicking on links while browsing in FF.  

One comment as to the accessibility of this is that there are a many more clicks
to gain access to this (assuming that the way I described above is the way it is
supposed to be accessed) than in Mozilla where I click on Tools>Password Manager
(selecting Manage Passwords from its submenu)>Show Passwords>Yes.  This amounts
to four clicks for Mozilla and six for FF with FF requiring two clicks to close
the window instead of one for Mozilla.  I do not know what the "conventions" are
as to where matters such as this are to be displayed, but the way this is done
in Mozilla seems, to me at least, to be less complicated.

Lastly, is there a plan to allow the web sites/User ID/Passwords to be printed
to a hard copy at some point in time?  I believe that there is more use for this
capability than people realize.  

Cheers!

> Question:  Is this incorporated in the FF 20040719 trunk build for OS X?
If you see the "Show Passwords" button: yes.

The dialog starts with two columns, "Site" and "Username". Upon clicking the
"Show Passwords" button, it displays a third column with the corresponding
passwords.

I don't know why the dialog is empty for you. It isn't for me. Passwords are
stored in signons.txt, not user.js.

The question whether to really show passwords is done for security reasons, and
it's in Mozilla as well. The user might want to reconsider showing passwords
when somebody is looking over his shoulder. If there's a master password
specified, one has to enter it at this point.

> Lastly, is there a plan to allow the web sites/User ID/Passwords to be printed
> to a hard copy at some point in time? 
None that I know of. And there doesn't seem to be a bug for that. Feel free to
file one.
(In reply to comment #15)
> > Question:  Is this incorporated in the FF 20040719 trunk build for OS X?
> If you see the "Show Passwords" button: yes.
> 
> The dialog starts with two columns, "Site" and "Username". Upon clicking the
> "Show Passwords" button, it displays a third column with the corresponding
> passwords.
> 
> I don't know why the dialog is empty for you. It isn't for me. Passwords are
> stored in signons.txt, not user.js.
> 
> The question whether to really show passwords is done for security reasons, and
> it's in Mozilla as well. The user might want to reconsider showing passwords
> when somebody is looking over his shoulder. If there's a master password
> specified, one has to enter it at this point.
> 
> > Lastly, is there a plan to allow the web sites/User ID/Passwords to be printed
> > to a hard copy at some point in time? 
> None that I know of. And there doesn't seem to be a bug for that. Feel free to
> file one.

I had all of the right columns & etc showing up but simply got a blank screen
even though passwords were obviously being used.  There was no signons.txt file
present in the profile and I tried adding one and so on without success.  I did
a reinstallation (after removing the existing profile) and imported data, but it
did not work out.  I just cleared the FF profiles and did a new install without
importing anything.  A signons.txt file appeared after I went to a web site that
required a log in and had done a manual log in as I was not recognized (no old
cookies & etc).  It looks like the problem was in using an existing/old profile
that also had the XXXXXXXX.s file which contained the passwords (in encrypted
form I believe).  I have brought in my user.js file and old bookmarks and will
just manually (re)enter passwords as I go to other sites.  All works well now.  

I do not know if others would like to have the ability to printout their
passwords/user IDs or not.  It seems useful to me, both as a backup and to take
with me if I am going somewhere else where I will have access to a different
computer.  What do you think?

Thanks again!
V w/ Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a3) Gecko/20040725 Firefox/0.9.1+

Thanks Steffen!
Status: RESOLVED → VERIFIED
I have the same problem with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a3)
Gecko/20040809 Firefox/0.9.1+ updated vrom CVS on Aug 09.
I do have a signons.txt, it is obviously used, but no entries are shown in the
window. Unfortunately I did not check when the displaying of stored passwords
vanished. There must be a different way in interpreting the signons.txt when
actually looking up for passwords and when displaying in the stored password window.
As I even don't remember all my passwords, I'd be happy if someone can tell me
how to get access to my stored passwords.


> As I even don't remember all my passwords, I'd be happy if someone can tell me
> how to get access to my stored passwords.
> 
OK, at least the perl script helps me, so I could revisit all pages and reenter
my data, but that'll take quite some time. Probably there was some problem
importing an old firefox profile, but then I wonder why the passwords ar still
corectly used   but not displayed in the window.

Please file a new bug, product Firefox, component Password Manager. This bug did
not introduce the Password Manager, it merely added an option in it to show
"Password" in addition to "Site" and "Username".
For those who are intested: Bug was already entered: #249761. Did add a comment
there.
Thanks for the hint. Should have checked the whole Bug DB first.

Gerhard
Please don't remove people from the cc list, thanks.
(In reply to comment #22)
Steffen,

After a period of the password manager and view saved passwords working I have
encountered the blank screen problem again.  This time there is absolutely
nothing in any of the three columns.  No site name, no user name, no saved
password.  It continued through a number of nightly updates and I also tried a
complete reinstallation creating a new profile on one or two occasions.  I have
since just done the upgrade installation to the trunk nightly builds.  As of the
20040812 Trunk build for OS X this condition persists.  (I am still using OS
10.3.4 and so I do not believe that there has been any change of the OS that
would have affected the situation.)  

Can you shed any light on this situation?

Thanks,

Richard

Apologies to all if I am making this comment in the wrong place:

I feel it is a REALLY BAD IDEA (for maintaining security in an
environment with lots of under-informed users--like an office
or school) to have a [Show Passwords] button!

Many users assume that their "managed passwords" are kept in
an encrypted file and never concealed except in their use with
the specified web-site.  Having the button available makes it
way too easy for co-workers to have a quick peek at passwords
of someone who is away from their desk--even for 30 seconds.

This option is in TB too, and should be removed from both.  The
only way this option should be allowed is if it is via an
extension that must be installed by an administrator who chooses
to do so--but even that seems like too much of a hole.

Sorry about the rant, but this would definitely get in the way
of our organization migrating from Eudora/IE to TB/FF.

Douglas Swiggum, University of Wisconsin
> Many users assume that their "managed passwords" are kept in an encrypted file

What do you base this estimate of users on? And why would they assume something
that's: 1) not alluded to or told to them anywhere in Firefox, 2) impossible.

Now that master password UI has been added, the first time you tell Firefox to
store a password for you, it should tell or prompt you to set a master password
if you want your stored data secured. If it doesn't explain this to users, file
a bug.

> quick peek at passwords of someone who is away from their desk--even for 30s

This has always been possible. They step away from their desk, I transfer their
single-signon file to my PC, and unobscure it at my leisure. There's just no way
around this fact. Don't tell a program you run to save sensitive information,
and then either leave the data itself unprotected, or leave your PC unprotected.

(Even a master password without a secured PC or trustable coworkers is of little
use--key loggers, trojans, etc are trivial to install).

The very slightly additional ease of access to this information for undetermined
malicious coworkers is certainly worth the MUCH easier access of a user's
information to the user themselves. The only question is, why does it take a
click on "Show Passwords" and then an annoying confirmation dialog to show
information that's already available.
(In reply to comment #25)
> The very slightly additional ease of access to this information for undetermined
> malicious coworkers is certainly worth the MUCH easier access of a user's
> information to the user themselves. The only question is, why does it take a
> click on "Show Passwords" and then an annoying confirmation dialog to show
> information that's already available.

I wrote the original implementation for this in Mozilla and (thankfully) others
picked it up and got it to where it is today.  The decision to have an annoying
confirmation dialog pop up seemed useful when a user did not yet have a master
password established.  Then, they don't click the 'Show Passwords' button and
BANG! suddenly all of their passwords are in plainview (as in my comment
http://bugzilla.mozilla.org/show_bug.cgi?id=78754#c20 ).
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: