]> Sergey Matveev's repositories - public-inbox.git/blobdiff - t/httpd-corner.t
No ext_urls
[public-inbox.git] / t / httpd-corner.t
index 5dc5734e96d1cb77028641c1ca47e0ad20afe0e8..7600c2b955f27b067d29328d3843832920b56506 100644 (file)
@@ -1,4 +1,5 @@
-# Copyright (C) 2016-2021 all contributors <meta@public-inbox.org>
+#!perl -w
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 # note: our HTTP server should be standalone and capable of running
 # generic PSGI/Plack apps.
@@ -6,7 +7,7 @@ use strict; use v5.10.1; use PublicInbox::TestCommon;
 use Time::HiRes qw(gettimeofday tv_interval);
 use PublicInbox::Spawn qw(spawn popen_rd);
 require_mods(qw(Plack::Util Plack::Builder HTTP::Date HTTP::Status));
-use Digest::SHA qw(sha1_hex);
+use PublicInbox::SHA qw(sha1_hex);
 use IO::Handle ();
 use IO::Socket::UNIX;
 use Fcntl qw(:seek);
@@ -19,7 +20,7 @@ ok(defined mkfifo($fifo, 0777), 'created FIFO');
 my $err = "$tmpdir/stderr.log";
 my $out = "$tmpdir/stdout.log";
 my $psgi = "./t/httpd-corner.psgi";
-my $sock = tcp_server() or die;
+my $sock = tcp_server();
 my @zmods = qw(PublicInbox::GzipFilter IO::Uncompress::Gunzip);
 
 # Make sure we don't clobber socket options set by systemd or similar
@@ -36,7 +37,7 @@ if ($^O eq 'linux') {
        }
 } elsif ($^O eq 'freebsd' && system('kldstat -m accf_data >/dev/null') == 0) {
        require PublicInbox::Daemon;
-       my $var = PublicInbox::Daemon::SO_ACCEPTFILTER();
+       my $var = $PublicInbox::Daemon::SO_ACCEPTFILTER;
        $accf_arg = pack('a16a240', 'dataready', '');
        setsockopt($sock, SOL_SOCKET, $var, $accf_arg) or die "setsockopt: $!";
 }
