On 30.11.2015 20:00, Martin Basti wrote:
> 
> 
> On 27.11.2015 16:06, Stanislav Laznicka wrote:
>> Please, see the modified patch attached.
>>
>> Standa
>>
>> On 11/27/2015 03:48 PM, Martin Basti wrote:
>>>
>>>
>>> On 27.11.2015 15:33, Petr Spacek wrote:
>>>> On 27.11.2015 15:32, Martin Basti wrote:
>>>>>
>>>>> On 25.11.2015 17:18, Stanislav Laznicka wrote:
>>>>>> There were two functions for the same purpose. Removed one.
>>>>>>
>>>>>>
>>>>> Hello,
>>>>>
>>>>> I would like to have "log" param of is_host_resolvable as optional
>>>> Is there an immediate need for the optional param? If not, I would not
>>>> clutter
>>>> the code.
>>>>
>>> So at least I would like to move log param as the last param, in case of
>>> need it can be modified to optional parameter.
>>>
>>> Or log can be default as root_logger, IMO we us only root_logger
>>> everywhere, but this need investigation.
>>>
>>> Martin
>>
> It works, but I would like to have Honza's opinion if ipalib/util.py is the
> right place for the new method and if we really need to use exceptions.
> 
> IMO is_record_resolvable() should return only True/False and then it can be
> located in ipapython module

Hmm, I do not see a reason why it should not throw an exception. IMHO the
exception should contain the original error/result from resolver, so it is
possible to print meaningful error message like "DNS query timed out" instead
of "it does not work for some reason, trust us".

-- 
Petr^2 Spacek

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to