gnome-keyring 2.29 breakages

[Sebastien Bacher <seb128 ubuntu com>]

Hi,

The gnome-keyring maintainer has announced that he would not have much
time for GNOME in the next months but the current 2.29 version seems to
be broken in several ways which seem to be show stoppers for 2.30. 

The goal of this email is to raise the issue there and to see if there
is some people who knows the gnome-keyring code who would feel like
trying to help fixing some of the issues or reviewing changes.

Those bugs seem good candidate to be on the 2.30 buglist:

* "g_k_store_password and g_k_find_password doesn't work properly when
keyring is locked"
https://bugzilla.gnome.org/show_bug.cgi?id=611310

There is a small testcase showing that those callbacks don't work as
they should when the keyring is locked.

* "Creates new "default" keyring each time program starts"
https://bugzilla.gnome.org/show_bug.cgi?id=611449

Calling gnome_keyring_create_sync() on an existent keyring used to
return "AlreadyExistsError" and doesn't now, it breaks gajim for example

* "session keyring is stored to disk"
https://bugzilla.gnome.org/show_bug.cgi?id=612977

The bug could be flagged as a security issue, the session keyring is
stored on disk where it's supposed to not be?


The effects of those breakages seem quite noticable in some cases:

* Empathy (or mission-control) seems unable to unlock the keyring with
libgnome-keyring 2.29, which means if an user use autologin and start
empathy creating or connecting to accounts fails. The issue seems
similar to the bug #611310. Empathy works correctly if the keyring is
unlocked by entering a password in gdm or using an another software in
the session before running empathy.

* Several users reported issues about gvfs-sftp or gvfs-smb eating cpu
when trying to use a stored password or asking for the password when
changing sftp directories rather than using the store one

* Bugzilla also has some patches fixing build issues which would be nice
to get reviewed and some other bugs worth investigating


Thanks,  
Sebastien Bacher