<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 26, 2016 at 7:33 AM, Martin Basti <span dir="ltr"><<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 26.01.2016 15:17, Petr Spacek wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 26.1.2016 15:06, Martin Basti wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 26.01.2016 15:00, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Jan 26, 2016 at 6:56 AM, Martin Basti <<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a><br>
<mailto:<a href="mailto:mbasti@redhat.com" target="_blank">mbasti@redhat.com</a>>> wrote:<br>
<br>
<br>
<br>
     On 26.01.2016 14:55, Petr Spacek wrote:<br>
<br>
         On 26.1.2016 14:02, Martin Basti wrote:<br>
<br>
             <a href="https://fedorahosted.org/freeipa/ticket/5634" rel="noreferrer" target="_blank">https://fedorahosted.org/freeipa/ticket/5634</a><br>
<br>
             Patch attached.<br>
<br>
         It works for me in API, CLI, and Web UI. The warning is shown<br>
         as expected.<br>
<br>
         Interestingly, Web UI behaves strangely when search limit is<br>
         hit. This needs<br>
         more investigation because it happens even without this patch :-)<br>
<br>
     I found different bug there, webUI passes sizelimit: 0<br>
     (unlimited), but this values is not passed to some searches inside<br>
     BaseldapSearch which raise error, I will file a ticket na provide<br>
     details there<br>
<br>
<br>
Works for me as well. However, it would be nice to have what ipasearchlimit<br>
is limited to in the error message as well. So something like:<br>
</blockquote>
thanks for testing.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
"Search result has been truncated, the current search limit is set to 10.<br>
Please increase the search limit."<br>
</blockquote>
Well this is not so easy to achieve in framework, I prefer not to add number<br>
there, it requires bigger change in framework or an extra ldap search.<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Does this also address <a href="https://fedorahosted.org/freeipa/ticket/4022" rel="noreferrer" target="_blank">https://fedorahosted.org/freeipa/ticket/4022</a>?<br>
</blockquote>
It should.<br>
</blockquote>
Maybe we can use some generic phrase like:<br>
"Search result has been truncated to configured search limit."<br>
and avoid advice like 'increase search limit' which may not be possible to do,<br>
e.g. because user does not have permission to do that etc.<br></blockquote></div></div></blockquote><div><br></div><div>Sounds good.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div>
Updated patch attached.<br>
</blockquote></div><br></div><div class="gmail_extra">Ack from me.<br></div><div class="gmail_extra"><br></div></div>