Mercurial > hg > nginx-tests
view http_expect_100_continue.t @ 1951:1867428f1673
Tests: fixed h3_limit_req.t spurious failures.
In the "reset stream - cancellation" test, HTTP/3 stream is closed without
sending the request body when the request is waiting in the limit_req
module, and this results in error 444. However, when the request is received
with some minor delay due to system load, it is not delayed by limit_req,
and the stream is closed during reading the request body, which results
in error 400 instead, breaking the test.
Fix is to introduce yet another request before the "reset stream" test,
so the stream in question is always delayed by limit_req.
author | Maxim Dounin <mdounin@mdounin.ru> |
---|---|
date | Thu, 14 Mar 2024 02:25:49 +0300 |
parents | 882267679006 |
children |
line wrap: on
line source
#!/usr/bin/perl # (C) Maxim Dounin # Tests for Expect: 100-continue support. ############################################################################### 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/)->plan(5); $t->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; location / { proxy_pass http://127.0.0.1:8080/local; } location /local { } } } EOF $t->run(); ############################################################################### like(http_100_request('/', '1.1'), qr/ 100 /, 'expect 100 continue'); # Comparison of expectation values is case-insensitive for unquoted tokens. like(http_100_request('/', '1.1', '100-Continue'), qr/ 100 /, 'expect 100 continue case-insensitive'); # From RFC 2616, 8.2.3 Use of the 100 (Continue) Status: # # - An origin server SHOULD NOT send a 100 (Continue) response if # the request message does not include an Expect request-header # field with the "100-continue" expectation, and MUST NOT send a # 100 (Continue) response if such a request comes from an HTTP/1.0 # (or earlier) client. unlike(http_100_request('/', '1.0'), qr/ 100 /, 'no 100 continue via http 1.0'); # From RFC 2616, 14.20 Expect: # # A server that does not understand or is unable to comply with any of # the expectation values in the Expect field of a request MUST respond # with appropriate error status. The server MUST respond with a 417 # (Expectation Failed) status if any of the expectations cannot be met. # # <..> If a server receives a request containing an # Expect field that includes an expectation-extension that it does not # support, it MUST respond with a 417 (Expectation Failed) status. TODO: { local $TODO = 'not yet'; like(http_100_request('/', '1.1', 'unknown'), qr/ 417 /, 'unknown expectation'); like(http_100_request('/', '1.1', 'token=param'), qr/ 417 /, 'unsupported expectation extension'); } ############################################################################### sub http_100_request { my ($url, $version, $value) = @_; $value = '100-continue' unless defined $value; http(<<EOF); POST $url HTTP/$version Host: localhost Expect: $value Content-Length: 0 Connection: close EOF } ###############################################################################