]> Sergey Matveev's repositories - public-inbox.git/commitdiff
msgmap: add limit to response for NNTP
authorEric Wong <e@80x24.org>
Tue, 24 Apr 2018 19:43:25 +0000 (19:43 +0000)
committerEric Wong <e@80x24.org>
Tue, 24 Apr 2018 19:43:34 +0000 (19:43 +0000)
All callers in expect to iterate through results.  This
was causing unfairness when fetching large ranges via XHDR
as rtin does :<

Fixes: b8c41362f2a5c8fc "nntp: simplify the long_response API"
lib/PublicInbox/Msgmap.pm

index 192e311aec2b1d459407e0853cfcecb26c2f0448..fdc71e46e214db9526406cf6a6cc76a6686667cf 100644 (file)
@@ -199,7 +199,7 @@ sub msg_range {
        my $attr = { Columns => [] };
        my $mids = $dbh->selectall_arrayref(<<'', $attr, $$beg, $end);
 SELECT num,mid FROM msgmap WHERE num >= ? AND num <= ?
-ORDER BY num ASC
+ORDER BY num ASC LIMIT 1000
 
        $$beg = $mids->[-1]->[0] + 1 if @$mids;
        $mids