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

remove the overwritten header in AsyncHttpClient constructor #650

Merged
merged 1 commit into from Aug 2, 2014

Conversation

Projects
None yet
3 participants
Contributor

zuoa commented Jul 23, 2014

when I call addHeader to change "Content-Type" header,the issue emerges.

like this:

POST /xampp/ HTTP/1.1
Content-Length: 99
Content-Type: application/x-www-form-urlencoded
Host: 192.168.22.199
Connection: Keep-Alive
Accept-Encoding: gzip
Content-Type: application/json

Collaborator

smarek commented Jul 23, 2014

That's strange, header should be overwritten by implementation, I'll add it as precaution measure.
Thanks for reporting.

@fineswap fineswap added this to the 1.4.6 milestone Jul 24, 2014

@smarek smarek was assigned by fineswap Jul 24, 2014

@smarek smarek added a commit that referenced this pull request Aug 2, 2014

@smarek smarek Merge pull request #650 from JedYu/master
remove the overwritten header in AsyncHttpClient  constructor
e66e8e6

@smarek smarek merged commit e66e8e6 into loopj:master Aug 2, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment