search

UIKit0 / mod-spdy

Automatically exported from code.google.com/p/mod-spdy
0 stars 0 forks source link

Some response streams are truncated (chunked encoding?) #17

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
It seems that some response streams end up getting truncated. For instance on 
our test server the file:
https://ec2-107-21-132-32.compute-1.amazonaws.com/wp-admin/load-styles.php?c=0&d
ir=ltr&load=admin-bar,wp-admin

gets served truncated, and the Chrome browser refuses to save even a byte of it 
to disk.

If you fetch the same resource over HTTP:

http://ec2-107-21-132-32.compute-1.amazonaws.com/wp-admin/load-styles.php?c=0&di
r=ltr&load=admin-bar,wp-admin

it works just fine.

I'm guessing that this may be due to some chunked encoding issues we haven't 
worked out yet.

Original issue reported on code.google.com by bmcqu...@google.com on 21 Dec 2011 at 8:50

GoogleCodeExporter commented 8 years ago 
I've implemented chunked encoding in r215.  Can you sync and see if this fixes 
the issue?  If not there may be some other problem.

Original comment by mdste...@google.com on 22 Dec 2011 at 3:56

GoogleCodeExporter commented 8 years ago 
I just tested this against head and it is still happening. Bumping to 
Priority-High. I can share a repro case with you on Monday.

Original comment by bmcqu...@google.com on 6 Feb 2012 at 1:26

GoogleCodeExporter commented 8 years ago 
The issue seems to be that the HTTP-to-SPDY output filter is failing to empty 
out the brigade passed to it before returning success, but mod_deflate is 
assuming that it will do so; as a result, the first half of the compressed data 
gets sent to the browser _twice_, and so of course decompression fails halfway 
through.  I don't know for sure if clearing the brigade is a contract of Apache 
output filters that we failed to fulfill, or if mod_deflate is making a bad 
assumption, but I assume it's our fault.  Either way, the fix is simple; I'll 
get that patched soon.

Original comment by mdste...@google.com on 7 Feb 2012 at 9:31

GoogleCodeExporter commented 8 years ago 
Fixed in r238.

Original comment by mdste...@google.com on 7 Feb 2012 at 10:12