]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/HTTP.pm
tls: epollbit: account for miscellaneous OpenSSL errors
[public-inbox.git] / lib / PublicInbox / HTTP.pm
index b19a15d643c23f3c9d19df6df69892d725fa322c..88020ae82438375e42a0854b6c6ae2c4defe2c89 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2016-2019 all contributors <meta@public-inbox.org>
+# Copyright (C) 2016-2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 #
 # Generic PSGI server for convenience.  It aims to provide
@@ -6,12 +6,21 @@
 # to learn different ways to admin both NNTP and HTTP components.
 # There's nothing which depends on public-inbox, here.
 # Each instance of this class represents a HTTP client socket
-
+#
+# fields:
+# httpd: PublicInbox::HTTPD ref
+# env: PSGI env hashref
+# input_left: bytes left to read in request body (e.g. POST/PUT)
+# remote_addr: remote IP address as a string (e.g. "127.0.0.1")
+# remote_port: peer port
+# forward: response body object, response to ->getline + ->close
+# alive: HTTP keepalive state:
+#      0: drop connection when done
+#      1: keep connection when done
+#      2: keep connection, chunk responses
 package PublicInbox::HTTP;
 use strict;
-use warnings;
-use base qw(PublicInbox::DS);
-use fields qw(httpd env input_left remote_addr remote_port forward alive);
+use parent qw(PublicInbox::DS);
 use bytes (); # only for bytes::length
 use Fcntl qw(:seek);
 use Plack::HTTPParser qw(parse_http_request); # XS or pure Perl
@@ -56,20 +65,18 @@ sub http_date () {
 
 sub new ($$$) {
        my ($class, $sock, $addr, $httpd) = @_;
-       my $self = fields::new($class);
+       my $self = bless { httpd => $httpd }, $class;
        my $ev = EPOLLIN;
        my $wbuf;
-       if (ref($sock) eq 'IO::Socket::SSL' && !$sock->accept_SSL) {
+       if ($sock->can('accept_SSL') && !$sock->accept_SSL) {
                return CORE::close($sock) if $! != EAGAIN;
-               $ev = PublicInbox::TLS::epollbit();
+               $ev = PublicInbox::TLS::epollbit() or return CORE::close($sock);
                $wbuf = [ \&PublicInbox::DS::accept_tls_step ];
        }
-       $self->SUPER::new($sock, $ev | EPOLLONESHOT);
-       $self->{httpd} = $httpd;
        $self->{wbuf} = $wbuf if $wbuf;
        ($self->{remote_addr}, $self->{remote_port}) =
                PublicInbox::Daemon::host_with_port($addr);
-       $self;
+       $self->SUPER::new($sock, $ev | EPOLLONESHOT);
 }
 
 sub event_step { # called by PublicInbox::DS
@@ -335,19 +342,31 @@ sub input_tmpfile ($) {
 
 sub input_prepare {
        my ($self, $env) = @_;
-       my $input;
-       my $len = $env->{CONTENT_LENGTH};
-       if ($len) {
-               if ($len > $MAX_REQUEST_BUFFER) {
-                       quit($self, 413);
-                       return;
-               }
-               $input = input_tmpfile($self);
-       } elsif (env_chunked($env)) {
+       my ($input, $len);
+
+       # rfc 7230 3.3.2, 3.3.3,: favor Transfer-Encoding over Content-Length
+       my $hte = $env->{HTTP_TRANSFER_ENCODING};
+       if (defined $hte) {
+               # rfc7230 3.3.3, point 3 says only chunked is accepted
+               # as the final encoding.  Since neither public-inbox-httpd,
+               # git-http-backend, or our WWW-related code uses "gzip",
+               # "deflate" or "compress" as the Transfer-Encoding, we'll
+               # reject them:
+               return quit($self, 400) if $hte !~ /\Achunked\z/i;
+
                $len = CHUNK_START;
                $input = input_tmpfile($self);
        } else {
-               $input = $null_io;
+               $len = $env->{CONTENT_LENGTH};
+               if (defined $len) {
+                       # rfc7230 3.3.3.4
+                       return quit($self, 400) if $len !~ /\A[0-9]+\z/;
+
+                       return quit($self, 413) if $len > $MAX_REQUEST_BUFFER;
+                       $input = $len ? input_tmpfile($self) : $null_io;
+               } else {
+                       $input = $null_io;
+               }
        }
 
        # TODO: expire idle clients on ENFILE / EMFILE
@@ -358,7 +377,7 @@ sub input_prepare {
        $self->{input_left} = $len || 0;
 }
 
-sub env_chunked { ($_[0]->{HTTP_TRANSFER_ENCODING} || '') =~ /\bchunked\b/i }
+sub env_chunked { ($_[0]->{HTTP_TRANSFER_ENCODING} // '') =~ /\Achunked\z/i }
 
 sub err ($$) {
        eval { $_[0]->{httpd}->{env}->{'psgi.errors'}->print($_[1]."\n") };
@@ -451,6 +470,7 @@ sub quit {
        my $h = "HTTP/1.1 $status " . status_message($status) . "\r\n\r\n";
        $self->write(\$h);
        $self->close;
+       undef; # input_prepare expects this
 }
 
 sub close {
@@ -468,6 +488,13 @@ sub busy () {
        ($self->{rbuf} || exists($self->{env}) || $self->{wbuf});
 }
 
+# runs $cb on the next iteration of the event loop at earliest
+sub next_step {
+       my ($self, $cb) = @_;
+       return unless exists $self->{sock};
+       $self->requeue if 1 == push(@{$self->{wbuf}}, $cb);
+}
+
 # Chunked and Identity packages are used for writing responses.
 # They may be exposed to the PSGI application when the PSGI app
 # returns a CODE ref for "push"-based responses