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.
I grabbed Pavel's WIP patch, and made some fixes (to separate the work,
attached as Patch #2).