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

Issue with gzip compressed page #3

Closed
fpietrosanti opened this Issue Apr 1, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Contributor

fpietrosanti commented Apr 1, 2012

It seems that t2w is not working properly for compressed pages.

TEST URL: https://g6lfrbqd3krju3ek.tor2web.org/ .

KO: Does not work with a browser (safari/firefox):

KO: Does not work with curl with enabled compression
curl -v -v --compressed https://g6lfrbqd3krju3ek.tor2web.org/

But it works with Curl/Wget default parameter: (no compression by default)
curl -v -v https://g6lfrbqd3krju3ek.tor2web.org/

So it seems that there is something related to gzip compression.

Collaborator

hellais commented Apr 5, 2012

This is actually not a problem with encoding, but it's related to Transfer-Encoding: chunked.

The naive thing I did is advertise that I only support HTTP 1.0, though web servers are not so kind to respect me :(.

This is a known issue with the twisted parts that I am using and some people have figured out a way to make it work, but it requires a little bit of effort.

I am going to dump here some related links and work on it as soon as I fix the other bugs:

http://groups.google.com/group/notify-io/browse_thread/thread/d84ce1f8909b22cd?pli=1

In particular tx-pendrell appears to be a good choise, though it is very poorly documented (even worse than twisted :P), but they have an implementation of a client that works with chunked transfer encoding. It needs some work to get it to fit to tor2web though:
https://github.com/olix0r/tx-pendrell/blob/master/lib/decoders.py

hellais added a commit that referenced this issue Apr 5, 2012

Collaborator

hellais commented Apr 5, 2012

I found a workaround for this. If the client advertised to not support gzip nginx will not send transfer-encoding: chunked. I will for the moment close this ticket.

@hellais hellais closed this Apr 5, 2012

hellais added a commit that referenced this issue Dec 1, 2012

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