chiark / gitweb /
logind: don't include session lists in PropertyChanged messages
authorLennart Poettering <lennart@poettering.net>
Wed, 4 May 2016 17:05:18 +0000 (19:05 +0200)
committerSven Eden <yamakuzure@gmx.net>
Fri, 16 Jun 2017 08:12:58 +0000 (10:12 +0200)
commit9648de5798eee557bd0ff8acaf3712ddcd7c22e3
tree51b125ef7ceaf5b46ef749ea873c9321a89d7fcd
parent451a2b7a44f3624b15bbf960e338791e0b0a4210
logind: don't include session lists in PropertyChanged messages

If we have a lot of simultaneous sessions we really shouldn't send the full
list of active sessions with each PropertyChanged message for user and seat
objects, as that can become quite substantial data, we probably shouldn't dump
on the bus on each login and logout.

Note that the global list of sessions doesn't send out changes like this
either, it only supports requesting the session list with ListSessions().

If cients want to get notified about sessions coming and going they should
subscribe to SessionNew and SessionRemoved signals, and clients generally do
that already.

This is kind of an API break, but then again the fact that this was included
was never documented.
src/login/logind-seat-dbus.c
src/login/logind-user-dbus.c