[Freeipa-devel] [PATCH 559] Fix kadmin for new users

[Martin Basti]

On 06.01.2016 09:49, Petr Vobornik wrote:
> On 01/06/2016 08:42 AM, Martin Kosek wrote:
>> On 01/06/2016 08:37 AM, Martin Babinsky wrote:
>>> On 11/25/2015 03:41 PM, Martin Kosek wrote:
>>>> On 11/25/2015 03:32 PM, Simo Sorce wrote:
>>>>> On Wed, 2015-11-25 at 14:13 +0100, Tomas Babej wrote:
>>>>>>
>>>>>> On 11/25/2015 02:13 PM, Tomas Babej wrote:
>>>>>>>
>>>>>>>
>>>>>>> On 11/25/2015 02:00 PM, Martin Babinsky wrote:
>>>>>>>> On 11/24/2015 11:32 PM, Simo Sorce wrote:
>>>>>>>>> Ticket #937 was reopened a while ago because one corner case, 
>>>>>>>>> new users
>>>>>>>>> that have never been assigned a password cause 
>>>>>>>>> kadmin/kadmin.local to
>>>>>>>>> throw a fit when they try to visualize information about those 
>>>>>>>>> user's
>>>>>>>>> principals.
>>>>>>>>>
>>>>>>>>> This patch fakes up modification information when no 
>>>>>>>>> krbExtraData is
>>>>>>>>> available for the principal so that kadmin is happy.
>>>>>>>>>
>>>>>>>>> Tested and working as designed.
>>>>>>>>>
>>>>>>>>> Simo.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>> ACK
>>>>>>>>
>>>>>>>
>>>>>>> Pushed to master: 0f52eddd1d2781ccc1941c191e9ab6e3ccf6919d
>>>>>>>
>>>>>>
>>>>>> On a related note, should we backport this to later branches?
>>>>>
>>>>> It wouldn't hurt, it should apply straight to any 4.x and probably
>>>>> latest 3.x branches too.
>>>>
>>>> I would not fix anything older than FreeIPA 4.1.x which is in F22, 
>>>> which is the
>>>> oldest supported Fedora (or rather fill be, one month after F23 GA).
>>>>
>>>
>>> https://fedorahosted.org/freeipa/ticket/937 is included in 4.2.4 
>>> milestone with
>>> priority critical. Shouldn't we backport the patch to ipa-4-2 branch?
>>
>> We should... Petr?
>>
>
> Right, that's why it is in 4.2.4.
Pushed to ipa-4-2: fec0f4621bd21c9a0f2f6cda11ade96271a10f5a