view http_location_auto.t @ 572:ca54b445d982

Tests: masked nginx bug in proxy next upstream. When iterating through several next upstreams per a worker cycle, a previously reported event about upstream connection error may be improperly applied to the next upstream, thus leading to the invalid connection error. E.g., in kqueue, where the problem is visible, this is caused by "ev->instance" that does not tolerate more than one next upstream at once, and kevents placed on the kqueue separately for read and write events. The change is to limit test case to the only one next upstream.
author Sergey Kandaurov <pluknet@nginx.com>
date Thu, 14 May 2015 11:54:24 +0300
parents 847ea345becb
children e9064d691790
line wrap: on
line source

#!/usr/bin/perl

# (C) Maxim Dounin

# Tests for location selection, an auto_redirect edge case.

###############################################################################

use warnings;
use strict;

use Test::More;

BEGIN { use FindBin; chdir($FindBin::Bin); }

use lib 'lib';
use Test::Nginx;

###############################################################################

select STDERR; $| = 1;
select STDOUT; $| = 1;

my $t = Test::Nginx->new()->has(qw/http proxy rewrite/)->plan(4)
	->write_file_expand('nginx.conf', <<'EOF');

%%TEST_GLOBALS%%

daemon off;

events {
}

http {
    %%TEST_GLOBALS_HTTP%%

    server {
        listen       127.0.0.1:8080;
        server_name  localhost;

        proxy_hide_header X-Location;
        add_header X-Location unset;

        # As of nginx 1.5.4, this results in the following
        # location tree:
        #
        #         "/a-b"
        # "/a-a"          "/a/"
        #
        # A request to "/a" is expected to match "/a/" with auto_redirect,
        # but with such a tree it tests locations "/a-b", "/a-a" and then
        # falls back to null location.
        #
        # Key factor is that "-" is less than "/".

        location /a/  { proxy_pass http://127.0.0.1:8080/a-a; }
        location /a-a { add_header X-Location a-a; return 204; }
        location /a-b { add_header X-Location a-b; return 204; }
    }
}

EOF

$t->run();

###############################################################################

like(http_get('/a'), qr/301 Moved/, 'auto redirect');
like(http_get('/a/'), qr/X-Location: unset/, 'match a');
like(http_get('/a-a'), qr/X-Location: a-a/, 'match a-a');
like(http_get('/a-b'), qr/X-Location: a-b/, 'match a-b');

###############################################################################