From: Eric Wong <e@80x24.org>
To: meta@public-inbox.org
Subject: [PATCH 7/7] nntp: XHDR lookups by Message-ID may cross groups
Date: Tue, 22 Sep 2015 10:09:48 +0000 [thread overview]
Message-ID: <20150922100948.8160-8-e@80x24.org> (raw)
In-Reply-To: <20150922100948.8160-1-e@80x24.org>
This is allowed by RFC 2980 and HDR (to-be-implemented) in
RFC 3977 supports it, too.
---
lib/PublicInbox/NNTP.pm | 55 ++++++++++++++++++++++++++++++++++---------------
t/nntpd.t | 11 +++++++++-
2 files changed, 48 insertions(+), 18 deletions(-)
diff --git a/lib/PublicInbox/NNTP.pm b/lib/PublicInbox/NNTP.pm
index c8ef01e..19e0848 100644
--- a/lib/PublicInbox/NNTP.pm
+++ b/lib/PublicInbox/NNTP.pm
@@ -544,17 +544,18 @@ sub long_response ($$$$) {
sub xhdr_message_id ($$) { # optimize XHDR Message-ID [range] for slrnpull.
my ($self, $range) = @_;
- my $mm = $self->{ng}->mm;
if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
- my $n = $mm->num_for($1);
+ my ($ng, $n) = mid_lookup($self, $1);
return '430 No article with that message-id' unless $n;
more($self, '221 Header follows');
+ my $self_ng = $self->{ng};
more($self, "$range $range");
'.';
} else { # numeric range
$range = $self->{article} unless defined $range;
my $r = get_range($self, $range);
return $r unless ref $r;
+ my $mm = $self->{ng}->mm;
my ($beg, $end) = @$r;
more($self, '221 Header follows');
$self->long_response($beg, $end, sub {
@@ -570,20 +571,36 @@ sub xref ($$) {
"$ng->{domain} $ng->{name}:$n"
}
+sub mid_lookup ($$) {
+ my ($self, $mid) = @_;
+ my $self_ng = $self->{ng};
+ if ($self_ng) {
+ my $n = $self_ng->mm->num_for($mid);
+ return ($self_ng, $n) if defined $n;
+ }
+ foreach my $ng (values %{$self->{nntpd}->{groups}}) {
+ next if $ng eq $self_ng;
+ my $n = $ng->mm->num_for($mid);
+ return ($ng, $n) if defined $n;
+ }
+ (undef, undef);
+}
+
sub xhdr_xref ($$) { # optimize XHDR Xref [range] for rtin
my ($self, $range) = @_;
- my $ng = $self->{ng};
- my $mm = $ng->mm;
if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
- my $n = $mm->num_for($1);
+ my ($ng, $n) = mid_lookup($self, $1);
+ return '430 No article with that message-id' unless $n;
more($self, '221 Header follows');
- more($self, "$range ".xref($ng, $n)) if defined $n;
+ more($self, "$range ".xref($ng, $n));
'.';
} else { # numeric range
$range = $self->{article} unless defined $range;
my $r = get_range($self, $range);
return $r unless ref $r;
+ my $ng = $self->{ng};
+ my $mm = $ng->mm;
my ($beg, $end) = @$r;
more($self, '221 Header follows');
$self->long_response($beg, $end, sub {
@@ -605,7 +622,6 @@ sub header_obj_for {
sub xhdr_searchmsg ($$$) {
my ($self, $sub, $range) = @_;
- my $srch = $self->{ng}->search;
my $emit = ($sub eq 'date') ? sub {
my ($pfx, $m) = @_;
my @t = gmtime($m->header('X-PI-TS'));
@@ -617,12 +633,20 @@ sub xhdr_searchmsg ($$$) {
};
if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
- more($self, '221 Header follows');
- my $m = header_obj_for($srch, $1);
- $emit->($range, $m) if defined $m;
- '.';
+ my ($ng, $n) = mid_lookup($self, $1);
+ return '430 No article with that message-id' unless $n;
+ if (my $srch = $ng->search) {
+ more($self, '221 Header follows');
+ my $m = header_obj_for($srch, $range);
+ $emit->($range, $m) if defined $m;
+ '.';
+ } else {
+ xhdr_slow($self, $sub, $range);
+ }
} else { # numeric range
$range = $self->{article} unless defined $range;
+ my $srch = $self->{ng}->search or
+ return xhdr_slow($self, $sub, $range);
my $mm = $self->{ng}->mm;
my $r = get_range($self, $range);
return $r unless ref $r;
@@ -639,14 +663,12 @@ sub xhdr_searchmsg ($$$) {
sub cmd_xhdr ($$;$) {
my ($self, $header, $range) = @_;
- my $ng = $self->{ng};
- defined $ng or return '412 no news group currently selected';
my $sub = lc $header;
if ($sub eq 'message-id') {
xhdr_message_id($self, $range);
} elsif ($sub eq 'xref') {
xhdr_xref($self, $range);
- } elsif ($sub =~ /\A(subject|references|date)\z/ && $ng->search) {
+ } elsif ($sub =~ /\A(subject|references|date)\z/) {
xhdr_searchmsg($self, $sub, $range);
} else {
xhdr_slow($self, $header, $range);
@@ -660,9 +682,8 @@ sub xhdr_slow ($$$) {
my $r = $self->art_lookup($range, 2);
return $r unless ref $r;
more($self, '221 Header follows');
- if (defined($r = xhdr($r, $header))) {
- more($self, "$range $r");
- }
+ $r = xhdr($r, $header);
+ more($self, "$range $r") if defined $r;
'.';
} else { # numeric range
$range = $self->{article} unless defined $range;
diff --git a/t/nntpd.t b/t/nntpd.t
index 090e150..28d8491 100644
--- a/t/nntpd.t
+++ b/t/nntpd.t
@@ -90,7 +90,8 @@ EOF
$! = 0;
fcntl($sock, F_SETFD, $fl |= FD_CLOEXEC);
ok(! $!, 'no error from fcntl(F_SETFD)');
- my $n = Net::NNTP->new($sock->sockhost . ':' . $sock->sockport);
+ my $host_port = $sock->sockhost . ':' . $sock->sockport;
+ my $n = Net::NNTP->new($host_port);
my $list = $n->list;
is_deeply($list, { $group => [ qw(1 1 n) ] }, 'LIST works');
is_deeply([$n->group($group)], [ qw(0 1 1), $group ], 'GROUP works');
@@ -112,6 +113,14 @@ EOF
"$k by article number works");
}
+ {
+ my $nogroup = Net::NNTP->new($host_port);
+ while (my ($k, $v) = each %xhdr) {
+ is_deeply($nogroup->xhdr("$k $mid"), { $mid => $v },
+ "$k by message-id works without group");
+ }
+ }
+
# TODO: upgrades and such
ok(kill('TERM', $pid), 'killed nntpd');
--
EW
prev parent reply other threads:[~2015-09-22 10:09 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-22 10:09 [PATCH 0/7] nntp: another round of updates Eric Wong
2015-09-22 10:09 ` [PATCH 1/7] t/nntpd.t: fix fcntl test to ensure no failures Eric Wong
2015-09-22 10:09 ` [PATCH 2/7] nntp: support YYYYMMDD dates for commands Eric Wong
2015-09-22 10:09 ` [PATCH 3/7] nntp: ensure body lines end with CRLF Eric Wong
2015-09-22 10:09 ` [PATCH 4/7] nntp: avoid setting Bytes and Lines headers Eric Wong
2015-09-22 10:09 ` [PATCH 5/7] nntp: implement OVER from RFC 3977 Eric Wong
2015-09-22 10:09 ` [PATCH 6/7] nntp: XHDR fixes for Message-ID lookups Eric Wong
2015-09-22 10:09 ` Eric Wong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://public-inbox.org/README
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150922100948.8160-8-e@80x24.org \
--to=e@80x24.org \
--cc=meta@public-inbox.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).