@@ -53,14 +54,40 @@ sub unix_server ($) {
 
 my $upath = "$tmpdir/s";
 my $unix = unix_server($upath);
+my $alt = tcp_server();
 my $td;
 my $spawn_httpd = sub {
        my (@args) = @_;
-       my $cmd = [ '-httpd', @args, "--stdout=$out", "--stderr=$err", $psgi ];
-       $td = start_script($cmd, undef, { 3 => $sock, 4 => $unix });
+       my $x = tcp_host_port($alt);
+       my $cmd = [ '-httpd', @args, "--stdout=$out", "--stderr=$err", $psgi,
+               '-l', "http://$x/?psgi=t/alt.psgi,env.PI_CONFIG=/path/to/alt".
+                       ",err=$tmpdir/alt.err" ];
+       my $env = { PI_CONFIG => '/dev/null' };
+       $td = start_script($cmd, $env, { 3 => $sock, 4 => $unix, 5 => $alt });
 };
 
 $spawn_httpd->();
+{
+       my $conn = conn_for($alt, 'alt PSGI path');
+       $conn->write("GET / HTTP/1.0\r\n\r\n");
+       $conn->read(my $buf, 4096);
+       like($buf, qr!^/path/to/alt\z!sm,
+               'alt.psgi loaded on alt socket with correct env');
+
+       $conn = conn_for($sock, 'default PSGI path');
+       $conn->write("GET /PI_CONFIG HTTP/1.0\r\n\r\n");
+       $conn->read($buf, 4096);
+       like($buf, qr!^/dev/null\z!sm,
+               'default PSGI on original socket');
+       my $log = capture("$tmpdir/alt.err");
+       ok(grep(/ALT/, @$log), 'alt psgi.errors written to');
+       $log = capture($err);
+       ok(!grep(/ALT/, @$log), 'STDERR not written to');
+       is(unlink($err, "$tmpdir/alt.err"), 2, 'unlinked stderr and alt.err');
+
+       $td->kill('USR1'); # trigger reopen_logs
+}
+
 if ('test worker death') {
        my $conn = conn_for($sock, 'killed worker');
        $conn->write("GET /pid HTTP/1.1\r\nHost:example.com\r\n\r\n");
@@ -82,7 +109,16 @@ if ('test worker death') {
        like($body, qr/\A[0-9]+\z/, '/pid response');
        isnt($body, $pid, 'respawned worker');
 }
-
+{ # check on prior USR1 signal
+       ok(-e $err, 'stderr recreated after USR1');
+       ok(-e "$tmpdir/alt.err", 'alt.err recreated after USR1');
+}
+{
+       my $conn = conn_for($sock, 'Header spaces bogus');
+       $conn->write("GET /empty HTTP/1.1\r\nSpaced-Out : 3\r\n\r\n");
+       $conn->read(my $buf, 4096);
+       like($buf, qr!\AHTTP/1\.[0-9] 400 !, 'got 400 response on bad request');
+}
 {
        my $conn = conn_for($sock, 'streaming callback');
        $conn->write("GET /callback HTTP/1.0\r\n\r\n");
@@ -284,7 +320,7 @@ sub conn_for {
        $spawn_httpd->('-W0');
 }
 
-sub delay { select(undef, undef, undef, shift || rand(0.02)) }
+sub delay { tick(shift || rand(0.02)) }
 
 my $str = 'abcdefghijklmnopqrstuvwxyz';
 my $len = length $str;
@@ -305,7 +341,7 @@ SKIP: {
        my $url = "$base/sha1";
        my ($r, $w);
        pipe($r, $w) or die "pipe: $!";
-       my $cmd = [$curl, qw(--tcp-nodelay -T- -HExpect: -sSN), $url];
+       my $cmd = [$curl, qw(--tcp-nodelay -T- -HExpect: -gsSN), $url];
        open my $cout, '+>', undef or die;
        open my $cerr, '>', undef or die;
        my $rdr = { 0 => $r, 1 => $cout, 2 => $cerr };
@@ -322,7 +358,7 @@ SKIP: {
        seek($cout, 0, SEEK_SET);
        is(<$cout>, sha1_hex($str), 'read expected body');
 
-       my $fh = popen_rd([$curl, '-sS', "$base/async-big"]);
+       my $fh = popen_rd([$curl, '-gsS', "$base/async-big"]);
        my $n = 0;
        my $non_zero = 0;
        while (1) {
@@ -336,12 +372,12 @@ SKIP: {
        is($non_zero, 0, 'read all zeros');
 
        require_mods(@zmods, 4);
-       my $buf = xqx([$curl, '-sS', "$base/psgi-return-gzip"]);
+       my $buf = xqx([$curl, '-gsS', "$base/psgi-return-gzip"]);
        is($?, 0, 'curl succesful');
        IO::Uncompress::Gunzip::gunzip(\$buf => \(my $out));
        is($out, "hello world\n");
        my $curl_rdr = { 2 => \(my $curl_err = '') };
-       $buf = xqx([$curl, qw(-sSv --compressed),
+       $buf = xqx([$curl, qw(-gsSv --compressed),
                        "$base/psgi-return-compressible"], undef, $curl_rdr);
        is($?, 0, 'curl --compressed successful');
        is($buf, "goodbye world\n", 'gzipped response as expected');
@@ -596,7 +632,7 @@ SKIP: {
 SKIP: {
        skip 'SO_ACCEPTFILTER is FreeBSD-only', 1 if $^O ne 'freebsd';
        skip 'accf_data not loaded: kldload accf_data' if !defined $accf_arg;
-       my $var = PublicInbox::Daemon::SO_ACCEPTFILTER();
+       my $var = $PublicInbox::Daemon::SO_ACCEPTFILTER;
        defined(my $x = getsockopt($sock, SOL_SOCKET, $var)) or die;
        is($x, $accf_arg, 'SO_ACCEPTFILTER unchanged if previously set');
 };