[Freeipa-devel] [PATCH 0096] Add authentication indicators support to Host objects

[Petr Vobornik]

On 06/29/2016 04:40 PM, Stanislav Laznicka wrote:
> On 06/29/2016 04:02 PM, Stanislav Laznicka wrote:
>> On 06/29/2016 03:53 PM, Martin Basti wrote:
>>>
>>>
>>> On 29.06.2016 15:52, Stanislav Laznicka wrote:
>>>> On 06/24/2016 03:14 PM, Martin Basti wrote:
>>>>>
>>>>>
>>>>> On 24.06.2016 15:11, Sumit Bose wrote:
>>>>>> On Tue, Jun 21, 2016 at 02:25:49PM -0400, Nathaniel McCallum wrote:
>>>>>>> https://fedorahosted.org/freeipa/ticket/433
>>>>>> The patch works for me as expected, but the API.txt update is
>>>>>> missing in
>>>>>> the patch.
>>>>>>
>>>>>> bye,
>>>>>> Sumit
>>>>>
>>>>> There are no updated managed permissions for krbprincipalauthind
>>>>> attribute in hosts.py, is this omitted on purpose?
>>>>> Martin^2
>>>>>
>>>> The attached patch adds them should these be required.
>>>>
>>>>
>>>
>>> Then we also needs patch for services.py, because there are missing
>>> ACIs too
>>>
>>> Martin^2
>>
>> It was already included but let me separate it in two patches, then.
>>
>>
> Good catch from Petr Vobornik - the rebuilt ACI.txt should also be
> included.
> 

Attaching new version of Nathnaniel's patch with API.txt and VERSION
updated.

ACK for 0096-2

Pushed to master
* 0855b014b1edcb1632a41e380220abd7bb5e481a Add authentication indicators
support to Host objects.

The  "{Service|Host} {Read|Modify} " permissions looks good to me. ACK
if Nathaniel agrees that it doesn't deserved it's own permission for
modify.
-- 
Petr Vobornik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-npmccallum-0096-2-Add-authentication-indicators-support-to-Host-object.patch
Type: text/x-patch
Size: 5702 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160630/dfd2e06f/attachment.bin>