]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/NNTP.pm
nntp: simplify the long_response API
[public-inbox.git] / lib / PublicInbox / NNTP.pm
index 7a735735ca5c6e60f44b70b8c20e43945ab4b7bc..e51793505cf515851f7031dc65c982918076f89c 100644 (file)
-# Copyright (C) 2015 all contributors <meta@public-inbox.org>
-# License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
+# Copyright (C) 2015-2018 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+#
+# Each instance of this represents a NNTP client socket
 package PublicInbox::NNTP;
 use strict;
 use warnings;
 use base qw(Danga::Socket);
-use fields qw(nntpd article ng long_res);
+use fields qw(nntpd article rbuf ng long_res);
+use PublicInbox::Search;
 use PublicInbox::Msgmap;
-use PublicInbox::GitCatFile;
-use PublicInbox::MID qw(mid2path);
+use PublicInbox::MID qw(mid_escape);
+use PublicInbox::Git;
+require PublicInbox::EvCleanup;
 use Email::Simple;
-use Data::Dumper qw(Dumper);
 use POSIX qw(strftime);
-use Time::HiRes qw(gettimeofday tv_interval ualarm);
+use Time::HiRes qw(clock_gettime CLOCK_MONOTONIC);
+use Digest::SHA qw(sha1_hex);
+use Time::Local qw(timegm timelocal);
 use constant {
        r501 => '501 command syntax error',
+       r221 => '221 Header follows',
+       r224 => '224 Overview information follows (multi-line)',
+       r225 => '225 Headers follow (multi-line)',
+       r430 => '430 No article with that message-id',
        long_response_limit => 0xffffffff,
 };
 
-my @OVERVIEW = qw(Subject From Date Message-ID References Bytes Lines);
-my %OVERVIEW = map { $_ => 1 } @OVERVIEW;
+sub now () { clock_gettime(CLOCK_MONOTONIC) };
+
+my @OVERVIEW = qw(Subject From Date Message-ID References);
+my $OVERVIEW_FMT = join(":\r\n", @OVERVIEW, qw(Bytes Lines)) . ":\r\n";
+my $LIST_HEADERS = join("\r\n", @OVERVIEW,
+                       qw(:bytes :lines Xref To Cc)) . "\r\n";
 
 # disable commands with easy DoS potential:
-# LISTGROUP could get pretty bad, too...
 my %DISABLED; # = map { $_ => 1 } qw(xover list_overview_fmt newnews xhdr);
 
+my $EXPMAP; # fd -> [ idle_time, $self ]
+my $expt;
+our $EXPTIME = 180; # 3 minutes
+my $nextt;
+
+my $nextq = [];
+sub next_tick () {
+       $nextt = undef;
+       my $q = $nextq;
+       $nextq = [];
+       foreach my $nntp (@$q) {
+               # for request && response protocols, always finish writing
+               # before finishing reading:
+               if (my $long_cb = $nntp->{long_res}) {
+                       $nntp->write($long_cb);
+               } elsif (&Danga::Socket::POLLIN & $nntp->{event_watch}) {
+                       event_read($nntp);
+               }
+       }
+}
+
+sub update_idle_time ($) {
+       my ($self) = @_;
+       my $fd = $self->{fd};
+       defined $fd and $EXPMAP->{$fd} = [ now(), $self ];
+}
+
+sub expire_old () {
+       my $now = now();
+       my $exp = $EXPTIME;
+       my $old = $now - $exp;
+       my $nr = 0;
+       my %new;
+       while (my ($fd, $v) = each %$EXPMAP) {
+               my ($idle_time, $nntp) = @$v;
+               if ($idle_time < $old) {
+                       $nntp->close; # idempotent
+               } else {
+                       ++$nr;
+                       $new{$fd} = $v;
+               }
+       }
+       $EXPMAP = \%new;
+       if ($nr) {
+               $expt = PublicInbox::EvCleanup::later(*expire_old);
+       } else {
+               $expt = undef;
+               # noop to kick outselves out of the loop ASAP so descriptors
+               # really get closed
+               PublicInbox::EvCleanup::asap(sub {});
+       }
+}
+
 sub new ($$$) {
        my ($class, $sock, $nntpd) = @_;
        my $self = fields::new($class);
        $self->SUPER::new($sock);
        $self->{nntpd} = $nntpd;
        res($self, '201 server ready - post via email');
+       $self->{rbuf} = '';
        $self->watch_read(1);
+       update_idle_time($self);
+       $expt ||= PublicInbox::EvCleanup::later(*expire_old);
        $self;
 }
 
