X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FHTTPD%2FAsync.pm;h=842aaf62e93fbfd22ceabc25269857c9282d1c99;hb=3d41aa23f35501ca92aab8aa42980fa73f7fa74f;hp=bd2eacbfdfee8a5be8f3e9b423907a9ac16b5851;hpb=347c6ee595c37d4e2214cb297811f154a41c452f;p=public-inbox.git
diff --git a/lib/PublicInbox/HTTPD/Async.pm b/lib/PublicInbox/HTTPD/Async.pm
index bd2eacbf..842aaf62 100644
--- a/lib/PublicInbox/HTTPD/Async.pm
+++ b/lib/PublicInbox/HTTPD/Async.pm
@@ -1,4 +1,4 @@
-# Copyright (C) 2016 all contributors
+# Copyright (C) 2016-2018 all contributors
# License: AGPL-3.0+
#
# XXX This is a totally unstable API for public-inbox internal use only
@@ -10,6 +10,7 @@ use strict;
use warnings;
use base qw(Danga::Socket);
use fields qw(cb cleanup);
+require PublicInbox::EvCleanup;
sub new {
my ($class, $io, $cb, $cleanup) = @_;
@@ -22,36 +23,49 @@ sub new {
$self;
}
-sub async_pass {
- my ($self, $io, $fh, $bref) = @_;
- my $restart_read = sub { $self->watch_read(1) };
- # In case the client HTTP connection ($io) dies, it
- # will automatically close this ($self) object.
- $io->{forward} = $self;
- $fh->write($$bref);
- $self->{cb} = sub {
+sub restart_read_cb ($) {
+ my ($self) = @_;
+ sub { $self->watch_read(1) }
+}
+
+sub main_cb ($$$) {
+ my ($http, $fh, $bref) = @_;
+ sub {
+ my ($self) = @_;
my $r = sysread($self->{sock}, $$bref, 8192);
if ($r) {
$fh->write($$bref);
- if ($io->{write_buf_size}) {
+ return if $http->{closed};
+ if ($http->{write_buf_size}) {
$self->watch_read(0);
- $io->write($restart_read); # D::S::write
+ $http->write(restart_read_cb($self));
}
- return; # stay in watch_read
+ # stay in watch_read, but let other clients
+ # get some work done, too.
+ return;
} elsif (!defined $r) {
return if $!{EAGAIN} || $!{EINTR};
}
# Done! Error handling will happen in $fh->close
# called by the {cleanup} handler
- $io->{forward} = undef;
+ $http->{forward} = undef;
$self->close;
}
}
-sub event_read { $_[0]->{cb}->() }
-sub event_hup { $_[0]->{cb}->() }
-sub event_err { $_[0]->{cb}->() }
+sub async_pass {
+ my ($self, $http, $fh, $bref) = @_;
+ # In case the client HTTP connection ($http) dies, it
+ # will automatically close this ($self) object.
+ $http->{forward} = $self;
+ $fh->write($$bref); # PublicInbox:HTTP::{chunked,identity}_wcb
+ $self->{cb} = main_cb($http, $fh, $bref);
+}
+
+sub event_read { $_[0]->{cb}->(@_) }
+sub event_hup { $_[0]->{cb}->(@_) }
+sub event_err { $_[0]->{cb}->(@_) }
sub sysread { shift->{sock}->sysread(@_) }
sub close {
@@ -61,10 +75,7 @@ sub close {
$self->SUPER::close(@_);
# we defer this to the next timer loop since close is deferred
- Danga::Socket->AddTimer(0, $cleanup) if $cleanup;
+ PublicInbox::EvCleanup::next_tick($cleanup) if $cleanup;
}
-# do not let ourselves be closed during graceful termination
-sub busy () { $_[0]->{cb} }
-
1;