annotate http_try_files.t @ 1961:fe6f22da53ec default tip

Tests: tests for usage of discarded body. The client_max_body_size limit should be ignored when the request body is already discarded. In HTTP/1.x, this is done by checking the r->discard_body flag when the body is being discarded, and because r->headers_in.content_length_n is 0 when it's already discarded. This, however, does not happen with HTTP/2 and HTTP/3, and therefore "error_page 413" does not work without relaxing the limit. Further, with proxy_pass, r->headers_in.content_length_n is used to determine length of the request body, and therefore is not correct if discarding of the request body isn't yet complete. While discarding the request body, r->headers_in.content_length_n contains the rest of the body to discard (or, in case of chunked request body, the rest of the current chunk to discard). Similarly, the $content_length variable uses r->headers_in.content_length if available, and also incorrect. The $content_length variable is used when proxying with fastcgi_pass, grpc_pass, and uwsgi_pass (scgi_pass uses the value calculated based on the actual request body buffers, and therefore works correctly).
author Maxim Dounin <mdounin@mdounin.ru>
date Sat, 27 Apr 2024 18:55:50 +0300
parents 882267679006
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
1 #!/usr/bin/perl
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
2
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
3 # (C) Maxim Dounin
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
4
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
5 # Tests for try_files directive.
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
6
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
7 ###############################################################################
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
8
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
9 use warnings;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
10 use strict;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
11
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
12 use Test::More;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
13
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
14 BEGIN { use FindBin; chdir($FindBin::Bin); }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
15
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
16 use lib 'lib';
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
17 use Test::Nginx;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
18
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
19 ###############################################################################
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
20
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
21 select STDERR; $| = 1;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
22 select STDOUT; $| = 1;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
23
658
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
24 my $t = Test::Nginx->new()->has(qw/http proxy rewrite/)->plan(10)
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
25 ->write_file_expand('nginx.conf', <<'EOF');
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
26
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
27 %%TEST_GLOBALS%%
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
28
249
6a0d934950bc Tests: remove extra spaces in "daemon off".
Maxim Dounin <mdounin@mdounin.ru>
parents: 224
diff changeset
29 daemon off;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
30
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
31 events {
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
32 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
33
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
34 http {
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
35 %%TEST_GLOBALS_HTTP%%
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
36
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
37 server {
974
882267679006 Tests: simplified parallel modifications in tests.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 952
diff changeset
38 listen 127.0.0.1:8080;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
39 server_name localhost;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
40
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
41 location / {
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
42 try_files $uri /fallback;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
43 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
44
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
45 location /nouri/ {
659
f3b5bee3a14e Tests: replaced underscores by hyphens in http_try_files.t.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 658
diff changeset
46 try_files $uri /fallback-nouri;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
47 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
48
214
35d050d7d5f1 Tests: add try_files test with short uri (ticket #152).
Maxim Dounin <mdounin@mdounin.ru>
parents: 189
diff changeset
49 location /short/ {
35d050d7d5f1 Tests: add try_files test with short uri (ticket #152).
Maxim Dounin <mdounin@mdounin.ru>
parents: 189
diff changeset
50 try_files /short $uri =404;
35d050d7d5f1 Tests: add try_files test with short uri (ticket #152).
Maxim Dounin <mdounin@mdounin.ru>
parents: 189
diff changeset
51 }
35d050d7d5f1 Tests: add try_files test with short uri (ticket #152).
Maxim Dounin <mdounin@mdounin.ru>
parents: 189
diff changeset
52
523
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
53 location /file-file/ {
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
54 try_files /found.html =404;
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
55 }
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
56
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
57 location /file-dir/ {
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
58 try_files /found.html/ =404;
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
59 }
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
60
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
61 location /dir-dir/ {
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
62 try_files /directory/ =404;
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
63 }
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
64
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
65 location /dir-file/ {
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
66 try_files /directory =404;
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
67 }
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
68
659
f3b5bee3a14e Tests: replaced underscores by hyphens in http_try_files.t.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 658
diff changeset
69 location ~ /alias-re.html {
635
bc91b86a71ee Tests: added test for try_files in regex location with alias.
Sergey Kandaurov <pluknet@nginx.com>
parents: 568
diff changeset
70 alias %%TESTDIR%%/directory;
bc91b86a71ee Tests: added test for try_files in regex location with alias.
Sergey Kandaurov <pluknet@nginx.com>
parents: 568
diff changeset
71 try_files $uri =404;
bc91b86a71ee Tests: added test for try_files in regex location with alias.
Sergey Kandaurov <pluknet@nginx.com>
parents: 568
diff changeset
72 }
bc91b86a71ee Tests: added test for try_files in regex location with alias.
Sergey Kandaurov <pluknet@nginx.com>
parents: 568
diff changeset
73
658
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
74 location /alias-nested/ {
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
75 alias %%TESTDIR%%/;
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
76 location ~ html {
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
77 try_files $uri =404;
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
78 }
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
79 }
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
80
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
81 location /fallback {
974
882267679006 Tests: simplified parallel modifications in tests.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 952
diff changeset
82 proxy_pass http://127.0.0.1:8081/fallback;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
83 }
659
f3b5bee3a14e Tests: replaced underscores by hyphens in http_try_files.t.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 658
diff changeset
84 location /fallback-nouri {
974
882267679006 Tests: simplified parallel modifications in tests.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 952
diff changeset
85 proxy_pass http://127.0.0.1:8081;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
86 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
87 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
88
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
89 server {
974
882267679006 Tests: simplified parallel modifications in tests.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 952
diff changeset
90 listen 127.0.0.1:8081;
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
91 server_name localhost;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
92
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
93 location / {
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
94 add_header X-URI $request_uri;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
95 return 204;
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
96 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
97 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
98 }
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
99
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
100 EOF
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
101
523
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
102 mkdir($t->testdir() . '/directory');
659
f3b5bee3a14e Tests: replaced underscores by hyphens in http_try_files.t.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 658
diff changeset
103 $t->write_file('directory/alias-re.html', 'SEE THIS');
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
104 $t->write_file('found.html', 'SEE THIS');
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
105 $t->run();
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
106
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
107 ###############################################################################
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
108
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
109 like(http_get('/found.html'), qr!SEE THIS!, 'found');
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
110 like(http_get('/uri/notfound'), qr!X-URI: /fallback!, 'not found uri');
224
ba992cfdc606 Tests: remove TODO from tests which pass in 1.2.x.
Maxim Dounin <mdounin@mdounin.ru>
parents: 214
diff changeset
111 like(http_get('/nouri/notfound'), qr!X-URI: /fallback!, 'not found nouri');
214
35d050d7d5f1 Tests: add try_files test with short uri (ticket #152).
Maxim Dounin <mdounin@mdounin.ru>
parents: 189
diff changeset
112 like(http_get('/short/long'), qr!404 Not!, 'short uri in try_files');
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
113
523
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
114 like(http_get('/file-file/'), qr!SEE THIS!, 'file matches file');
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
115 like(http_get('/file-dir/'), qr!404 Not!, 'file does not match dir');
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
116 like(http_get('/dir-dir/'), qr!301 Moved Permanently!, 'dir matches dir');
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
117 like(http_get('/dir-file/'), qr!404 Not!, 'dir does not match file');
64eabe6aa1f2 Tests: try_files tests for files and directory matching.
Damien Tournoud <damien@commerceguys.com>
parents: 310
diff changeset
118
659
f3b5bee3a14e Tests: replaced underscores by hyphens in http_try_files.t.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 658
diff changeset
119 like(http_get('/alias-re.html'), qr!SEE THIS!, 'alias in regex location');
658
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
120 like(http_get('/alias-nested/found.html'), qr!SEE THIS!,
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
121 'alias with nested location');
b9eb462d8973 Tests: added try_files test for alias with nested location.
Andrey Zelenkov <zelenkov@nginx.com>
parents: 657
diff changeset
122
189
802fc0786165 Tests: some try_files tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
diff changeset
123 ###############################################################################