After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 356575 - Add a way to change gnome-keyring password
Add a way to change gnome-keyring password
Status: RESOLVED FIXED
Product: seahorse
Classification: Applications
Component: general
unspecified
Other Linux
: Normal enhancement
: 2.20.0
Assigned To: Seahorse Maintainer
Seahorse Maintainer
Depends on:
Blocks:
 
 
Reported: 2006-09-18 16:39 UTC by Michael Monreal
Modified: 2007-06-18 14:31 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
Adds function to GkeyringSource (1.66 KB, patch)
2006-12-29 15:23 UTC, Adam Schreiber
none Details | Review
Handles return message (1.89 KB, patch)
2006-12-29 15:38 UTC, Adam Schreiber
reviewed Details | Review
Adds list keyrings operation (4.76 KB, patch)
2007-01-01 18:37 UTC, Adam Schreiber
none Details | Review
changes the master password on the default ring (119.61 KB, patch)
2007-06-16 17:35 UTC, Adam Schreiber
committed Details | Review

Description Michael Monreal 2006-09-18 16:39:37 UTC
Now that seahorse supports gnome-keyring management, it should also be able to change the password of the keyring (also see bug 38088)
Comment 1 Adam Schreiber 2006-09-18 17:08:57 UTC
Is that bug on a different bugzilla?  It doesn't appear to exist.
Comment 2 Michael Monreal 2006-09-18 17:48:16 UTC
Sorry, that should have been 338088 on bgo. It's about adding password changing support to gnome-keyring manager.
Comment 3 Stef Walter 2006-09-28 15:28:44 UTC
Yay. A lot of people have been asking about this functionality. Will implement.
Comment 4 Adam Schreiber 2006-12-29 15:23:48 UTC
Created attachment 79032 [details] [review]
Adds function to GkeyringSource

This is a pretty simple function that hooks up to the synchronous interface for changing a gnome keyring password.  

ToDo:
 * Determine what to do with the result code as it should probably be more than an int, but nothing else includes the gnome keyring header.
 * Hook up to UI.  The change password/passphrase UI could probably be reused here.
 * Interface with the main application.  I'm not sure how this functionality should be exposed.  Perhaps in the preferences dialog?  There could be a tab that lists the available key rings in a treeview and then one could be selected and a change password button could be made sensitive.  We could also add buttons to create and delete keyrings.  

/end braindump
Comment 5 Adam Schreiber 2006-12-29 15:38:05 UTC
Created attachment 79033 [details] [review]
Handles return message

*Whoops* After digging a little I realized handling the error messages was already implemented.
Comment 6 Stef Walter 2006-12-29 19:21:55 UTC
Thanks for working on this... 

A UI idea I've been kicking around for a while would be to have this main password in bold and expandable, with all the other gnome-keyring passwords underneath it. Perhaps something like:

 + *Nate's Master Password*
   - SMB password 
   - HTTP password 

Then we could have other 'major' passwords like:

 + *Nate's Login Password*

Also this would allow us to support multiple keyrings (such an overloaded term) which gnome-keyring supports.

Obviously this would mean changing significant bits of the plumbing that the GtkTreeStore uses to show the passwords. 

Comment 7 Adam Schreiber 2006-12-29 20:22:48 UTC
I'm not sure that exposing the multiple key rings functionality there is needed.  I'm not sure about the use case for them anyway.  When you're saving a password to  a key ring, do you even get the choice about which to save it to?
Comment 8 Adam Schreiber 2007-01-01 18:35:54 UTC
Additionally, for the Keyring management info to go in the preferences dialog, the keyring key source and accouterments will have to be moved to libseahorse.  I can't work on this any more until we decide what to do.  I have implemented an operation to list the keyrings but won't create any more helper functions for the now.
Comment 9 Adam Schreiber 2007-01-01 18:37:39 UTC
Created attachment 79145 [details] [review]
Adds list keyrings operation
Comment 10 Adam Schreiber 2007-05-03 22:40:15 UTC
Nate, 

Following the discussion on D-D-L regarding the use of multiple key rings, where do you plan on taking gnome-keyring so that I may work on implementing missing gnome-keyring-manager functionality?
Comment 11 Stef Walter 2007-05-05 18:55:20 UTC
It's almost certain that gnome-keyring will continue to support multiple keyrings. 
Jon Nettleton came up with an alternate solution which doesn't take away any current gnome-keyring functionality.

However I might suggest not confusing the user needlessly with multiple keyrings in seahorse, but focusing on the default keyring.
Comment 12 Adam Schreiber 2007-05-05 19:45:19 UTC
Will do.  In UI, should I refer to it as the Default Key Ring's Master Password, or GNOME Keyring's Master password so that I may be consistent with your plans?  I would guess that the second would confuse users less if other key rings aren't being exposed.
Comment 13 Adam Schreiber 2007-06-16 17:35:59 UTC
Created attachment 90084 [details] [review]
changes the master password on the default ring

This patch adds a tab onto the preferences notepad that allows for changing the default keyring's master password.  If there are no objections to phrasing on the tab, I'll commit it.