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 141624 - Use a generic password dialog API
Use a generic password dialog API
Status: RESOLVED INCOMPLETE
Product: gnome-keyring-manager
Classification: Deprecated
Component: general
unspecified
Other Linux
: Normal major
: ---
Assigned To: Keyring manager maintainers
Keyring manager maintainers
Depends on:
Blocks:
 
 
Reported: 2004-05-02 10:55 UTC by Mariano Suárez-Alvarez
Modified: 2007-12-28 21:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Allow at most 1 passwd dialog at a time. (3.86 KB, patch)
2004-05-04 22:51 UTC, Mariano Suárez-Alvarez
committed Details | Review
Patch makes the password dialog generic (8.35 KB, patch)
2004-05-05 15:06 UTC, Patanjali
none Details | Review
updated patch for "patch makes the password dialog generic" (8.35 KB, patch)
2004-05-05 17:01 UTC, Patanjali
none Details | Review

Description Mariano Suárez-Alvarez 2004-05-02 10:55:15 UTC
Currently, the passwd dialog for the unlock keyring action is beign recycled,
but that does not work because the action can be called for different keyrings,
and that causes problems, clearly. 

Think a bit about this before coding, so that it does not get too complicated,
while retaining the ability to recycle passwd dialogs when the user tries to
unlock _the_ _same_ keyring twice.
Comment 1 Mariano Suárez-Alvarez 2004-05-04 22:50:33 UTC
I've commited the following patch which sets things up so that at most one
passwd dialog is shown at a time. It thus closes any existing passwd dialog
before presenting a new one. 

The alternative is more complicated to code, and would end up be confusing.
Comment 2 Mariano Suárez-Alvarez 2004-05-04 22:51:04 UTC
Created attachment 27375 [details] [review]
Allow at most 1 passwd dialog at a time.
Comment 3 Patanjali 2004-05-05 15:06:42 UTC
Created attachment 27395 [details] [review]
Patch makes the password dialog generic

Patch makes it possible to use a password dialog API for various purposes in
gnome-keyring-manager, e.g. Unlock keyring, Open Keyring, Create new keyring,
etc. 

The dialogs shown by gnome-keyring-daemon are not consistent with
gnome-keyring-manager and hence custom password dialogs are required.
Comment 4 Patanjali 2004-05-05 17:01:30 UTC
Created attachment 27399 [details] [review]
updated patch for "patch makes the password dialog generic"

Patch corrects API documentation bug in earlier patch.
Comment 5 Mariano Suárez-Alvarez 2004-06-07 14:21:18 UTC
Hmm, we now have a whole gnome-keyring-manager-password-dialog.c... 

Does this patch still make sense, Patanjali?
Comment 6 Germán Poo-Caamaño 2006-08-03 21:22:44 UTC
gnome-keyring-manager-password-dialog.c was removed after a cleanup and change in UI.  It this patch still valid?
Comment 7 Teppo Turtiainen 2007-12-28 21:16:31 UTC
Closing this bug report as no further information has been provided. Please feel free to reopen this bug if you can provide the information asked for.
Thanks!