From dc77752eb7bfe528c60dc0ac028717877342e22a Mon Sep 17 00:00:00 2001
From: Eric Wong <normalperson@yhbt.net>
Date: Thu, 30 Dec 2010 08:32:21 +0000
Subject: coolio_thread_*: fix 416 response handling

After beefing up and enabling byte range tests for "sendfile"
(and no just IO.copy_stream), we noticed threaded-Coolio
variants did not handle invalid byte ranges correctly.
---
 t/byte-range-common.sh | 118 +++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 118 insertions(+)
 create mode 100644 t/byte-range-common.sh

(limited to 't/byte-range-common.sh')

diff --git a/t/byte-range-common.sh b/t/byte-range-common.sh
new file mode 100644
index 0000000..120aad6
--- /dev/null
+++ b/t/byte-range-common.sh
@@ -0,0 +1,118 @@
+t_begin "byte-range setup vars" && {
+	random_blob_size=$(wc -c < random_blob)
+	rb_1=$(( $random_blob_size - 1 ))
+	range_head=-r-365
+	range_tail=-r155-
+	range_mid=-r200-300
+	range_n1=-r0-$rb_1
+	range_n2=-r0-$(($rb_1 - 1))
+	range_1b_head=-r0-0
+	range_1b_tail=-r$rb_1-$rb_1
+	range_1b_mid=-r200-200
+	range_all=-r0-$random_blob_size
+	url=http://$listen/random_blob
+}
+
+check_content_range () {
+	grep 'Range:' $err
+	# Content-Range: bytes #{offset}-#{offset+count-1}/#{clen}
+	d='\([0-9]\+\)'
+	start= end= size=
+	eval $(< $err sed -n -e \
+	  "s/^< Content-Range: bytes $d-$d\/$d"'.*$/start=\1 end=\2 size=\3/p')
+	test -n "$start"
+	test -n "$end"
+	test -n "$size"
+
+	# ensure we didn't screw up the sed invocation
+	expect="< Content-Range: bytes $start-$end/$size"
+	test x"$(grep -F "$expect" $err)" = x"$(grep '^< Content-Range:' $err)"
+
+	test $start -le $end
+	test $end -lt $size
+}
+
+t_begin "read random blob sha1s" && {
+	sha1_head=$(curl -sSff $range_head file://random_blob | rsha1)
+	sha1_tail=$(curl -sSff $range_tail file://random_blob | rsha1)
+	sha1_mid=$(curl -sSff $range_mid file://random_blob | rsha1)
+	sha1_n1=$(curl -sSff $range_n1 file://random_blob | rsha1)
+	sha1_n2=$(curl -sSff $range_n2 file://random_blob | rsha1)
+	sha1_1b_head=$(curl -sSff $range_1b_head file://random_blob | rsha1)
+	sha1_1b_tail=$(curl -sSff $range_1b_tail file://random_blob | rsha1)
+	sha1_1b_mid=$(curl -sSff $range_1b_mid file://random_blob | rsha1)
+	sha1_all=$(rsha1 < random_blob)
+	echo "$sha1_all=$sha1_n1"
+}
+
+t_begin "normal full request matches" && {
+	sha1="$(curl -v 2>$err -sSf $url | rsha1)"
+	test x"$sha1_all" = x"$sha1"
+	grep 'Content-Range:' $err && die "Content-Range unexpected"
+	grep 'HTTP/1.1 200 OK' $err || die "200 response expected"
+}
+
+t_begin "crazy offset goes over" && {
+	range_insane=-r$(($random_blob_size * 2))-$(($random_blob_size * 4))
+	curl -vsS 2>$err $range_insane $url
+	grep 'HTTP/1\.[01] 416 ' $err || die "expected 416 error"
+}
+
+t_begin "full request matches with explicit ranges" && {
+	sha1="$(curl -v 2>$err $range_all -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_all" = x"$sha1"
+
+	sha1="$(curl -v 2>$err $range_n1 -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_all" = x"$sha1"
+
+	range_over=-r0-$(($random_blob_size * 2))
+	sha1="$(curl -v 2>$err $range_over -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_all" = x"$sha1"
+}
+
+t_begin "no fence post errors" && {
+	sha1="$(curl -v 2>$err $range_n2 -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_n2" = x"$sha1"
+
+	sha1="$(curl -v 2>$err $range_1b_head -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_1b_head" = x"$sha1"
+
+	sha1="$(curl -v 2>$err $range_1b_tail -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_1b_tail" = x"$sha1"
+
+	sha1="$(curl -v 2>$err $range_1b_mid -sSf $url | rsha1)"
+	check_content_range
+	test x"$sha1_1b_mid" = x"$sha1"
+}
+
+t_begin "head range matches" && {
+	sha1="$(curl -sSfv 2>$err $range_head $url | rsha1)"
+	check_content_range
+	test x"$sha1_head" = x"$sha1"
+}
+
+t_begin "tail range matches" && {
+	sha1="$(curl -sSfv 2>$err $range_tail $url | rsha1)"
+	check_content_range
+	test x"$sha1_tail" = x"$sha1"
+}
+
+t_begin "mid range matches" && {
+	sha1="$(curl -sSfv 2>$err $range_mid $url | rsha1)"
+	check_content_range
+	test x"$sha1_mid" = x"$sha1"
+}
+
+t_begin "shutdown server" && {
+	kill -QUIT $rainbows_pid
+}
+
+t_begin "check stderr" && check_stderr
+
+t_done
-- 
cgit v1.2.3-24-ge0c7