On Wed, Jan 25, 2012 at 08:23:23PM +0100, Jakub Hrozek wrote:
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).
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).