Mercurial > hg > nginx-tests
annotate proxy_cache_vary.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 | 907e89fba9c3 |
children | 2cd00179f4b2 |
rev | line source |
---|---|
471 | 1 #!/usr/bin/perl |
2 | |
3 # (C) Maxim Dounin | |
4 | |
5 # Tests for http proxy cache, the Vary header. | |
6 | |
7 ############################################################################### | |
8 | |
9 use warnings; | |
10 use strict; | |
11 | |
12 use Test::More; | |
13 | |
14 BEGIN { use FindBin; chdir($FindBin::Bin); } | |
15 | |
16 use lib 'lib'; | |
17 use Test::Nginx; | |
18 | |
19 ############################################################################### | |
20 | |
21 select STDERR; $| = 1; | |
22 select STDOUT; $| = 1; | |
23 | |
24 plan(skip_all => 'win32') if $^O eq 'MSWin32'; | |
25 | |
568
907e89fba9c3
Tests: removed TODO and try_run() checks for legacy versions.
Sergey Kandaurov <pluknet@nginx.com>
parents:
507
diff
changeset
|
26 my $t = Test::Nginx->new()->has(qw/http proxy cache gzip rewrite/)->plan(42) |
471 | 27 ->write_file_expand('nginx.conf', <<'EOF'); |
28 | |
29 %%TEST_GLOBALS%% | |
30 | |
31 daemon off; | |
32 | |
33 events { | |
34 } | |
35 | |
36 http { | |
37 %%TEST_GLOBALS_HTTP%% | |
38 | |
39 proxy_cache_path %%TESTDIR%%/cache keys_zone=one:1m inactive=5s; | |
40 proxy_cache_key $uri; | |
41 | |
42 server { | |
43 listen 127.0.0.1:8080; | |
44 server_name localhost; | |
45 | |
46 add_header X-Cache-Status $upstream_cache_status; | |
47 | |
48 location / { | |
49 proxy_pass http://127.0.0.1:8081/; | |
50 proxy_cache one; | |
51 } | |
52 | |
53 location /replace/ { | |
54 proxy_pass http://127.0.0.1:8081/; | |
55 proxy_cache one; | |
56 } | |
57 | |
58 location /revalidate/ { | |
59 proxy_pass http://127.0.0.1:8081/; | |
60 proxy_cache one; | |
61 proxy_cache_revalidate on; | |
62 } | |
63 | |
64 location /ignore/ { | |
65 proxy_pass http://127.0.0.1:8081/; | |
66 proxy_cache one; | |
67 proxy_ignore_headers Vary; | |
68 } | |
69 } | |
70 | |
71 server { | |
72 listen 127.0.0.1:8081; | |
73 server_name localhost; | |
74 | |
75 gzip on; | |
76 gzip_min_length 0; | |
77 gzip_http_version 1.0; | |
78 gzip_vary on; | |
79 | |
80 expires 2s; | |
81 | |
82 location / { | |
83 if ($args = "novary") { | |
84 return 200 "the only variant\n"; | |
85 } | |
86 } | |
87 | |
88 location /asterisk { | |
89 gzip off; | |
90 add_header Vary "*"; | |
91 } | |
92 | |
93 location /complex { | |
94 gzip off; | |
95 add_header Vary ",, Accept-encoding , ,"; | |
96 } | |
97 } | |
98 } | |
99 | |
100 EOF | |
101 | |
102 $t->write_file('index.html', 'SEE-THIS'); | |
103 $t->write_file('asterisk', 'SEE-THIS'); | |
104 $t->write_file('complex', 'SEE-THIS'); | |
105 | |
568
907e89fba9c3
Tests: removed TODO and try_run() checks for legacy versions.
Sergey Kandaurov <pluknet@nginx.com>
parents:
507
diff
changeset
|
106 $t->run(); |
471 | 107 |
108 ############################################################################### | |
109 | |
110 like(get('/', 'gzip'), qr/MISS/ms, 'first request'); | |
111 like(get('/', 'gzip'), qr/HIT/ms, 'vary match cached'); | |
112 like(get('/', 'deflate'), qr/MISS/ms, 'vary mismatch'); | |
113 like(get('/', 'deflate'), qr/HIT/ms, 'vary mismatch cached'); | |
114 like(get('/', 'foo'), qr/MISS/ms, 'vary mismatch 2'); | |
115 like(get('/', 'foo'), qr/HIT/ms, 'vary mismatch 2 cached'); | |
116 like(get('/', 'gzip'), qr/HIT/ms, 'multiple representations cached'); | |
117 | |
118 SKIP: { | |
119 skip 'long tests', 6 unless $ENV{TEST_NGINX_UNSAFE}; | |
120 | |
121 # make sure all variants are properly expire | |
122 # and removed after inactive timeout | |
123 | |
124 sleep(3); | |
125 | |
126 like(get('/', 'gzip'), qr/EXPIRED/ms, 'first expired'); | |
127 like(get('/', 'deflate'), qr/EXPIRED/ms, 'second variant expired'); | |
128 | |
129 like(get('/', 'gzip'), qr/HIT/ms, 'first cached after expire'); | |
130 like(get('/', 'deflate'), qr/HIT/ms, 'second cached after expire'); | |
131 | |
132 sleep(12); | |
133 | |
134 like(get('/', 'gzip'), qr/MISS/ms, 'first inactive removed'); | |
135 like(get('/', 'deflate'), qr/MISS/ms, 'second variant removed'); | |
136 | |
137 } | |
138 | |
139 SKIP: { | |
140 skip 'long tests', 6 unless $ENV{TEST_NGINX_UNSAFE}; | |
141 | |
142 # check if the variant which was loaded first will be properly | |
143 # removed if it's not requested (but another variant is requested | |
144 # at the same time) | |
145 | |
146 sleep(3); | |
147 like(get('/', 'deflate'), qr/EXPIRED/ms, 'bump1'); | |
148 sleep(3); | |
149 like(get('/', 'deflate'), qr/EXPIRED/ms, 'bump2'); | |
150 sleep(3); | |
151 like(get('/', 'deflate'), qr/EXPIRED/ms, 'bump3'); | |
152 sleep(3); | |
153 like(get('/', 'deflate'), qr/EXPIRED/ms, 'bump4'); | |
154 | |
155 TODO: { | |
156 local $TODO = 'not yet'; | |
157 | |
476
4e335141aa4b
Tests: fix head() remnants in Vary tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
471
diff
changeset
|
158 like(get('/', 'gzip'), qr/MISS/ms, 'first not bumped by second requests'); |
471 | 159 |
160 } | |
161 | |
476
4e335141aa4b
Tests: fix head() remnants in Vary tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
471
diff
changeset
|
162 like(get('/', 'deflate'), qr/HIT/ms, 'second variant cached'); |
471 | 163 |
164 } | |
165 | |
166 # if a response without Vary is returned to replace previously returned | |
167 # responses with Vary, make sure it is then used in all cases | |
168 | |
169 like(get('/replace/', 'gzip'), qr/MISS/, 'replace first'); | |
170 like(get('/replace/', 'deflate'), qr/MISS/, 'replace second'); | |
171 | |
172 sleep(3); | |
173 | |
174 like(get('/replace/?novary', 'deflate'), qr/EXPIRED/, 'replace novary'); | |
175 like(get('/replace/?zztest', 'gzip'), qr/HIT/, 'all replaced'); | |
176 | |
177 # make sure revalidation of variants works fine | |
178 | |
179 like(get('/revalidate/', 'gzip'), qr/MISS/, 'revalidate first'); | |
180 like(get('/revalidate/', 'deflate'), qr/MISS/, 'revalidate second'); | |
181 | |
182 sleep(3); | |
183 | |
184 like(get('/revalidate/', 'gzip'), qr/REVALIDATED/, 'revalidated first'); | |
185 like(get('/revalidate/', 'deflate'), qr/REVALIDATED/, 'revalidated second'); | |
186 like(get('/revalidate/', 'gzip'), qr/HIT/, 'revalidate first after'); | |
187 like(get('/revalidate/', 'deflate'), qr/HIT/, 'revalidate second after'); | |
188 | |
189 # if the Vary header is ignored, cached version can be returned | |
190 # regardless of request headers | |
191 | |
192 like(get('/ignore/', 'gzip'), qr/MISS/ms, 'another request'); | |
193 like(get('/ignore/', 'deflate'), qr/HIT/ms, 'vary ignored'); | |
194 | |
195 # check parsing of Vary with multiple headers listed | |
196 | |
197 like(get('/complex', 'gzip'), qr/MISS/ms, 'vary complex first'); | |
198 like(get('/complex', 'deflate'), qr/MISS/ms, 'vary complex second'); | |
199 like(get('/complex', 'gzip'), qr/HIT/ms, 'vary complex first cached'); | |
200 like(get('/complex', 'deflate'), qr/HIT/ms, 'vary complex second cached'); | |
201 | |
202 # From RFC 7231, "7.1.4. Vary", | |
203 # http://tools.ietf.org/html/rfc7231#section-7.1.4: | |
204 # | |
205 # A Vary field value of "*" signals that anything about the request | |
206 # might play a role in selecting the response representation, possibly | |
207 # including elements outside the message syntax (e.g., the client's | |
208 # network address). A recipient will not be able to determine whether | |
209 # this response is appropriate for a later request without forwarding | |
210 # the request to the origin server. | |
211 # | |
212 # In theory, If-None-Match can be used to check if the representation | |
213 # present in the cache is appropriate. This seems to be only possible | |
214 # with strong entity tags though, as representation with different | |
215 # content condings may share the same weak entity tag. | |
216 | |
217 like(get('/asterisk', 'gzip'), qr/MISS/ms, 'vary asterisk first'); | |
218 like(get('/asterisk', 'gzip'), qr/MISS/ms, 'vary asterisk second'); | |
219 | |
220 # From RFC 7234, "4.1. Calculating Secondary Keys with Vary", | |
221 # http://tools.ietf.org/html/rfc7234#section-4.1: | |
222 # | |
223 # The selecting header fields from two requests are defined to match if | |
224 # and only if those in the first request can be transformed to those in | |
225 # the second request by applying any of the following: | |
226 # | |
227 # o adding or removing whitespace, where allowed in the header field's | |
228 # syntax | |
507 | 229 # |
471 | 230 # o combining multiple header fields with the same field name (see |
231 # Section 3.2 of [RFC7230]) | |
507 | 232 # |
471 | 233 # o normalizing both header field values in a way that is known to |
234 # have identical semantics, according to the header field's | |
235 # specification (e.g., reordering field values when order is not | |
236 # significant; case-normalization, where values are defined to be | |
237 # case-insensitive) | |
489
3036e3af0e08
Tests: more Vary normalization tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
481
diff
changeset
|
238 # |
3036e3af0e08
Tests: more Vary normalization tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
481
diff
changeset
|
239 # Only whitespace normalization is currently implemented. |
471 | 240 |
241 like(get('/', 'foo, bar'), qr/MISS/ms, 'normalize first'); | |
481
7e823c8f7d31
Tests: adjusted TODOs for cache Vary support committed in 1.7.7.
Sergey Kandaurov <pluknet@nginx.com>
parents:
478
diff
changeset
|
242 like(get('/', 'foo,bar'), qr/HIT/ms, 'normalize whitespace'); |
7e823c8f7d31
Tests: adjusted TODOs for cache Vary support committed in 1.7.7.
Sergey Kandaurov <pluknet@nginx.com>
parents:
478
diff
changeset
|
243 like(get('/', 'foo,, ,bar , '), qr/HIT/ms, 'normalize empty'); |
489
3036e3af0e08
Tests: more Vary normalization tests.
Maxim Dounin <mdounin@mdounin.ru>
parents:
481
diff
changeset
|
244 like(get('/', 'foobar'), qr/MISS/ms, 'normalize no whitespace mismatch'); |
471 | 245 |
246 TODO: { | |
247 local $TODO = 'not yet'; | |
248 | |
249 like(get('/', 'bar,foo'), qr/HIT/ms, 'normalize order'); | |
250 | |
251 } | |
252 | |
253 ############################################################################### | |
254 | |
255 sub get { | |
256 my ($url, $extra) = @_; | |
257 return http(<<EOF); | |
258 GET $url HTTP/1.1 | |
259 Host: localhost | |
260 Connection: close | |
261 Accept-Encoding: $extra | |
262 | |
263 EOF | |
264 } | |
265 | |
266 ############################################################################### |