Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

fix: added back catch of 4 tuple response to transferdecode #7

Merged
merged 1 commit into from

2 participants

@tsloughter

Just ran into a bug when not doing chunked responses due to my removal of a case clause in transfer_decode. Since the Length was simple ignore and I thought I had take it out of any place that returned it I thought it would be fine. I guess I missed some. For now I've just added the clause back.

@benoitc benoitc merged commit 9f5e041 into benoitc:master
@benoitc
Owner

I also added trailer parsing in a553bdc . It was missing somehow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 25, 2012
  1. fix: added back catch of 4 tuple response to transferdecode

    Tristan Sloughter authored
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 0 deletions.
  1. +3 −0  src/hackney_response.erl
View
3  src/hackney_response.erl
@@ -195,6 +195,9 @@ transfer_decode(Data, Client=#client{
{done, Data2, Rest} ->
Client2 = transfer_decode_done(Rest, Client),
content_decode(ContentDecode, Data2, Client2);
+ {done, Data2, _Length, Rest} ->
+ Client2 = transfer_decode_done(Rest, Client),
+ content_decode(ContentDecode, Data2, Client2);
done ->
Client2 = transfer_decode_done(<<>>, Client),
{done, Client2};
Something went wrong with that request. Please try again.