[PATCH] nntpd: fix LISTGROUP with range

[PATCH] nntpd: fix LISTGROUP with range

[mephi42]

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")
---
 lib/PublicInbox/NNTP.pm |  2 +-
 t/nntpd.t               | 11 +++++++++--
 2 files changed, 10 insertions(+), 3 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;
 }
 
diff --git a/t/nntpd.t b/t/nntpd.t
index d352c3c1..058bd796 100644
--- a/t/nntpd.t
+++ b/t/nntpd.t
@@ -93,6 +93,13 @@ close $cfgfh or BAIL_OUT;
 	is_deeply([$n->group($group)], [ qw(0 1 1), $group ], 'GROUP works');
 	is_deeply($n->listgroup($group), [1], 'listgroup OK');
 	# TODO: Net::NNTP::listgroup does not support range at the moment
+	my $s = tcp_connect($sock);
+	sysread($s, my $buf, 4096);
+	is($buf, "201 " . hostname . " ready - post via email\r\n",
+		'got greeting');
+	syswrite($s, "LISTGROUP $group 1-1\r\n");
+	$buf = read_til_dot($s);
+	like($buf, qr/\r\n1\r\n/s, 'LISTGROUP with range works');
 
 	{
 		my $expect = [ qw(Subject: From: Date: Message-ID:
@@ -120,8 +127,8 @@ close $cfgfh or BAIL_OUT;
 		'references' => '<reftabsqueezed>',
 	);
 
-	my $s = tcp_connect($sock);
-	sysread($s, my $buf, 4096);
+	$s = tcp_connect($sock);
+	sysread($s, $buf, 4096);
 	is($buf, "201 " . hostname . " ready - post via email\r\n",
 		'got greeting');
 
-- 
2.37.2

Re: [PATCH] nntpd: fix LISTGROUP with range

[Eric Wong]

mephi42 <mephi42@gmail.com> wrote:
> 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")

Thanks, pushed as commit fd06bcb8356a34ee9aef56e83fe96d959dc18123
Apologies for the breakage.

Do you happen to know which NNTP client(s) were affected?

Re: [PATCH] nntpd: fix LISTGROUP with range

[Eric Wong]

Eric Wong <e@80x24.org> wrote:
> Thanks, pushed as commit fd06bcb8356a34ee9aef56e83fe96d959dc18123

Oops, that should be b66f13a1764606b9c4d0a6a8e22ab1a8e30570ad

Accidentally included another incomplete debugging patch for *BSD :x