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 710187 - Allow unlocking login keyring while starting daemon from command line
Allow unlocking login keyring while starting daemon from command line
Status: RESOLVED FIXED
Product: gnome-keyring
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME keyring maintainer(s)
GNOME keyring maintainer(s)
Depends on:
Blocks: 704956
 
 
Reported: 2013-10-15 13:34 UTC by Stef Walter
Modified: 2014-03-05 12:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
daemon: Add new --unlock option to prompt for login password (3.32 KB, patch)
2013-10-15 14:23 UTC, Stef Walter
committed Details | Review

Description Stef Walter 2013-10-15 13:34:26 UTC
Currently we read a password on stdin when starting the daemon from PAM. Some other non-PAM callers want to do this as well.

Reusing the --login option for this doesn't work as it has other semantics such as delayed initialization.

Add a new --unlock option to prompt for password.
Comment 1 Stef Walter 2013-10-15 14:23:00 UTC
Created attachment 257349 [details] [review]
daemon: Add new --unlock option to prompt for login password
Comment 2 Stef Walter 2013-10-15 14:24:15 UTC
Request testing/review of attached patch, before I push it.
Comment 3 Matthias Clasen 2014-03-04 23:37:30 UTC
I've tested this in gnome-initial-setup now - it works
Comment 4 Matthias Clasen 2014-03-04 23:39:14 UTC
Review of attachment 257349 [details] [review]:

::: docs/gnome-keyring-daemon.xml
@@ +98,3 @@
 				<command>gnome-keyring-daemon</command>&nbsp;<option>--start</option>
 				invocation.</para>
+			<para>This option implies <option>--unlock</option>. This option

This option ... This option 
sounds a bit awkward. I'd just say 'It' the second time around
Comment 5 Stef Walter 2014-03-05 12:06:12 UTC
Thanks for the testing. Made that documentation change.

Attachment 257349 [details] pushed as ed727e3 - daemon: Add new --unlock option to prompt for login password