* [PATCH 3/4] proxy_pass: expand pipelining tests for after-upload behavior
@ 2015-04-04 1:08 7% ` Eric Wong
0 siblings, 0 replies; 1+ results
From: Eric Wong @ 2015-04-04 1:08 UTC (permalink / raw)
To: yahns-public
Pipelining uploads is rare in practice, but they must behave
properly in case some brave soul wants to start doing it.
---
test/test_proxy_pass.rb | 51 +++++++++++++++++++++++++++++++++++--------------
1 file changed, 37 insertions(+), 14 deletions(-)
diff --git a/test/test_proxy_pass.rb b/test/test_proxy_pass.rb
index cebd004..163a0f7 100644
--- a/test/test_proxy_pass.rb
+++ b/test/test_proxy_pass.rb
@@ -286,21 +286,44 @@ def check_pipelining(host, port)
r2 << pl.readpartial(666)
end
- if false
- pl.write "ET / HTTP/1.1\r\nHost: example.com\r\n\r\n"
- until r3 =~ /hi\n/
- r3 << pl.readpartial(666)
- end
- else
- pl.write "UT / HTTP/1.1\r\nHost: example.com\r\n"
- pl.write "Transfer-Encoding: chunked\r\n\r\n"
- pl.write "6\r\nchunky\r\n"
- pl.write "0\r\n\r\n"
-
- until r3 =~ /chunky/
- r3 << pl.readpartial(666)
- end
+ pl.write "UT / HTTP/1.1\r\nHost: example.com\r\n"
+ pl.write "Transfer-Encoding: chunked\r\n\r\n"
+ pl.write "6\r\nchunky\r\n"
+ pl.write "0\r\n\r\n"
+
+ until r3 =~ /chunky/
+ r3 << pl.readpartial(666)
+ end
+
+ # ensure stuff still works after a chunked upload:
+ pl.write "GET / HTTP/1.1\r\nHost: example.com\r\n\r\nP"
+ after_up = ''
+ until after_up =~ /hi\n/
+ after_up << pl.readpartial(666)
+ end
+ re = /^Date:[^\r\n]+/
+ assert_equal after_up.sub(re, ''), r1.sub(re, '')
+
+ # another upload, this time without chunking
+ pl.write "UT / HTTP/1.1\r\nHost: example.com\r\n"
+ pl.write "Content-Length: 8\r\n\r\n"
+ pl.write "identity"
+ identity = ''
+
+ until identity =~ /identity/
+ identity << pl.readpartial(666)
+ end
+ assert_match %r{identity\z}, identity
+ assert_match %r{\AHTTP/1\.1 201\b}, identity
+
+ # ensure stuff still works after an identity upload:
+ pl.write "GET / HTTP/1.1\r\nHost: example.com\r\n\r\n"
+ after_up = ''
+ until after_up =~ /hi\n/
+ after_up << pl.readpartial(666)
end
+ re = /^Date:[^\r\n]+/
+ assert_equal after_up.sub(re, ''), r1.sub(re, '')
end
r1 = r1.split("\r\n").reject { |x| x =~ /^Date: / }
r2 = r2.split("\r\n").reject { |x| x =~ /^Date: / }
--
EW
^ permalink raw reply related [relevance 7%]
Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2015-04-04 1:08 [PATCH 0/4] proxy_pass updates Eric Wong
2015-04-04 1:08 7% ` [PATCH 3/4] proxy_pass: expand pipelining tests for after-upload behavior Eric Wong
Code repositories for project(s) associated with this public inbox
https://yhbt.net/yahns.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).