GitHub user trotterdylan opened a pull request:
https://github.com/apache/thrift/pull/1411
Fix remote client for HTTP transport
This change fixes two issues:
- Assign parsedUrl to the variable in the outer scope instead of creating a
new one. Previously the outer parsedUrl was never assigned and was therefore
always empty.
- Create a POST client using NewTHttpPostClient instead of NewTHttpClient
since the latter has no requestBuffer to write to the request body.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trotterdylan/thrift patch-1
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1411.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1411
----
commit acbc50f5a3ab559cdb0937986709388350282d87
Author: Dylan Trotter <[email protected]>
Date: 2017-11-07T17:56:17Z
Fix remote client for HTTP transport
This change fixes two issues:
- Assign parsedUrl to the variable in the outer scope instead of creating a
new one. Previously the outer parsedUrl was never assigned and was therefore
always empty.
- Create a POST client using NewTHttpPostClient instead of NewTHttpClient
since the latter has no requestBuffer to write to the request body.
----
---