@@ -58,8 +126,9 @@ sub process_line ($$) {
        my $res = eval { $req->($self, @args) };
        my $err = $@;
        if ($err && !$self->{closed}) {
-               chomp($l = Dumper(\$l));
-               warning('error from: ', $l, ' ', $err);
+               local $/ = "\n";
+               chomp($l);
+               err($self, 'error from: %s (%s)', $l, $err);
                $res = '503 program fault - command not performed';
        }
        return 0 unless defined $res;
@@ -70,7 +139,7 @@ sub cmd_mode ($$) {
        my ($self, $arg) = @_;
        $arg = uc $arg;
        return r501 unless $arg eq 'READER';
-       '200 reader status acknowledged';
+       '201 Posting prohibited';
 }
 
 sub cmd_slave ($) { '202 slave status noted' }
@@ -82,16 +151,21 @@ sub cmd_xgtitle ($;$) {
        '.'
 }
 
-sub list_overview_fmt ($$) {
+sub list_overview_fmt ($) {
+       my ($self) = @_;
+       do_more($self, $OVERVIEW_FMT);
+}
+
+sub list_headers ($;$) {
        my ($self) = @_;
-       more($self, $_ . ':') foreach @OVERVIEW;
+       do_more($self, $LIST_HEADERS);
 }
 
 sub list_active ($;$) {
        my ($self, $wildmat) = @_;
        wildmat2re($wildmat);
-       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
-               $ng->{name} =~ $wildmat or next;
+       foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
+               $ng->{newsgroup} =~ $wildmat or next;
                group_line($self, $ng);
        }
 }
@@ -99,31 +173,31 @@ sub list_active ($;$) {
 sub list_active_times ($;$) {
        my ($self, $wildmat) = @_;
        wildmat2re($wildmat);
-       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
-               $ng->{name} =~ $wildmat or next;
+       foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
+               $ng->{newsgroup} =~ $wildmat or next;
                my $c = eval { $ng->mm->created_at } || time;
-               more($self, "$ng->{name} $c $ng->{address}");
+               more($self, "$ng->{newsgroup} $c $ng->{-primary_address}");
        }
 }
 
 sub list_newsgroups ($;$) {
        my ($self, $wildmat) = @_;
        wildmat2re($wildmat);
-       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
-               $ng->{name} =~ $wildmat or next;
+       foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
+               $ng->{newsgroup} =~ $wildmat or next;
                my $d = $ng->description;
-               more($self, "$ng->{name} $d");
+               more($self, "$ng->{newsgroup} $d");
        }
 }
 
