# HG changeset patch # User Sergey Kandaurov # Date 1631713052 -10800 # Node ID 696322b7e2c3a05cddba1d9f40a9da0da37692f9 # Parent 14b5ee6eee6c968beb189724d30e57fa166b1361 Tests: added proxy_max_temp_file_size tests. diff --git a/h2_proxy_max_temp_file_size.t b/h2_proxy_max_temp_file_size.t new file mode 100644 --- /dev/null +++ b/h2_proxy_max_temp_file_size.t @@ -0,0 +1,101 @@ +#!/usr/bin/perl + +# (C) Sergey Kandaurov +# (C) Nginx, Inc. + +# Tests for http proxy module, proxy_max_temp_file_size directive. + +############################################################################### + +use warnings; +use strict; + +use Test::More; + +BEGIN { use FindBin; chdir($FindBin::Bin); } + +use lib 'lib'; +use Test::Nginx; +use Test::Nginx::HTTP2; + +############################################################################### + +select STDERR; $| = 1; +select STDOUT; $| = 1; + +my $t = Test::Nginx->new()->has(qw/http http_v2 proxy/)->plan(4); + +$t->write_file_expand('nginx.conf', <<'EOF'); + +%%TEST_GLOBALS%% + +daemon off; + +events { +} + +http { + %%TEST_GLOBALS_HTTP%% + + server { + listen 127.0.0.1:8080 http2; + server_name localhost; + + proxy_buffer_size 4k; + proxy_buffers 8 4k; + + location / { + proxy_max_temp_file_size 4k; + proxy_pass http://127.0.0.1:8081/; + } + + location /off/ { + proxy_max_temp_file_size 0; + proxy_pass http://127.0.0.1:8081/; + } + } + + server { + listen 127.0.0.1:8081; + server_name localhost; + + location / { } + } +} + +EOF + +$t->write_file('1', 'X' x (1024 * 1024)); +$t->run(); + +############################################################################### + +# test that the response is wholly proxied when all event pipe buffers are full + +my $s = Test::Nginx::HTTP2->new(); +my $sid = $s->new_stream({ path => '/1' }); + +select undef, undef, undef, 0.4; +$s->h2_window(1024 * 1024, $sid); +$s->h2_window(1024 * 1024); + +my $frames = $s->read(all => [{ sid => $sid, fin => 1 }]); +my $body = join '', map { $_->{data} } grep { $_->{type} eq "DATA" } @$frames; +like($body, qr/^X+$/m, 'no pipe bufs - body'); +is(length($body), 1024 * 1024, 'no pipe bufs - body length'); + +# also with disabled proxy temp file + +$s = Test::Nginx::HTTP2->new(); +$sid = $s->new_stream({ path => '/off/1' }); + +select undef, undef, undef, 0.4; +$s->h2_window(1024 * 1024, $sid); +$s->h2_window(1024 * 1024); + +$frames = $s->read(all => [{ sid => $sid, fin => 1 }]); +$body = join '', map { $_->{data} } grep { $_->{type} eq "DATA" } @$frames; +like($body, qr/^X+$/m, 'no temp file - body'); +is(length($body), 1024 * 1024, 'no temp file - body length'); + +############################################################################### diff --git a/proxy_max_temp_file_size.t b/proxy_max_temp_file_size.t new file mode 100644 --- /dev/null +++ b/proxy_max_temp_file_size.t @@ -0,0 +1,84 @@ +#!/usr/bin/perl + +# (C) Sergey Kandaurov +# (C) Nginx, Inc. + +# Tests for http proxy module, proxy_max_temp_file_size directive. + +############################################################################### + +use warnings; +use strict; + +use Test::More; + +BEGIN { use FindBin; chdir($FindBin::Bin); } + +use lib 'lib'; +use Test::Nginx qw/ :DEFAULT http_content /; + +############################################################################### + +select STDERR; $| = 1; +select STDOUT; $| = 1; + +my $t = Test::Nginx->new()->has(qw/http proxy/)->plan(4); + +$t->write_file_expand('nginx.conf', <<'EOF'); + +%%TEST_GLOBALS%% + +daemon off; + +events { +} + +http { + %%TEST_GLOBALS_HTTP%% + + server { + listen 127.0.0.1:8080 sndbuf=32k; + server_name localhost; + + proxy_buffer_size 4k; + proxy_buffers 8 4k; + + location / { + proxy_max_temp_file_size 4k; + proxy_pass http://127.0.0.1:8081/; + } + + location /off/ { + proxy_max_temp_file_size 0; + proxy_pass http://127.0.0.1:8081/; + } + } + + server { + listen 127.0.0.1:8081; + server_name localhost; + + location / { } + } +} + +EOF + +$t->write_file('1', 'X' x (1024 * 1024)); +$t->run(); + +############################################################################### + +# test that the response is wholly proxied when all event pipe buffers are full + +my $body = http_content(http_get('/1', sleep => 0.4)); +like($body, qr/^X+$/m, 'no pipe bufs - body'); +is(length($body), 1024 * 1024, 'no pipe bufs - body length'); + +# also with disabled proxy temp file + +$body = http_content(http_get('/off/1', sleep => 0.4)); +like($body, qr/^X+$/m, 'no temp file - body'); +is(length($body), 1024 * 1024, 'no temp file - body length'); + +###############################################################################