Re: [PATCH] Allow specification of the number of search results to return

[Jonathan Corbet <corbet@lwn.net>]

On Tue, 27 Mar 2018 10:44:57 +0000
Eric Wong <e@80x24.org> wrote:

> Jonathan Corbet <corbet@lwn.net> wrote:
> > Add an "l=" parameter to the search query syntax to specify how many
> > results should be returned.  The default remains 200.
> > ---
> > 200 is a lot of results for one page, so allow it to be tweaked.  I've not
> > added anything to the query form to access this (don't need it) but could.  
> 
> There's the "feedmax" config variable, perhaps a config option
> along those lines can be used with search results.

That would solve my problem nicely, but takes the control out of the hands
of the reader.  Is this the approach you would rather see?

> > --- a/lib/PublicInbox/SearchView.pm
> > +++ b/lib/PublicInbox/SearchView.pm
> > @@ -35,7 +35,7 @@ sub sres_top_html {
> >  	my $code = 200;
> >  	# double the limit for expanded views:
> >  	my $opts = {
> > -		limit => $LIM,
> > +		limit => $q->{l},  
> 
> Blindly trusting user input here could cause memory problems on
> the server and should be clamped to a reasonable value.

Yeah...I saw it sort of like the threat from doing a lot of simultaneous
searches.  It's better to limit it, though; I can certainly do that if you
don't prefer the config-option approach.

Thanks,

jon