<div dir="ltr">I have not seen the load yet, we will do some load testing and let you know.  <div><br></div><div>Thanks,</div><div>Michael </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 10, 2013 at 11:48 AM, John Keeping <span dir="ltr"><<a href="mailto:john@keeping.me.uk" target="_blank">john@keeping.me.uk</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">On Thu, Oct 10, 2013 at 10:44:55AM -0400, Michael Hess wrote:<br>
> I am looking into using cgit for Drupal.org's repos, and wondering if we<br>
> could remove code like this:<br>
> <a href="http://git.zx2c4.com/cgit/tree/shared.c#n79" target="_blank">http://git.zx2c4.com/cgit/tree/shared.c#n79</a><br>
><br>
><br>
> We have almost 10,000 repos and are worried about the load from loops, and<br>
> building the index it loops over. All of the repos are under a directory<br>
> (in 2 different sub directories), so I was hoping we could just validate<br>
> the directory path (making sure someone is not trying to do a ../../,etc)<br>
> and allow it?<br>
><br>
> Could that be done?  Please let me know your thoughts.<br>
<br>
</div></div>Have you actually seen this causing excessive load, or is it only a<br>
theoretical issue?<br>
<br>
I expect it would be possible, in the case of scan-path, to load repos<br>
from disk lazily, but that will probably add quite a lot of complexity<br>
and I'm not convinced it's worthwhile.<br>
<br>
That particular loop will only be executed once and I suspect it is<br>
dwarfed by the time spent loading and parsing the config (cached project<br>
list if you're using scan-path).<br>
</blockquote></div><br></div>