<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 30, 2015 at 1:32 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 30.7.2015 v 09:23 Jan Cholasta napsal(a):<span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Dne 29.7.2015 v 17:23 Gabe Alford napsal(a):<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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>
</blockquote>
<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>
</blockquote>
<br></span>
After reading the ticket I think this should be fixed the other way around: make 0 mean unlimited for both time and size limit and fix the config plugin and LDAPClient to respect that.<span><font color="#888888"><br>
<br></font></span></blockquote><div><br>Thanks for the review. Updated patch attached.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888">
-- <br>
Jan Cholasta<br>
</font></span></blockquote></div><br></div></div>