]> Sergey Matveev's repositories - public-inbox.git/commitdiff
split out NNTPD and HTTPD* modules
authorEric Wong <e@80x24.org>
Mon, 25 Apr 2016 05:12:43 +0000 (05:12 +0000)
committerEric Wong <e@80x24.org>
Mon, 25 Apr 2016 05:26:37 +0000 (05:26 +0000)
Hopefully this modularizes things a little and allows us
to work on a combined super server to save RAM.

lib/PublicInbox/HTTPD.pm [new file with mode: 0644]
lib/PublicInbox/HTTPD/Async.pm [new file with mode: 0644]
lib/PublicInbox/NNTPD.pm [new file with mode: 0644]
script/public-inbox-httpd
script/public-inbox-nntpd

diff --git a/lib/PublicInbox/HTTPD.pm b/lib/PublicInbox/HTTPD.pm
new file mode 100644 (file)
index 0000000..78efaa5
--- /dev/null
@@ -0,0 +1,46 @@
+# Copyright (C) 2016 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+
+package PublicInbox::HTTPD;
+use strict;
+use warnings;
+use Plack::Util;
+require PublicInbox::HTTPD::Async;
+require PublicInbox::Daemon;
+
+sub pi_httpd_async {
+       my ($io, $cb) = @_;
+       PublicInbox::HTTPD::Async->new($io, $cb);
+}
+
+sub new {
+       my ($class, $sock, $app) = @_;
+       my $n = getsockname($sock) or die "not a socket: $sock $!\n";
+       my ($host, $port) = PublicInbox::Daemon::host_with_port($n);
+
+       my %env = (
+               SERVER_NAME => $host,
+               SERVER_PORT => $port,
+               SCRIPT_NAME => '',
+               'psgi.version' => [ 1, 1 ],
+               'psgi.errors' => \*STDERR,
+               'psgi.url_scheme' => 'http',
+               'psgi.nonblocking' => Plack::Util::TRUE,
+               'psgi.streaming' => Plack::Util::TRUE,
+               'psgi.run_once'  => Plack::Util::FALSE,
+               'psgi.multithread' => Plack::Util::FALSE,
+               'psgi.multiprocess' => Plack::Util::TRUE,
+               'psgix.harakiri'=> Plack::Util::FALSE,
+               'psgix.input.buffered' => Plack::Util::TRUE,
+               'pi-httpd.async' => do {
+                       no warnings 'once';
+                       *pi_httpd_async
+               },
+       );
+       bless {
+               app => $app,
+               env => \%env
+       }, $class;
+}
+
+1;
diff --git a/lib/PublicInbox/HTTPD/Async.pm b/lib/PublicInbox/HTTPD/Async.pm
new file mode 100644 (file)
index 0000000..6398502
--- /dev/null
@@ -0,0 +1,35 @@
+# Copyright (C) 2016 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+#
+# XXX This is a totally unstable API for public-inbox internal use only
+# This is exposed via the 'pi-httpd.async' key in the PSGI env hash.
+# The name of this key is not even stable!
+# Currently is is intended for use with read-only pipes.
+package PublicInbox::HTTPD::Async;
+use strict;
+use warnings;
+use base qw(Danga::Socket);
+use fields qw(cb);
+
+sub new {
+       my ($class, $io, $cb) = @_;
+       my $self = fields::new($class);
+       IO::Handle::blocking($io, 0);
+       $self->SUPER::new($io);
+       $self->{cb} = $cb;
+       $self->watch_read(1);
+       $self;
+}
+
+sub event_read { $_[0]->{cb}->() }
+sub event_hup { $_[0]->{cb}->() }
+sub event_err { $_[0]->{cb}->() }
+sub sysread { shift->{sock}->sysread(@_) }
+
+sub close {
+       my $self = shift;
+       $self->{cb} = undef;
+       $self->SUPER::close(@_);
+}
+
+1;
diff --git a/lib/PublicInbox/NNTPD.pm b/lib/PublicInbox/NNTPD.pm
new file mode 100644 (file)
index 0000000..85109ea
--- /dev/null
@@ -0,0 +1,59 @@
+# Copyright (C) 2016 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
+
+# represents an NNTPD (currently a singleton),
+# see script/public-inbox-nntpd for how it is used
+package PublicInbox::NNTPD;
+use strict;
+use warnings;
+require PublicInbox::NewsGroup;
+require PublicInbox::Config;
+
+sub new {
+       my ($class) = @_;
+       bless {
+               groups => {},
+               err => \*STDERR,
+               out => \*STDOUT,
+               grouplist => [],
+       }, $class;
+}
+
+sub refresh_groups () {
+       my ($self) = @_;
+       my $pi_config = PublicInbox::Config->new;
+       my $new = {};
+       my @list;
+       foreach my $k (keys %$pi_config) {
+               $k =~ /\Apublicinbox\.([^\.]+)\.mainrepo\z/ or next;
+               my $g = $1;
+               my $git_dir = $pi_config->{$k};
+               my $addr = $pi_config->{"publicinbox.$g.address"};
+               my $ngname = $pi_config->{"publicinbox.$g.newsgroup"};
+               if (defined $ngname) {
+                       next if ($ngname eq ''); # disabled
+                       $g = $ngname;
+               }
+               my $ng = PublicInbox::NewsGroup->new($g, $git_dir, $addr);
+               my $old_ng = $self->{groups}->{$g};
+
+               # Reuse the old one if possible since it can hold
+               # references to valid mm and gcf objects
+               if ($old_ng) {
+                       $old_ng->update($ng);
+                       $ng = $old_ng;
+               }
+
+               # Only valid if msgmap and search works
+               if ($ng->usable) {
+                       $new->{$g} = $ng;
+                       push @list, $ng;
+               }
+       }
+       @list = sort { $a->{name} cmp $b->{name} } @list;
+       $self->{grouplist} = \@list;
+       # this will destroy old groups that got deleted
+       %{$self->{groups}} = %$new;
+}
+
+1;
index 3ca974c9a780e1b101661691930a407a4d2345ef..b29effcccb4ff25c3966de527b271e92253cbaa3 100755 (executable)
@@ -8,6 +8,7 @@ use warnings;
 use Plack::Util;
 use PublicInbox::Daemon;
 use PublicInbox::HTTP;
