[SSSD] [PATCH] SUDO Integration - sudo_timed option for responder

Jakub Hrozek jhrozek at redhat.com
Wed Jan 25 19:23:23 UTC 2012


On Fri, Jan 20, 2012 at 01:22:19PM +0100, Pavel Březina wrote:
> Dne 18.1.2012 10:30, Jakub Hrozek napsal(a):
> >On Wed, Jan 18, 2012 at 10:16:14AM +0100, Jakub Hrozek wrote:
> >>On Wed, Jan 18, 2012 at 10:05:23AM +0100, Jan Zelený wrote:
> >>>>https://fedorahosted.org/sssd/ticket/1116
> >>>>
> >>>>Adds option 'sudo_timed' to responder.
> >>>
> >>>Ack. I didn't perform any real-life testing but the patch seems simple enough.
> >>>
> >>>Thanks
> >>>Jan
> >>
> >>Real life testing cannot be performed until the sudo side is ready.
> >>
> >>That's also why I asked Pavel to hold sending out this patch for review
> >>until he can actuall test it.
> >
> >Also, nack. The new option needs to be added into the config API.
> 
> I think this should be done as part of #1144 [1] or #1144 should be
> done as a part of this patch (preferably because this is going to be
> the only responder related option so far).
> 
> [1] https://fedorahosted.org/sssd/ticket/1144

You're right, I didn't realize it was the only option. Then please
include the changes into this patch and we can close both tickets.

Also, this patch needs rebasing on top of the latest iteration of
"cn=defaults" patches.



More information about the sssd-devel mailing list