Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 1 | #!/usr/bin/perl |
| 2 | |
| 3 | # (C) Maxim Dounin |
| 4 | |
| 5 | # Test for http backend not closing connection properly after sending full |
| 6 | # reply. This is in fact backend bug, but it seems common, and anyway |
| 7 | # correct handling is required to support persistent connections. |
| 8 | |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 9 | # There are actually 2 nginx problems here: |
| 10 | # |
| 11 | # 1. It doesn't send reply in-time even if got Content-Length and all the data. |
| 12 | # |
| 13 | # 2. If upstream times out some data may be left in input buffer and won't be |
| 14 | # sent to downstream. |
| 15 | |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 16 | ############################################################################### |
| 17 | |
| 18 | use warnings; |
| 19 | use strict; |
| 20 | |
Maxim Dounin | ff9b36a | 2009-12-26 16:06:17 +0300 | [diff] [blame] | 21 | use Test::More; |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 22 | |
| 23 | use IO::Select; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 24 | |
| 25 | BEGIN { use FindBin; chdir($FindBin::Bin); } |
| 26 | |
| 27 | use lib 'lib'; |
| 28 | use Test::Nginx; |
| 29 | |
| 30 | ############################################################################### |
| 31 | |
| 32 | select STDERR; $| = 1; |
| 33 | select STDOUT; $| = 1; |
| 34 | |
Maxim Dounin | ff9b36a | 2009-12-26 16:06:17 +0300 | [diff] [blame] | 35 | my $t = Test::Nginx->new()->has(qw/http proxy/)->plan(4); |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 36 | |
| 37 | $t->write_file_expand('nginx.conf', <<'EOF'); |
| 38 | |
Maxim Dounin | c6acedb | 2009-10-14 02:23:52 +0400 | [diff] [blame] | 39 | %%TEST_GLOBALS%% |
| 40 | |
Maxim Dounin | 35773f6 | 2013-01-24 02:17:36 +0400 | [diff] [blame] | 41 | daemon off; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 42 | |
| 43 | events { |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 44 | } |
| 45 | |
| 46 | http { |
Maxim Dounin | c6acedb | 2009-10-14 02:23:52 +0400 | [diff] [blame] | 47 | %%TEST_GLOBALS_HTTP%% |
Maxim Dounin | e3aaba5 | 2008-10-16 19:16:46 +0400 | [diff] [blame] | 48 | |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 49 | server { |
Maxim Dounin | 3c47220 | 2008-11-02 15:01:41 +0300 | [diff] [blame] | 50 | listen 127.0.0.1:8080; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 51 | server_name localhost; |
| 52 | |
| 53 | location / { |
Maxim Dounin | 3c47220 | 2008-11-02 15:01:41 +0300 | [diff] [blame] | 54 | proxy_pass http://127.0.0.1:8081; |
Sergey Kandaurov | 48036f2 | 2015-10-23 16:24:53 +0300 | [diff] [blame] | 55 | proxy_read_timeout 2s; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 56 | } |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 57 | |
| 58 | location /uselen { |
Maxim Dounin | 3c47220 | 2008-11-02 15:01:41 +0300 | [diff] [blame] | 59 | proxy_pass http://127.0.0.1:8081; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 60 | |
| 61 | # test will wait only 2s for reply, we it will fail if |
| 62 | # Content-Length not used as a hint |
Maxim Dounin | 35f59e2 | 2009-11-24 13:57:43 +0300 | [diff] [blame] | 63 | |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 64 | proxy_read_timeout 10s; |
| 65 | } |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 66 | } |
| 67 | } |
| 68 | |
| 69 | EOF |
| 70 | |
| 71 | $t->run_daemon(\&http_noclose_daemon); |
Maxim Dounin | 5f3f7ae | 2013-06-08 07:02:11 +0400 | [diff] [blame] | 72 | $t->run()->waitforsocket('127.0.0.1:8081'); |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 73 | |
| 74 | ############################################################################### |
| 75 | |
Maxim Dounin | 323cf24 | 2008-10-11 10:58:43 +0400 | [diff] [blame] | 76 | like(http_get('/'), qr/SEE-THIS/, 'request to bad backend'); |
| 77 | like(http_get('/multi'), qr/AND-THIS/, 'bad backend - multiple packets'); |
Maxim Dounin | 323cf24 | 2008-10-11 10:58:43 +0400 | [diff] [blame] | 78 | like(http_get('/uselen'), qr/SEE-THIS/, 'content-length actually used'); |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 79 | |
Maxim Dounin | ebdcfc6 | 2012-06-03 14:47:10 +0400 | [diff] [blame] | 80 | TODO: { |
| 81 | local $TODO = 'not yet'; |
| 82 | local $SIG{__WARN__} = sub {}; |
| 83 | |
| 84 | like(http_get('/nolen'), qr/SEE-THIS/, 'bad backend - no content length'); |
| 85 | |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 86 | } |
| 87 | |
| 88 | ############################################################################### |
| 89 | |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 90 | sub http_noclose_daemon { |
| 91 | my $server = IO::Socket::INET->new( |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 92 | Proto => 'tcp', |
| 93 | LocalAddr => '127.0.0.1:8081', |
| 94 | Listen => 5, |
| 95 | Reuse => 1 |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 96 | ) |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 97 | or die "Can't create listening socket: $!\n"; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 98 | |
Maxim Dounin | 5f3f7ae | 2013-06-08 07:02:11 +0400 | [diff] [blame] | 99 | local $SIG{PIPE} = 'IGNORE'; |
| 100 | |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 101 | while (my $client = $server->accept()) { |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 102 | $client->autoflush(1); |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 103 | |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 104 | my $multi = 0; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 105 | my $nolen = 0; |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 106 | |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 107 | while (<$client>) { |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 108 | $multi = 1 if /multi/; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 109 | $nolen = 1 if /nolen/; |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 110 | last if (/^\x0d?\x0a?$/); |
| 111 | } |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 112 | |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 113 | if ($nolen) { |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 114 | |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 115 | print $client <<'EOF'; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 116 | HTTP/1.1 200 OK |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 117 | Connection: close |
| 118 | |
| 119 | TEST-OK-IF-YOU-SEE-THIS |
| 120 | EOF |
| 121 | } elsif ($multi) { |
| 122 | |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 123 | print $client <<"EOF"; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 124 | HTTP/1.1 200 OK |
| 125 | Content-Length: 32 |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 126 | Connection: close |
| 127 | |
| 128 | TEST-OK-IF-YOU-SEE-THIS |
| 129 | EOF |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 130 | |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 131 | select undef, undef, undef, 0.1; |
| 132 | print $client 'AND-THIS'; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 133 | |
| 134 | } else { |
| 135 | |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 136 | print $client <<"EOF"; |
Maxim Dounin | 140801b | 2008-09-30 22:58:25 +0400 | [diff] [blame] | 137 | HTTP/1.1 200 OK |
| 138 | Content-Length: 24 |
| 139 | Connection: close |
| 140 | |
| 141 | TEST-OK-IF-YOU-SEE-THIS |
| 142 | EOF |
Maxim Dounin | c68be76 | 2008-09-29 02:10:07 +0400 | [diff] [blame] | 143 | } |
| 144 | |
| 145 | my $select = IO::Select->new($client); |
Maxim Dounin | b56a54c | 2009-04-12 06:46:21 +0400 | [diff] [blame] | 146 | $select->can_read(10); |
| 147 | close $client; |
Maxim Dounin | 53eb4ff | 2008-09-26 18:28:02 +0400 | [diff] [blame] | 148 | } |
| 149 | } |
| 150 | |
| 151 | ############################################################################### |