Rated 5 out of 5 stars

I completely love it, the only thing that kinda bothers me is that it asks for permission multiple times when entering my info on a text field, one time is cool but 3 times gets a little anoying. Overall an awesome addon for OS X

This review is for a previous version of the add-on (1.0b1). 

There are some cases where I think Firefox is just looking up the password for accounts it doesn't need to (ie. if you have multiple account names saved, it will ask for the keychain password even for accounts that do not match).

The extension ensures that we only ask the keychain for the password if Firefox asks for it. And the keychain shouldn't prompt you unless you ask for the password or your keychain is locked. I assume if you say "Always allow" that you are no longer prompted?

Rated 5 out of 5 stars

Great add on! The lack of keychain integration is the one thing that discouraged me from using firefox but this changes everything. I now use it more than safari!
One thing I would suggest is requiring to type the authentication password when viewing your saved passwords.

This review is for a previous version of the add-on (1.0b1). 

What would probably be better is to simply replace the \"Saved Passwords...\" button with one that opens Keychain Access. But I don\'t actually know if it\'s possible to modify the standard Firefox dialogs from an add-on... guess it might be but I don\'t know how (yet). :)

Rated 5 out of 5 stars

Great add on! The lack of keychain integration is the one thing that discouraged me from using firefox but this changes everything. I now use it more than safari!
One thing I would suggest is requiring to type the authentication password when viewing your saved passwords.

This review is for a previous version of the add-on (1.0b1). 

Rated 5 out of 5 stars

Can this be used with 1Password (working fine on SL/alpha3)? I assume this would be a bad thing?

This review is for a previous version of the add-on (1.0b1). 

Without knowing how 1Password does its integration, I couldn't say for sure. I imagine it must intercept Firefox's normal password mechanism so it probably works fine. But if it's overriding Firefox's password mechanism, there's no point having an extension that changes the backend used by that mechanism. My guess is you would see no change in behaviour with this extension loaded if you are already using 1Password.

Rated 5 out of 5 stars

So far I didn't observe bugs, or anything that would not work in this add-on.
Probably a lot of features are requested :) - so i'm going to ask for an other one: be able to select a keychain where to place the passwords !

I really like this plugin ! thanks for creating it !:)

This review is for a previous version of the add-on (1.0b1). 

Rated 5 out of 5 stars

So far I didn't observe bugs, or anything that would not work in this add-on.
Probably a lot of features are requested :) - so i'm going to ask for an other one: be able to select a keychain where to place the passwords !

I really like this plugin ! thanks for creating it !:)

This review is for a previous version of the add-on (1.0b1). 

Rated 3 out of 5 stars

This version doesn't work together with Weave. I really look forward to see that fixed, though :)

http://code.google.com/p/mozilla-keychain/issues/detail?id=2

This review is for a previous version of the add-on (1.0a7). 

Apparently this is now working with newer versions of Weave (see the referenced issue for details).

Rated 4 out of 5 stars

Exactly what I searched for.
The only problem is, that it is not possible to store the password of the Xmakrs-Addon, which is needed in order to sync the bookmarks automatically.

This review is for a previous version of the add-on (1.0a7). 

Rated 3 out of 5 stars

without actually trying this in my main profile, this is a great solution to a long-standing OS integration issue.

I wonder how hard it would be to query the Pwmgr if no entry is found in Keychain, and if one exists, ask the user if they'd like to add it?

This review is for a previous version of the add-on (1.0a7). 

Rob, probably not that hard. I already have an instance of the standard storage class around to pass a few calls through to. I just need to figure out what the right pattern would be...

Rated 4 out of 5 stars

> Importing or "falling through" to the
> existing storage mechanism

First thought: a more useful approach might be a one-off export from Firefox, to a keychain. FWIW I would not expect Mac OS Keychain 1.x to include export features.

An assumption: users who prefer integration with keychains may also prefer Sync Services, truth database etc. (in lieu of Mozilla Weave and the like).

This review is for a previous version of the add-on (1.0a6). 

Yes, a one-off is what I meant by \"importing\"; I would expect it to prompt you the first time the extension is loaded or something. This is almost certainly the simplest approach and what I would do first. I don\'t actually think it will be overly difficult other than possibly trying to look for partial match duplicates.

Rated 4 out of 5 stars

A minor point: a better name for the add-on might be:

  Password integrator for Keychain Services

References:
http://developer.apple.com/documentation/Security/Conceptual/keychainServConcepts/01introduction/introduction.html
https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsILoginManagerStorage
https://addons.mozilla.org/en-US/firefox/search?q=keychain&cat=all
https://bugzilla.mozilla.org/show_bug.cgi?id=106400#c65

Nice work, thanks!

This review is for a previous version of the add-on (1.0a6).