ipsilon

#146 Modify infosssd plugin to fetch attributes from dbus
Merged by dtheisen. Opened by dtheisen.
dtheisen/ipsilon infosssd-dbus-lookup  into  master

Download 146.patch

This removes the need to do mapping of environment variables
from mod_lookup_identity through Ipsilon, and fetches the
attributes from infosssd via DBus. This also removes the
mod_lookup_identiy config Installer within Ipsilon, and
updates other references to it throughout the project.

Signed-off-by: Dan Theisen daniel.theisen@foresee.com

The Installer class still needs to be fixed, but this is a mostly working PoC of the feature.

rebased

rebased

rebased

rebased

Could you add an attr_name = str(attr_name) here?

Or at the str() here and in the other reply[...]'s

rebased

Please change this back to mapping gecos -> fullname.

Please add something like this to get the users fullname (gecos): 

    reply['gecos'] = str(user_obj.Get(
       'org.freedesktop.sssd.infopipe.Users.User',
        'gecos',
       dbus_interface=dbus.PROPERTIES_IFACE))
Edited by puiterwijk

rebased

Could you move this to the on_enable function so that this doesn't crash when run on boxes that don't have DBus (but also don't need infosssd)?

This probably needs rephrasing.

rebased

rebased

rebased

Commit fcaa214a fixes this pull-request

Pull-Request has been merged by daniel.theisen@foresee.com
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.