-# LIST SUBSCRIPTIONS not supported
+# LIST SUBSCRIPTIONS, DISTRIB.PATS are not supported
 sub cmd_list ($;$$) {
        my ($self, @args) = @_;
        if (scalar @args) {
                my $arg = shift @args;
                $arg =~ tr/A-Z./a-z_/;
                $arg = "list_$arg";
-               return '503 function not performed' if $DISABLED{$arg};
+               return r501 if $DISABLED{$arg};
 
                $arg = eval {
                        no strict 'refs';
@@ -134,7 +208,7 @@ sub cmd_list ($;$$) {
                $arg->($self, @args);
        } else {
                more($self, '215 list of newsgroups follows');
-               foreach my $ng (values %{$self->{nntpd}->{groups}}) {
+               foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
                        group_line($self, $ng);
                }
        }
@@ -150,44 +224,44 @@ sub cmd_listgroup ($;$) {
        }
 
        $self->{ng} or return '412 no newsgroup selected';
-       $self->long_response(0, long_response_limit, sub {
-               my ($i) = @_;
-               my $nr = $self->{ng}->mm->id_batch($$i, sub {
-                       my ($ary) = @_;
-                       more($self, join("\r\n", @$ary));
-               });
-
-               # -1 to adjust for implicit increment in long_response
-               $$i = $nr ? $$i + $nr - 1 : long_response_limit;
+       my $n = 0;
+       long_response($self, sub {
+               my $ary = $self->{ng}->mm->ids_after(\$n);
+               scalar @$ary or return;
+               more($self, join("\r\n", @$ary));
+               1;
        });
 }
 
 sub parse_time ($$;$) {
        my ($date, $time, $gmt) = @_;
-       use Time::Local qw();
-       my ($YY, $MM, $DD) = unpack('A2A2A2', $date);
        my ($hh, $mm, $ss) = unpack('A2A2A2', $time);
        if (defined $gmt) {
-               $gmt =~ /\A(?:UTC|GMT)\z/i or die "GM invalid: $gmt\n";
+               $gmt =~ /\A(?:UTC|GMT)\z/i or die "GM invalid: $gmt";
                $gmt = 1;
        }
        my @now = $gmt ? gmtime : localtime;
-       if ($YY > strftime('%y', @now)) {
+       my ($YYYY, $MM, $DD);
+       if (length($date) == 8) { # RFC 3977 allows YYYYMMDD
+               ($YYYY, $MM, $DD) = unpack('A4A2A2', $date);
+       } else { # legacy clients send YYMMDD
+               ($YYYY, $MM, $DD) = unpack('A2A2A2', $date);
                my $cur_year = $now[5] + 1900;
-               $YY += int($cur_year / 1000) * 1000 - 100;
+               if ($YYYY > $cur_year) {
+                       $YYYY += int($cur_year / 1000) * 1000 - 100;
+               }
        }
-
        if ($gmt) {
-               Time::Local::timegm($ss, $mm, $hh, $DD, $MM - 1, $YY);
+               timegm($ss, $mm, $hh, $DD, $MM - 1, $YYYY);
        } else {
-               Time::Local::timelocal($ss, $mm, $hh, $DD, $MM - 1, $YY);
+               timelocal($ss, $mm, $hh, $DD, $MM - 1, $YYYY);
        }
 }
 
 sub group_line ($$) {
        my ($self, $ng) = @_;
        my ($min, $max) = $ng->mm->minmax;
-       more($self, "$ng->{name} $max $min n") if defined $min && defined $max;
+       more($self, "$ng->{newsgroup} $max $min n") if defined $min && defined $max;
 }
 
 sub cmd_newgroups ($$$;$$) {
@@ -197,7 +271,7 @@ sub cmd_newgroups ($$$;$$) {
 
        # TODO dists
        more($self, '231 list of new newsgroups follows');
-       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
+       foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
                my $c = eval { $ng->mm->created_at } || 0;
                next unless $c > $ts;
                group_line($self, $ng);
@@ -209,7 +283,6 @@ sub wildmat2re (;$) {
        return $_[0] = qr/.*/ if (!defined $_[0] || $_[0] eq '*');
        my %keep;
        my $salt = rand;
-       use Digest::SHA qw(sha1_hex);
        my $tmp = $_[0];
 
        $tmp =~ s#(?<!\\)\[(.+)(?<!\\)\]#
@@ -246,32 +319,30 @@ sub cmd_newnews ($$$$;$$) {
        ngpat2re($keep);
        ngpat2re($skip);
        my @srch;
-       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
-               $ng->{name} =~ $keep or next;
-               $ng->{name} =~ $skip and next;
+       foreach my $ng (@{$self->{nntpd}->{grouplist}}) {
+               $ng->{newsgroup} =~ $keep or next;
+               $ng->{newsgroup} =~ $skip and next;
                my $srch = $ng->search or next;
                push @srch, $srch;
        };
        return '.' unless @srch;
 
-       $ts .= '..';
-       my $opts = { asc => 1, limit => 1000, offset => 0 };
-       $self->long_response(0, long_response_limit, sub {
-               my ($i) = @_;
+       my $prev = 0;
+       long_response($self, sub {
                my $srch = $srch[0];
-               my $res = $srch->query($ts, $opts);
-               my $msgs = $res->{msgs};
-               if (my $nr = scalar @$msgs) {
+               my $msgs = $srch->query_ts($ts, $prev);
+               if (scalar @$msgs) {
                        more($self, '<' .
                                join(">\r\n<", map { $_->mid } @$msgs ).
                                '>');
-                       $opts->{offset} += $nr;
+                       $prev = $msgs->[-1]->{num};
                } else {
                        shift @srch;
                        if (@srch) { # continue onto next newsgroup
-                               $opts->{offset} = 0;
+                               $prev = 0;
+                               return 1;
                        } else { # break out of the long response.
-                               $$i = long_response_limit;
+                               return;
                        }
                }
        });
@@ -316,7 +387,8 @@ sub cmd_last ($) { article_adj($_[0], -1) }
 sub cmd_post ($) {
        my ($self) = @_;
        my $ng = $self->{ng};
-       $ng ? "440 mailto:$ng->{address} to post" : '440 posting not allowed'
+       $ng ? "440 mailto:$ng->{-primary_address} to post"
+               : '440 posting not allowed'
 }
 
 sub cmd_quit ($) {
@@ -326,9 +398,32 @@ sub cmd_quit ($) {
        undef;
 }
 
+sub header_append ($$$) {
+       my ($hdr, $k, $v) = @_;
+       my @v = $hdr->header($k);
+       foreach (@v) {
+               return if $v eq $_;
+       }
+       $hdr->header_set($k, @v, $v);
+}
+
+sub set_nntp_headers {
+       my ($hdr, $ng, $n, $mid) = @_;
+
+       # clobber some
+       $hdr->header_set('Newsgroups', $ng->{newsgroup});
+       $hdr->header_set('Xref', xref($ng, $n));
+       header_append($hdr, 'List-Post', "<mailto:$ng->{-primary_address}>");
+       if (my $url = $ng->base_url) {
+               $mid = mid_escape($mid);
+               header_append($hdr, 'Archived-At', "<$url$mid/>");
+               header_append($hdr, 'List-Archive', "<$url>");
+       }
+}
+
 sub art_lookup ($$$) {
        my ($self, $art, $set_headers) = @_;
-       my $ng = $self->{ng} or return '412 no newsgroup has been selected';
+       my $ng = $self->{ng};
        my ($n, $mid);
        my $err;
        if (defined $art) {
@@ -337,10 +432,18 @@ sub art_lookup ($$$) {
                        $n = int($art);
                        goto find_mid;
                } elsif ($art =~ /\A<([^>]+)>\z/) {
-                       $err = '430 no such article found';
                        $mid = $1;
-                       $n = $ng->mm->num_for($mid);
-                       defined $mid or return $err;
+                       $err = r430;
+                       $n = $ng->mm->num_for($mid) if $ng;
+                       goto found if defined $n;
+                       foreach my $g (values %{$self->{nntpd}->{groups}}) {
+                               $n = $g->mm->num_for($mid);
+                               if (defined $n) {
+                                       $ng = $g;
+                                       goto found;
+                               }
+                       }
+                       return $err;
                } else {
                        return r501;
                }
@@ -349,25 +452,21 @@ sub art_lookup ($$$) {
                $n = $self->{article};
                defined $n or return $err;
 find_mid:
+               $ng or return '412 no newsgroup has been selected';
                $mid = $ng->mm->mid_for($n);
                defined $mid or return $err;
        }
-
-       my $o = 'HEAD:' . mid2path($mid);
-       my $s = eval { Email::Simple->new($ng->gcf->cat_file($o)) };
-       return $err unless $s;
+found:
+       my $smsg = $ng->search->{over_ro}->get_art($n) or return $err;
+       my $msg = $ng->msg_by_smsg($smsg) or return $err;
+       my $s = Email::Simple->new($msg);
        if ($set_headers) {
-               $s->header_set('Newsgroups', $ng->{name});
-               $s->header_set('Lines', $s->body =~ tr!\n!\n!);
-               $s->header_set('Xref', "$ng->{domain} $ng->{name}:$n");
+               set_nntp_headers($s->header_obj, $ng, $n, $mid);
 
                # must be last
-               if ($set_headers == 2) {
-                       $s->header_set('Bytes', bytes::length($s->as_string));
-                       $s->body_set('');
-               }
+               $s->body_set('') if ($set_headers == 2);
        }
-       [ $n, $mid, $s ];
+       [ $n, $mid, $s, $smsg->bytes, $smsg->lines, $ng ];
 }
 
 sub simple_body_write ($$) {
@@ -375,52 +474,62 @@ sub simple_body_write ($$) {
        my $body = $s->body;
        $s->body_set('');
        $body =~ s/^\./../smg;
+       $body =~ s/(?<!\r)\n/\r\n/sg;
        do_more($self, $body);
+       do_more($self, "\r\n") unless $body =~ /\r\n\z/s;
        '.'
 }
 
-sub header_str ($) {
-       my ($s) = @_;
-       my $h = $s->header_obj;
-       $h->header_set('Bytes');
-       $h->as_string
+sub set_art {
+       my ($self, $art) = @_;
+       $self->{article} = $art if defined $art && $art =~ /\A\d+\z/;
+}
+
+sub _header ($) {
+       my $hdr = $_[0]->header_obj->as_string;
+       utf8::encode($hdr);
+       $hdr
 }
 
-sub cmd_article ($$) {
+sub cmd_article ($;$) {
        my ($self, $art) = @_;
-       my $r = $self->art_lookup($art, 1);
+       my $r = art_lookup($self, $art, 1);
        return $r unless ref $r;
        my ($n, $mid, $s) = @$r;
+       set_art($self, $art);
        more($self, "220 $n <$mid> article retrieved - head and body follow");
-       do_more($self, header_str($s));
+       do_more($self, _header($s));
        do_more($self, "\r\n");
        simple_body_write($self, $s);
 }
 
-sub cmd_head ($$) {
+sub cmd_head ($;$) {
        my ($self, $art) = @_;
-       my $r = $self->art_lookup($art, 2);
+       my $r = art_lookup($self, $art, 2);
        return $r unless ref $r;
        my ($n, $mid, $s) = @$r;
+       set_art($self, $art);
        more($self, "221 $n <$mid> article retrieved - head follows");
-       do_more($self, header_str($s));
+       do_more($self, _header($s));
        '.'
 }
 
-sub cmd_body ($$) {
+sub cmd_body ($;$) {
        my ($self, $art) = @_;
-       my $r = $self->art_lookup($art, 0);
+       my $r = art_lookup($self, $art, 0);
        return $r unless ref $r;
        my ($n, $mid, $s) = @$r;
+       set_art($self, $art);
        more($self, "222 $n <$mid> article retrieved - body follows");
        simple_body_write($self, $s);
 }
 
-sub cmd_stat ($$) {
+sub cmd_stat ($;$) {
        my ($self, $art) = @_;
-       my $r = $self->art_lookup($art, 0);
+       my $r = art_lookup($self, $art, 0);
        return $r unless ref $r;
        my ($n, $mid, undef) = @$r;
+       set_art($self, $art);
        "223 $n <$mid> article retrieved - request text separately";
 }
 
@@ -434,7 +543,7 @@ sub cmd_help ($) {
        '.'
 }
 
-sub get_range ($;$) {
+sub get_range ($$) {
        my ($self, $range) = @_;
        my $ng = $self->{ng} or return '412 no news group has been selected';
        defined $range or return '420 No article(s) selected';
@@ -455,104 +564,308 @@ sub get_range ($;$) {
        [ $beg, $end ];
 }
 
-sub xhdr ($$) {
-       my ($r, $header) = @_;
-       $r = $r->[2]->header_obj->header($header);
-       defined $r or return;
-       $r =~ s/[\r\n\t]+/ /sg;
-       $r;
-}
-
-sub long_response ($$$$) {
-       my ($self, $beg, $end, $cb) = @_;
+sub long_response ($$) {
+       my ($self, $cb) = @_;
        die "BUG: nested long response" if $self->{long_res};
 
+       my $fd = $self->{fd};
+       defined $fd or return;
        # make sure we disable reading during a long response,
        # clients should not be sending us stuff and making us do more
        # work while we are stream a response to them
        $self->watch_read(0);
+       my $t0 = now();
        $self->{long_res} = sub {
-               # limit our own running time for fairness with other
-               # clients and to avoid buffering too much:
-               my $yield;
-               local $SIG{ALRM} = sub { $yield = 1 };
-               ualarm(100000);
-
-               my $err;
-               do {
-                       eval { $cb->(\$beg) };
-               } until (($err = $@) || $self->{closed} || $yield ||
-                        $self->{write_buf_size} || ++$beg > $end);
-               ualarm(0);
-
-               if ($err || $self->{closed}) {
+               my $more = eval { $cb->() };
+               if ($@ || $self->{closed}) {
                        $self->{long_res} = undef;
-                       warning("$err during long response") if $err;
-                       $self->watch_read(1) unless $self->{closed};
-               } elsif ($yield || $self->{write_buf_size}) {
+
+                       if ($@) {
+                               err($self,
+                                   "%s during long response[$fd] - %0.6f",
+                                   $@, now() - $t0);
+                       }
+                       if ($self->{closed}) {
+                               out($self, " deferred[$fd] aborted - %0.6f",
+                                          now() - $t0);
+                       } else {
+                               update_idle_time($self);
+                               $self->watch_read(1);
+                       }
+               } elsif ($more) { # $self->{write_buf_size}:
                        # no recursion, schedule another call ASAP
                        # but only after all pending writes are done
-                       Danga::Socket->AddTimer(0, sub {
-                               $self->write($self->{long_res});
-                       });
+                       update_idle_time($self);
+
+                       push @$nextq, $self;
+                       $nextt ||= PublicInbox::EvCleanup::asap(*next_tick);
                } else { # all done!
                        $self->{long_res} = undef;
                        $self->watch_read(1);
                        res($self, '.');
+                       out($self, " deferred[$fd] done - %0.6f", now() - $t0);
                }
        };
        $self->{long_res}->(); # kick off!
        undef;
 }
 
-sub cmd_xhdr ($$;$) {
-       my ($self, $header, $range) = @_;
-       defined $self->{ng} or return '412 no news group currently selected';
-       unless (defined $range) {
-               defined($range = $self->{article}) or
-                       return '420 no current article has been selected';
+sub hdr_message_id ($$$) { # optimize XHDR Message-ID [range] for slrnpull.
+       my ($self, $xhdr, $range) = @_;
+
+       if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
+               my ($ng, $n) = mid_lookup($self, $1);
+               return r430 unless $n;
+               hdr_mid_response($self, $xhdr, $ng, $n, $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, $xhdr ? r221 : r225);
+               long_response($self, sub {
+                       my $r = $mm->msg_range(\$beg, $end);
+                       @$r or return;
+                       more($self, join("\r\n", map {
+                               "$_->[0] <$_->[1]>"
+                       } @$r));
+                       1;
+               });
+       }
+}
+
+sub xref ($$) {
+       my ($ng, $n) = @_;
+       "$ng->{domain} $ng->{newsgroup}:$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 defined $self_ng && $ng eq $self_ng;
+               my $n = $ng->mm->num_for($mid);
+               return ($ng, $n) if defined $n;
        }
-       if ($range =~ /\A<(.+)>\z/) { # Message-ID
-               my $r = $self->art_lookup($range, 2);
+       (undef, undef);
+}
+
+sub hdr_xref ($$$) { # optimize XHDR Xref [range] for rtin
+       my ($self, $xhdr, $range) = @_;
+
+       if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
+               my ($ng, $n) = mid_lookup($self, $1);
+               return r430 unless $n;
+               hdr_mid_response($self, $xhdr, $ng, $n, $range, xref($ng, $n));
+       } else { # numeric range
+               $range = $self->{article} unless defined $range;
+               my $r = get_range($self, $range);
                return $r unless ref $r;
-               more($self, '221 Header follows');
-               if (defined($r = xhdr($r, $header))) {
-                       more($self, "<$range> $r");
-               }
-               '.';
+               my $ng = $self->{ng};
+               my $mm = $ng->mm;
+               my ($beg, $end) = @$r;
+               more($self, $xhdr ? r221 : r225);
+               long_response($self, sub {
+                       my $r = $mm->msg_range(\$beg, $end);
+                       @$r or return;
+                       more($self, join("\r\n", map {
+                               # TODO: use $_->[1] (mid) to fill
+                               # Xref: from other inboxes
+                               my $num = $_->[0];
+                               "$num ".xref($ng, $num);
+                       } @$r));
+                       1;
+               });
+       }
+}
+
+sub search_header_for {
+       my ($srch, $num, $field) = @_;
+       my $smsg = $srch->{over_ro}->get_art($num) or return;
+       return PublicInbox::SearchMsg::date($smsg) if $field eq 'date';
+       $smsg->{$field};
+}
+
+sub hdr_searchmsg ($$$$) {
+       my ($self, $xhdr, $field, $range) = @_;
+       if (defined $range && $range =~ /\A<(.+)>\z/) { # Message-ID
+               my ($ng, $n) = mid_lookup($self, $1);
+               return r430 unless defined $n;
+               my $v = search_header_for($ng->search, $n, $field);
+               hdr_mid_response($self, $xhdr, $ng, $n, $range, $v);
        } else { # numeric range
+               $range = $self->{article} unless defined $range;
+               my $srch = $self->{ng}->search;
+               my $mm = $self->{ng}->mm;
                my $r = get_range($self, $range);
                return $r unless ref $r;
                my ($beg, $end) = @$r;
-               more($self, '221 Header follows');
-               $self->long_response($beg, $end, sub {
-                       my ($i) = @_;
-                       $r = $self->art_lookup($$i, 2);
-                       return unless ref $r;
-                       defined($r = xhdr($r, $header)) or return;
-                       more($self, "$$i $r");
+               more($self, $xhdr ? r221 : r225);
+               my $cur = $beg;
+               long_response($self, sub {
+                       my $msgs = $srch->query_xover($cur, $end);
+                       my $nr = scalar @$msgs or return;
+                       my $tmp = '';
+                       foreach my $s (@$msgs) {
+                               $tmp .= $s->num . ' ' . $s->$field . "\r\n";
+                       }
+                       utf8::encode($tmp);
+                       do_more($self, $tmp);
+                       $cur = $msgs->[-1]->{num} + 1;
                });
        }
 }
 
+sub do_hdr ($$$;$) {
+       my ($self, $xhdr, $header, $range) = @_;
+       my $sub = lc $header;
+       if ($sub eq 'message-id') {
+               hdr_message_id($self, $xhdr, $range);
+       } elsif ($sub eq 'xref') {
+               hdr_xref($self, $xhdr, $range);
+       } elsif ($sub =~ /\A(?:subject|references|date|from|to|cc|
+                               bytes|lines)\z/x) {
+               hdr_searchmsg($self, $xhdr, $sub, $range);
+       } elsif ($sub =~ /\A:(bytes|lines)\z/) {
+               hdr_searchmsg($self, $xhdr, $1, $range);
+       } else {
+               $xhdr ? (r221 . "\r\n.") : "503 HDR not permitted on $header";
+       }
+}
+
+# RFC 3977
+sub cmd_hdr ($$;$) {
+       my ($self, $header, $range) = @_;
+       do_hdr($self, 0, $header, $range);
+}
+
+# RFC 2980
+sub cmd_xhdr ($$;$) {
+       my ($self, $header, $range) = @_;
+       do_hdr($self, 1, $header, $range);
+}
+
+sub hdr_mid_prefix ($$$$$) {
+       my ($self, $xhdr, $ng, $n, $mid) = @_;
+       return $mid if $xhdr;
+
+       # HDR for RFC 3977 users
+       if (my $self_ng = $self->{ng}) {
+               ($self_ng eq $ng) ? $n : '0';
+       } else {
+               '0';
+       }
+}
+
+sub hdr_mid_response ($$$$$$) {
+       my ($self, $xhdr, $ng, $n, $mid, $v) = @_; # r: art_lookup result
+       my $res = '';
+       if ($xhdr) {
+               $res .= r221 . "\r\n";
+               $res .= "$mid $v\r\n";
+       } else {
+               $res .= r225 . "\r\n";
+               my $pfx = hdr_mid_prefix($self, $xhdr, $ng, $n, $mid);
+               $res .= "$pfx $v\r\n";
+       }
+       res($self, $res .= '.');
+       undef;
+}
+
+sub cmd_xrover ($;$) {
+       my ($self, $range) = @_;
+       my $ng = $self->{ng} or return '412 no newsgroup selected';
+       (defined $range && $range =~ /[<>]/) and
+               return '420 No article(s) selected'; # no message IDs
+
+       $range = $self->{article} unless defined $range;
+       my $r = get_range($self, $range);
+       return $r unless ref $r;
+       my ($beg, $end) = @$r;
+       my $mm = $ng->mm;
+       my $srch = $ng->search;
+       more($self, '224 Overview information follows');
+
+       long_response($self, sub {
+               my $h = search_header_for($srch, $beg, 'references');
+               more($self, "$beg $h") if defined($h);
+               $beg++ < $end;
+       });
+}
+
+sub over_line ($$) {
+       my ($num, $smsg) = @_;
+       # n.b. field access and procedural calls can be
+       # 10%-15% faster than OO method calls:
+       my $s = join("\t", $num,
+               $smsg->{subject},
+               $smsg->{from},
+               PublicInbox::SearchMsg::date($smsg),
+               "<$smsg->{mid}>",
+               $smsg->{references},
+               $smsg->{bytes},
+               $smsg->{lines});
+       utf8::encode($s);
+       $s
+}
+
+sub cmd_over ($;$) {
+       my ($self, $range) = @_;
+       if ($range && $range =~ /\A<(.+)>\z/) {
+               my ($ng, $n) = mid_lookup($self, $1);
+               defined $n or return r430;
+               my $smsg = $ng->search->{over_ro}->get_art($n) or return r430;
+               more($self, '224 Overview information follows (multi-line)');
+
+               # Only set article number column if it's the current group
+               my $self_ng = $self->{ng};
+               $n = 0 if (!$self_ng || $self_ng ne $ng);
+               more($self, over_line($n, $smsg));
+               '.';
+       } else {
+               cmd_xover($self, $range);
+       }
+}
+
 sub cmd_xover ($;$) {
        my ($self, $range) = @_;
+       $range = $self->{article} unless defined $range;
        my $r = get_range($self, $range);
        return $r unless ref $r;
        my ($beg, $end) = @$r;
        more($self, "224 Overview information follows for $beg to $end");
-       $self->long_response($beg, $end, sub {
-               my ($i) = @_;
-               my $r = $self->art_lookup($$i, 2);
-               return unless ref $r;
-               more($self, join("\t", $r->[0],
-                               map {
-                                       my $h = xhdr($r, $_);
-                                       defined $h ? $h : '';
-                               } @OVERVIEW ));
+       my $srch = $self->{ng}->search;
+       my $cur = $beg;
+       long_response($self, sub {
+               my $msgs = $srch->query_xover($cur, $end);
+               my $nr = scalar @$msgs or return;
+
+               # OVERVIEW.FMT
+               more($self, join("\r\n", map {
+                       over_line($_->{num}, $_);
+                       } @$msgs));
+               $cur = $msgs->[-1]->{num} + 1;
        });
 }
 
+sub cmd_xpath ($$) {
+       my ($self, $mid) = @_;
+       return r501 unless $mid =~ /\A<(.+)>\z/;
+       $mid = $1;
+       my @paths;
+       foreach my $ng (values %{$self->{nntpd}->{groups}}) {
+               my $n = $ng->mm->num_for($mid);
+               push @paths, "$ng->{newsgroup}/$n" if defined $n;
+       }
+       return '430 no such article on server' unless @paths;
+       '223 '.join(' ', @paths);
+}
+
 sub res ($$) {
        my ($self, $line) = @_;
        do_write($self, $line . "\r\n");
@@ -566,7 +879,7 @@ sub more ($$) {
 sub do_write ($$) {
        my ($self, $data) = @_;
        my $done = $self->write($data);
-       die if $self->{closed};
+       return if $self->{closed};
 
        # Do not watch for readability if we have data in the queue,
        # instead re-enable watching for readability when we can
@@ -575,6 +888,16 @@ sub do_write ($$) {
        $done;
 }
 
+sub err ($$;@) {
+       my ($self, $fmt, @args) = @_;
+       printf { $self->{nntpd}->{err} } $fmt."\n", @args;
+}
+
+sub out ($$;@) {
+       my ($self, $fmt, @args) = @_;
+       printf { $self->{nntpd}->{out} } $fmt."\n", @args;
+}
+
 use constant MSG_MORE => ($^O eq 'linux') ? 0x8000 : 0;
 
 sub do_more ($$) {
@@ -582,43 +905,80 @@ sub do_more ($$) {
        if (MSG_MORE && !$self->{write_buf_size}) {
                my $n = send($self->{sock}, $data, MSG_MORE);
                if (defined $n) {
-                       my $dlen = bytes::length($data);
+                       my $dlen = length($data);
                        return 1 if $n == $dlen; # all done!
-                       $data = bytes::substr($data, $n, $dlen - $n);
+                       $data = substr($data, $n, $dlen - $n);
                }
        }
-       $self->do_write($data);
+       do_write($self, $data);
 }
 
-# callbacks for by Danga::Socket
+# callbacks for Danga::Socket
 
 sub event_hup { $_[0]->close }
 sub event_err { $_[0]->close }
 
 sub event_write {
        my ($self) = @_;
+       update_idle_time($self);
        # only continue watching for readability when we are done writing:
-       $self->write(undef) == 1 and $self->watch_read(1);
+       if ($self->write(undef) == 1 && !$self->{long_res}) {
+               $self->watch_read(1);
+       }
 }
 
 sub event_read {
        my ($self) = @_;
        use constant LINE_MAX => 512; # RFC 977 section 2.3
        my $r = 1;
+
        my $buf = $self->read(LINE_MAX) or return $self->close;
-       while ($r > 0 && $$buf =~ s/\A([^\r\n]+)\r?\n//) {
+       $self->{rbuf} .= $$buf;
+       while ($r > 0 && $self->{rbuf} =~ s/\A\s*([^\r\n]+)\r?\n//) {
                my $line = $1;
-               my $t0 = [ gettimeofday ];
-               $r = eval { $self->process_line($line) };
-               printf(STDERR "$line %0.6f\n",
-                       tv_interval($t0, [gettimeofday]));
+               return $self->close if $line =~ /[[:cntrl:]]/s;
+               my $t0 = now();
+               my $fd = $self->{fd};
+               $r = eval { process_line($self, $line) };
+               my $d = $self->{long_res} ?
+                       " deferred[$fd]" : '';
+               out($self, "[$fd] %s - %0.6f$d", $line, now() - $t0);
        }
+
        return $self->close if $r < 0;
-       my $len = bytes::length($$buf);
+       my $len = length($self->{rbuf});
        return $self->close if ($len >= LINE_MAX);
-       $self->push_back_read($buf) if ($len);
+       update_idle_time($self);
+}
+
+sub watch_read {
+       my ($self, $bool) = @_;
+       my $rv = $self->SUPER::watch_read($bool);
+       if ($bool && $self->{rbuf} ne '') {
+               # Force another read if there is a pipelined request.
+               # We don't know if the socket has anything for us to read,
+               # and we must double-check again by the time the timer fires
+               # in case we really did dispatch a read event and started
+               # another long response.
+               push @$nextq, $self;
+               $nextt ||= PublicInbox::EvCleanup::asap(*next_tick);
+       }
+       $rv;
+}
+
+sub not_idle_long ($$) {
+       my ($self, $now) = @_;
+       defined(my $fd = $self->{fd}) or return;
+       my $ary = $EXPMAP->{$fd} or return;
+       my $exp_at = $ary->[0] + $EXPTIME;
+       $exp_at > $now;
 }
 
-sub warning { print STDERR @_, "\n" }
+# for graceful shutdown in PublicInbox::Daemon:
+sub busy {
+       my ($self, $now) = @_;
+       ($self->{rbuf} ne '' || $self->{long_res} || $self->{write_buf_size} ||
+        not_idle_long($self, $now));
+}
 
 1;