From: Eric Wong <e@80x24.org>
Date: Tue, 24 Apr 2018 19:43:25 +0000 (+0000)
Subject: msgmap: add limit to response for NNTP
X-Git-Tag: v1.1.0-pre1~6
X-Git-Url: http://www.git.stargrave.org/?p=public-inbox.git;a=commitdiff_plain;h=77704e711685d119fde9d1737029e102c76c9a14

msgmap: add limit to response for NNTP

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"
---

diff --git a/lib/PublicInbox/Msgmap.pm b/lib/PublicInbox/Msgmap.pm
index 192e311a..fdc71e46 100644
--- a/lib/PublicInbox/Msgmap.pm
+++ b/lib/PublicInbox/Msgmap.pm
@@ -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