[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ On Wed, Sep 21, 2016 at 03:57:19AM -0700, Pavel Březina wrote: > On 09/21/2016 12:29 PM, Jakub Hrozek wrote: > > On Tue, Sep 20, 2016 at 05:24:45AM -0700, sumit-bose

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ * master: e9a2e7afbd09c23dd8748246e09831ed7b17d7c5 @tequeter thank you very much for the contribution """ See the full comment at

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus pbrezina commented: """ On 09/21/2016 12:29 PM, Jakub Hrozek wrote: > On Tue, Sep 20, 2016 at 05:24:45AM -0700, sumit-bose wrote: >> > With the SIDs we already have a library thay pretty

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ On Tue, Sep 20, 2016 at 05:24:45AM -0700, sumit-bose wrote: > > With the SIDs we already have a library thay pretty much anyone can call > > and retrieve the SID for

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-20 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus sumit-bose commented: """ > With the SIDs we already have a library thay pretty much anyone can call and > retrieve the SID for ID. But not for GUIDs.. CC @sbose-rh for another > opinion..

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ On Tue, Sep 20, 2016 at 03:11:59AM -0700, Pavel Březina wrote: > On 09/19/2016 02:26 PM, tequeter wrote: > > Yes, I considered creating a |group_attributes| instead.

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-20 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus pbrezina commented: """ On 09/19/2016 02:26 PM, tequeter wrote: > Yes, I considered creating a |group_attributes| instead. However it > would have taken more refactoring to avoid duplicating

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-19 Thread tequeter
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus tequeter commented: """ Yes, I considered creating a ```group_attributes``` instead. However it would have taken more refactoring to avoid duplicating the ```user_attributes``` code than I

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-19 Thread tequeter
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus tequeter commented: """ > > I considered using the gid provided by SSSD for that purpose (but it is not > > guaranteed to be consistent on all computers, from sssd-ldap(5)/ID MAPPING), > >

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-16 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus lslebodn commented: """ On (16/09/16 06:12), Jakub Hrozek wrote: >I also wonder if the SIDs are needed. I don't have anything to add them in >principle, except I would prefer to not add

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ I also wonder if the SIDs are needed. I don't have anything to add them in principle, except I would prefer to not add more attributes to the bus unless necessary,