[Bug 33485] Provide an interface for CMs that provide password storage

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=33485

--- Comment #9 from Will Thompson <will.thompson at collabora.co.uk> 2011-02-22 03:21:02 PST ---
(In reply to comment #8)
> Since I am now attempting to implement the Mission Control parts of this, I am
> reopening the bug to point out issues in the draft.
> 
>  - It is not specified whether Acct.I.EPS should be present on all accounts or
> just ones who's CM implements CM.I.AS. I have assumed the latter case, because
> this allows us to do a tp_proxy_has_interface() to discover whether the
> functionality is available.

This sounds right to me.

>  - CM.I.AS.Accounts has no change notification. As a result, it's not possible
> to implement the Acct.I.EPS.PasswordSaved property, since we can't make async
> method calls from a property getter function, and we can't request and monitor
> the value without any change notification.

The AM could cache the value … and if someone removes the saved password from
the CM behind the AM's back, it will get out of synch, but hey.

>  - Similarly. Acct.I.EPS provides no change notification:
> AccountPropertyChanged cannot be used here:
> 
> """
> The values of one or more properties on this interface (that do not specify
> that this signal does not apply to them) may have changed. This does not cover
> properties of other interfaces, which must provide their own change
> notification if appropriate.
> """

AccountPropertyChanged is a bit upsetting.

You could push the boat out and become the first user, in Telepathy, of
PropertiesChanged
<http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-properties>.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.