+use PublicInbox::HTTPD;
 use Plack::Request;
 use Plack::Builder;
 my %httpds;
@@ -54,80 +55,3 @@ PublicInbox::Daemon::run('0.0.0.0:8080', $refresh,
                my $h = $httpds{$fd} ||= PublicInbox::HTTPD->new($srv, $app);
                PublicInbox::HTTP->new($client, $addr, $h),
        });
-
-1;
-
-# XXX This is a totally unstable API for public-inbox internal use only
-# This is exposed via the 'pi-httpd.async' key in the PSGI env hash.
-# The name of this key is not even stable!
-# Currently is is intended for use with read-only pipes.
-package PublicInbox::HTTPD::Async;
-use strict;
-use warnings;
-use base qw(Danga::Socket);
-use fields qw(cb);
-
-sub new {
-       my ($class, $io, $cb) = @_;
-       my $self = fields::new($class);
-       IO::Handle::blocking($io, 0);
-       $self->SUPER::new($io);
-       $self->{cb} = $cb;
-       $self->watch_read(1);
-       $self;
-}
-
-sub event_read { $_[0]->{cb}->() }
-sub event_hup { $_[0]->{cb}->() }
-sub event_err { $_[0]->{cb}->() }
-sub sysread { shift->{sock}->sysread(@_) }
-
-sub close {
-       my $self = shift;
-       $self->{cb} = undef;
-       $self->SUPER::close(@_);
-}
-
-1;
-
-package PublicInbox::HTTPD;
-use strict;
-use warnings;
-use Plack::Util;
-
-sub pi_httpd_async {
-       my ($io, $cb) = @_;
-       PublicInbox::HTTPD::Async->new($io, $cb);
-}
-
-sub new {
-       my ($class, $sock, $app) = @_;
-       my $n = getsockname($sock) or die "not a socket: $sock $!\n";
-       my ($host, $port) = PublicInbox::Daemon::host_with_port($n);
-
-       my %env = (
-               SERVER_NAME => $host,
-               SERVER_PORT => $port,
-               SCRIPT_NAME => '',
-               'psgi.version' => [ 1, 1 ],
-               'psgi.errors' => \*STDERR,
-               'psgi.url_scheme' => 'http',
-               'psgi.nonblocking' => Plack::Util::TRUE,
-               'psgi.streaming' => Plack::Util::TRUE,
-               'psgi.run_once'  => Plack::Util::FALSE,
-               'psgi.multithread' => Plack::Util::FALSE,
-               'psgi.multiprocess' => Plack::Util::TRUE,
-               'psgix.harakiri'=> Plack::Util::FALSE,
-               'psgix.input.buffered' => Plack::Util::TRUE,
-               'pi-httpd.async' => do {
-                       no warnings 'once';
-                       *pi_httpd_async
-               },
-       );
-       bless {
-               app => $app,
-               env => \%env,
-       }, $class;
-}
-
-1;
index cea881632ca5e6318d471df78e212ff1bc83d6df..be860a54c223724a2d0e76467b6c8e5dcb708b1c 100755 (executable)
@@ -6,64 +6,9 @@
 use strict;
 use warnings;
 require PublicInbox::Daemon;
