Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Invalid chunked response on "Content-Length: 0" #8

Closed
nihen opened this Issue Jun 1, 2010 · 2 comments

Comments

Projects
None yet
2 participants

nihen commented Jun 1, 2010

%starman -e "sub {return ['302', ['Content-Length' => '0'], ['']]}"

%telnet localhost 5000
GET / HTTP/1.1
Host: example.com

HTTP/1.1 302 Found
Content-Length: 0
Transfer-Encoding: chunked
Date: Tue, 01 Jun 2010 10:47:56 GMT
Connection: keep-alive

0

patch and test code is http://gist.github.com/420748

Owner

miyagawa commented Jun 1, 2010

Thanks for the report - i'm wondering if it's better to use Plack::Util::status_with_no_entity_body here because actually, 302 with C-L: 0 is invalid per HTTP/1.1 spec.

Owner

miyagawa commented Jul 3, 2010

Fixed an invalid chunked response when Content-Length is 0. Closed by 9454ac7

clkao pushed a commit to clkao/Starman that referenced this issue Jun 30, 2012

clkao pushed a commit to clkao/Starman that referenced this issue Jun 30, 2012

Checking in changes prior to tagging of version 0.2006.
Changelog diff is:

diff --git a/Changes b/Changes
index d84de99..e7f8cfd 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,9 @@
 Revision history for Perl extension Starman

+0.2006 Fri Jul  2 17:21:22 PDT 2010
+        - Fixed a bug in chunked response when Content-Length is 0. #8 (chiba)
+        - Documented --pid and --daemonize
+
 0.2005 Fri Jul  2 17:02:16 PDT 2010
         - Don't use lib 'lib'
         - Documentation updates (miyagawa, grantm)

clkao pushed a commit to clkao/Starman that referenced this issue Jun 30, 2012

Checking in changes prior to tagging of version 0.2007.
Changelog diff is:

diff --git a/Changes b/Changes
index e7f8cfd..91ec48a 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,8 @@
 Revision history for Perl extension Starman

+0.2007 Thu Sep 30 14:09:00 PDT 2010
+        - Fixed a bug where Date header can be duplicate if the app generates one (spleenjack)
+
 0.2006 Fri Jul  2 17:21:22 PDT 2010
         - Fixed a bug in chunked response when Content-Length is 0. #8 (chiba)
         - Documented --pid and --daemonize

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment