<div dir="ltr"><div><div><div><div>Hello,<br></div><div><br>Updated patch attached.<br><br></div>- Time limit is -1 for unlimited. I found this <a href="https://www.redhat.com/archives/freeipa-devel/2011-January/msg00330.html">https://www.redhat.com/archives/freeipa-devel/2011-January/msg00330.html</a> in reference to keeping the time limit as -1 for unlimited. <br><span class="im"><span style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255)">Sure enough, testing time limit at 0 did not work for unlimited as well as appeared to have negative effects on IPA. Also, I believe that </span></span><br><a href="http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s" rel="noreferrer" target="_blank">http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s</a></span> specifies unlimited for time limit as -1. (Please correct me if I am wrong.)<br><br></div>- Size limit is 0 for unlimited per Jan's comment including a conversion from -1 to 0 if -1 is entered for unlimited size limit.<br><br><span class="im">>            Actually, 0 means unlimited for size limit, see<br>
>            <<a href="http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s" rel="noreferrer" target="_blank">http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s</a>></span><br><br></div>Thanks,<br><br></div>Gabe<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 4, 2015 at 3:28 AM, Jan Cholasta <span dir="ltr"><<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dne 31.7.2015 v 17:08 Gabe Alford napsal(a):<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Updated patch attached.<br>
<br>
Thanks,<br>
<br>
Gabe<br>
<br>
On Thu, Jul 30, 2015 at 7:15 AM, Gabe Alford <<a href="mailto:redhatrises@gmail.com" target="_blank">redhatrises@gmail.com</a><br></span><span class="">
<mailto:<a href="mailto:redhatrises@gmail.com" target="_blank">redhatrises@gmail.com</a>>> wrote:<br>
<br>
    On Thu, Jul 30, 2015 at 1:32 AM, Jan Cholasta <<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a><br></span><span class="">
    <mailto:<a href="mailto:jcholast@redhat.com" target="_blank">jcholast@redhat.com</a>>> wrote:<br>
<br>
        Dne 30.7.2015 v 09:23 Jan Cholasta napsal(a):<br>
<br>
            Hi,<br>
<br>
            Dne 29.7.2015 v 17:23 Gabe Alford napsal(a):<br>
<br>
                Hello,<br>
<br>
                Fix for <a href="https://fedorahosted.org/freeipa/ticket/4023" rel="noreferrer" target="_blank">https://fedorahosted.org/freeipa/ticket/4023</a><br>
<br>
<br>
            Actually, 0 means unlimited for size limit, see<br>
            <<a href="http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s" rel="noreferrer" target="_blank">http://www.python-ldap.org/doc/html/ldap.html#ldap.LDAPObject.search_ext_s</a>>.<br>
<br>
<br>
        After reading the ticket I think this should be fixed the other<br>
        way around: make 0 mean unlimited for both time and size limit<br>
        and fix the config plugin and LDAPClient to respect that.<br>
<br>
<br>
    Thanks for the review. Updated patch attached.<br>
</span></blockquote>
<br>
We still need to accept -1 in config-mod for backward compatibility - when received, it should be converted to 0.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Jan Cholasta<br>
</font></span></blockquote></div><br></div>