-require PublicInbox::NewsGroup;
 require PublicInbox::NNTP;
-require PublicInbox::Config;
+require PublicInbox::NNTPD;
 my $nntpd = PublicInbox::NNTPD->new;
 PublicInbox::Daemon::run('0.0.0.0:119',
        sub { $nntpd->refresh_groups }, # refresh
        sub ($$$) { PublicInbox::NNTP->new($_[0], $nntpd) }); # post_accept
-
-1;
-package PublicInbox::NNTPD;
-use strict;
-use warnings;
-
-sub new {
-       my ($class) = @_;
-       bless {
-               groups => {},
-               err => \*STDERR,
-               out => \*STDOUT,
-               grouplist => [],
-       }, $class;
-}
-
-sub refresh_groups () {
-       my ($self) = @_;
-       my $pi_config = PublicInbox::Config->new;
-       my $new = {};
-       my @list;
-       foreach my $k (keys %$pi_config) {
-               $k =~ /\Apublicinbox\.([^\.]+)\.mainrepo\z/ or next;
-               my $g = $1;
-               my $git_dir = $pi_config->{$k};
-               my $addr = $pi_config->{"publicinbox.$g.address"};
-               my $ngname = $pi_config->{"publicinbox.$g.newsgroup"};
-               if (defined $ngname) {
-                       next if ($ngname eq ''); # disabled
-                       $g = $ngname;
-               }
-               my $ng = PublicInbox::NewsGroup->new($g, $git_dir, $addr);
-               my $old_ng = $self->{groups}->{$g};
-
-               # Reuse the old one if possible since it can hold
-               # references to valid mm and gcf objects
-               if ($old_ng) {
-                       $old_ng->update($ng);
-                       $ng = $old_ng;
-               }
-
-               # Only valid if msgmap and search works
-               if ($ng->usable) {
-                       $new->{$g} = $ng;
-                       push @list, $ng;
-               }
-       }
-       @list = sort { $a->{name} cmp $b->{name} } @list;
-       $self->{grouplist} = \@list;
-       # this will destroy old groups that got deleted
-       %{$self->{groups}} = %$new;
-}
-
-1;