about summary refs log tree commit homepage
path: root/lib/PublicInbox/NNTP.pm
diff options
context:
space:
mode:
authormephi42 <mephi42@gmail.com>2022-11-28 21:25:21 +0100
committerEric Wong <e@80x24.org>2022-11-28 20:34:27 +0000
commitb66f13a1764606b9c4d0a6a8e22ab1a8e30570ad (patch)
tree0417e4486e4e9b0aea72880be892e83b41fd8102 /lib/PublicInbox/NNTP.pm
parent17b66e5704ce9287e04ae00012c280a3be914d7e (diff)
downloadpublic-inbox-b66f13a1764606b9c4d0a6a8e22ab1a8e30570ad.tar.gz
This reverts 0c62cffc2389 ("nntp: listgroup_range_i: remove useless
`map' op") and adds a test that demonstrates the breakage: the server
returns lines like

    ARRAY(0x556dace73f08)

instead of message numbers.

Fixes: 0c62cffc2389 ("nntp: listgroup_range_i: remove useless `map' op")
Diffstat (limited to 'lib/PublicInbox/NNTP.pm')
-rw-r--r--lib/PublicInbox/NNTP.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/NNTP.pm b/lib/PublicInbox/NNTP.pm
index ceaf05f6..dd33a232 100644
--- a/lib/PublicInbox/NNTP.pm
+++ b/lib/PublicInbox/NNTP.pm
@@ -188,7 +188,7 @@ sub listgroup_range_i {
         my ($self, $beg, $end) = @_;
         my $r = $self->{ibx}->mm(1)->msg_range($beg, $end, 'num');
         scalar(@$r) or return;
-        $self->msg_more(join("\r\n", @$r, ''));
+        $self->msg_more(join('', map { "$_->[0]\r\n" } @$r));
         1;
 }